Closed
Bug 1478782
Opened 7 years ago
Closed 7 years ago
PTO app does not send email notifications for me
Categories
(Infrastructure & Operations :: Runtime, task)
Infrastructure & Operations
Runtime
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: mcote, Assigned: rtucker)
Details
(Whiteboard: [kanban:https://webops.kanbanize.com/ctrl_board/2/6807])
For the last while, a few months I think, I always get the "email has not been sent" error when I file PTO. It used to work fine.
If I had to guess, I'd say the non-ASCII characters in my last name are causing some problems, perhaps due to some underlying infra change.
Updated•7 years ago
|
Assignee: server-ops-webops → nobody
Component: WebOps: IT-Managed Tools → Runtime
| Assignee | ||
Comment 1•7 years ago
|
||
We switched servers when moving from SCL3 -> MDC1, but not any code changes.
It does seem as though the issue does have to do with the non-ASCII characters. I'm going to see if there is a way to skip this validation. If not, we'll have to just stop including the name portion in the From: header.
Assignee: nobody → rtucker
| Assignee | ||
Comment 2•7 years ago
|
||
AFter some further testing, ptoapp uses pear::mail.
The native mail() function is much smarter here and allows unicode in the name portion of the envelope.
I'm going to switch to just using native mail()
| Assignee | ||
Comment 3•7 years ago
|
||
I've switched to native mail() and done a test by changing my name to "Rôb Tucker" via phonebook.
The mail logs confirm as well as my manager smani@mozilla.com that the mail has gone through.
Hopefully this is the last of the PHP RFC822 fun, if not, please reopen.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
| Reporter | ||
Comment 4•7 years ago
|
||
Awesome thanks! Will let you know the next time I file PTO (feel free to transfer me some so I can test sooner ;).
| Reporter | ||
Comment 5•7 years ago
|
||
Filed PTO this morning and the email went out fine. Thanks!
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•