#2191 closed defect (fixed)
[patch] Notification mails use LF instead of CRLF
Reported by: | Owned by: | Christopher Lenz | |
---|---|---|---|
Priority: | high | Milestone: | 0.9 |
Component: | ticket system | Version: | 0.9b1 |
Severity: | normal | Keywords: | |
Cc: | m.kooijman@…, nunterberg@… | Branch: | |
Release Notes: | |||
API Changes: | |||
Internal Changes: |
Description
About half the newlines in notify emails are generated as LF instead of CRLF, which causes some mailservers to just remove them and screwing up formatting for the notify message and making email notification not workable. (I have observed a direct path using only exim and/or postfix to work, but when run via exim, ESMTP and fetchmail, it broke).
I have been playing around with a few fixes, some of which refused to work at first for some weird reason. Anyway, I've found two sources of LF:
- The email template is a file with newlines. I think setting svn:eol-style = CRLF on this file should do the trick.
- A few hardcoded '\n' in Notify.py. Replacing these with the CRLF constant fixes that.
Tested with 0.9b1, but there are no relevant changes in b2 or trunk which would fix this.
Priority set to high, since this makes email notification unusable on some systems.
Attachments (2)
Change History (10)
by , 19 years ago
Attachment: | notify_crlf.diff added |
---|
by , 19 years ago
Attachment: | notify_crlf2.diff added |
---|
Same file, but with two lines of debug code removed. Oops…
comment:1 by , 19 years ago
Cc: | added |
---|
comment:2 by , 19 years ago
To show the problem: This how I received the email message of the last change. Notice the missing newlines… (Kinda takes the poing out of the signature, doesn't it? ;-)
comment:3 by , 19 years ago
Milestone: | → 0.9 |
---|---|
Owner: | changed from | to
Status: | new → assigned |
Looks good.
comment:4 by , 19 years ago
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Applied modified patch in [2329]. Please test.
comment:7 by , 19 years ago
Cc: | added |
---|
Patch for this problem, made against 0.9b1, but should apply cleanly against latest trunk