Closed
Bug 405350
Opened 17 years ago
Closed 17 years ago
Email is broken on tip, "can't send mail from '1' to '1'"
Categories
(Bugzilla :: Email Notifications, defect)
Tracking
()
RESOLVED
FIXED
Bugzilla 3.2
People
(Reporter: mkanat, Assigned: mkanat)
References
Details
(Keywords: regression)
Attachments
(1 file)
(deleted),
patch
|
LpSolit
:
review+
|
Details | Diff | Splinter Review |
In the recent UTF-8 bug, there's a small regression that completely breaks email sending.
Essentially, utf8::decode changes a scalar in-place, and it returns whether or not it succeeded. It doesn't return the decoded string.
Attachment #290142 -
Flags: review?(LpSolit)
Assignee | ||
Updated•17 years ago
|
Assignee: email-notifications → mkanat
Assignee | ||
Updated•17 years ago
|
Status: NEW → ASSIGNED
Updated•17 years ago
|
Attachment #290142 -
Flags: review?(LpSolit) → review+
Assignee | ||
Comment 1•17 years ago
|
||
Checking in Bugzilla/Mailer.pm;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla/Mailer.pm,v <-- Mailer.pm
new revision: 1.14; previous revision: 1.13
done
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Flags: approval+
Resolution: --- → FIXED
Comment 2•17 years ago
|
||
I'd like to understand why e-mail worked when I tested bug 363153... I tested with the testfile setting, may this be the reason?
Assignee | ||
Comment 3•17 years ago
|
||
(In reply to comment #2)
> I'd like to understand why e-mail worked when I tested bug 363153... I tested
> with the testfile setting, may this be the reason?
Yeah. Testfile doesn't actually try to send an email, so it didn't complain that the address was "1@localhost" or something like that.
You need to log in
before you can comment on or make changes to this bug.
Description
•