Opened 7 years ago
Closed 5 years ago
#1483852 closed Bugs (duplicate)
E-mails aren´t delivered correctly - BAD E-MAIL HEADER
| Reported by: | cyril.dent | Owned by: | |
|---|---|---|---|
| Priority: | 5 | Milestone: | 0.2-beta |
| Component: | Client Scripts | Version: | git-master |
| Severity: | normal | Keywords: | |
| Cc: |
Description
When I send e-mail from webmail to another webmail account, it is delivered correctly. But after delivering to Microsoft Outlook or Outlook Express, it is bad delivered.
This is an example of delivered e-mail to outlook express:
From: (clear)
object: test
e-mail content:
Date: Mon, 19 Jun 2006 10:24:51 +0200
From: <info@…>
Message-ID: <aac450e19d847d2f8ba4c9145ebb8b8f@…>
X-Sender: info@…
User-Agent: RoundCube Webmail/0.1b
Content-Type: text/plain; charset="ISO-8859-2"
Content-Transfer-Encoding: 7bit
I´ve installed version 0.1-beta from 2006-02-19.
I think it´s problem of main.inc.php, it didn´t work.
For example: $rcmail_configmail_header_delimiter? = NULL; or ="\r\n" or ="|", after delivering = no change! :-(
E-mail header isn´t divided correctly.
E-mail source:
Return-Path: <info@…>
X-Original-To: sadlo@…
Delivered-To: sadlo@…
Received: by smtp.sadlo.cz (Postfix, from userid 33)
id 48D0410643EC; Wed, 21 Jun 2006 19:22:53 +0200(CEST)
To: sadlo@…
Subject: test
MIME-Version:1.0[[BR]]
Date: Wed,21 Jun 2006 19:23:53 +0200 From: sadlo <info@…> Message-ID:<ba65babab6a5b5ababa65bab@…> X-Sender: info@… User-Agent:RoundCube Webmail/0.1b Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 8bit X-Antivirus: AVG for E-mail 7.1.394 [268.9.2/370]
test
I think bad CR/LF before "From: sadlo...."
Can you help me?
Attachments (1)
Change History (16)
comment:1 Changed 7 years ago by kwach
comment:2 Changed 7 years ago by adriaan
- Version 0.1-beta deleted
I'm using the default ISPConfig version, when I open the mail in Evolution it's display is correct.
But some ISP's (like demon.nl) don't accept my headers:
HEADER TYPE NOT SUPPORTED
comment:3 Changed 6 years ago by till
- Resolution set to worksforme
- Status changed from new to closed
I am closing this, if the problem perists please update from SVN first and then report back here.
comment:4 Changed 6 years ago by thomasb
This is a PHP mail() issue. Should be fixed in 0.1-rc1 which has auto-selection for header delimiters and also provides a config parameter to override this.
comment:5 Changed 5 years ago by mankytongue
- Resolution worksforme deleted
- Status changed from closed to reopened
This bug is still an issue in [d4cbfabc].
I've had a look at the headers in the sent email, and the problem seems to be the use of line breaks in the "References" header, for example in the following:
...
Organization: Imperial College
In-Reply-To: <652b30b70801210407v5b12fd74t35d25bb602f53f75@…>
References: <652b30b70801181358r63099407wabf82d43e4e0514d@…>
<479479D4.6040909@…>
<214a6f6db48860b71048a0d0fec665fd@…> <652b30b70801210407v5b12fd74t35d25bb602f53f75@…>
Message-ID: <c2e9aa657812eb073afe43711ac709c9@…>
X-Sender: dtw107@…
...
Everything from <479479D4.6040909@…> onwards will appear in the message body. I think the headers should in fact be :
...
Organization: Imperial College
In-Reply-To: <652b30b70801210407v5b12fd74t35d25bb602f53f75@…>
References: <652b30b70801181358r63099407wabf82d43e4e0514d@…> <479479D4.6040909@…> <214a6f6db48860b71048a0d0fec665fd@…> <652b30b70801210407v5b12fd74t35d25bb602f53f75@…>
Message-ID: <c2e9aa657812eb073afe43711ac709c9@…>
X-Sender: dtw107@…
...
(References is one line)
comment:6 Changed 5 years ago by seansan
mankytongue please retest with latest SVN or new release.
comment:7 Changed 5 years ago by seansan
- Milestone set to 0.1.5
- Version set to svn-trunk
Review in 0.1.5 (duplicates exist)
comment:9 Changed 5 years ago by thomasb
Ticket #1484790 marked as duplicate of this bug
comment:10 follow-up: ↓ 11 Changed 5 years ago by ghz
this minor modification solved the REFERENCES header issue for me. anyway it might be against the rfc as i do not really know the exact rules. anyway, what do you think about this kind of sanitization?
--- imap.inc.orig Thu Feb 21 23:43:25 2008
+++ imap.inc Thu Feb 21 23:47:15 2008
@@ -1638,7 +1638,7 @@
$result[$id]->in_reply_to = ereg_replace("[\n<>]", '', $string);
break;
case 'references':
- $result[$id]->references = $string;
+ $result[$id]->references = str_replace("\n", " ", $string);
break;
case 'return-receipt-to':
case 'disposition-notification-to':
comment:11 in reply to: ↑ 10 Changed 5 years ago by A321
This simple fix worked for me.. Now i'm able to have normal e-mail conversations with hotmail and gmail users :) , without header garbage everywhere
Replying to ghz:
this minor modification solved the REFERENCES header issue for me. anyway it might be against the rfc as i do not really know the exact rules. anyway, what do you think about this kind of sanitization?
--- imap.inc.orig Thu Feb 21 23:43:25 2008 +++ imap.inc Thu Feb 21 23:47:15 2008 @@ -1638,7 +1638,7 @@ $result[$id]->in_reply_to = ereg_replace("[\n<>]", '', $string); break; case 'references': - $result[$id]->references = $string; + $result[$id]->references = str_replace("\n", " ", $string); break; case 'return-receipt-to': case 'disposition-notification-to':
comment:12 Changed 5 years ago by carmachy
I have the same problem. If I write a long subject and send into another mail address the mail is broken. Gmail shows the sender as unknown and in the mail I can see the header. Picture attached
Changed 5 years ago by carmachy
comment:13 Changed 5 years ago by till
- Milestone changed from 0.1.1 to 0.2-beta
Postponing it.
If you have the same issue, don't comment "me too". Please provide a sample message'''
comment:14 Changed 5 years ago by sky44
Why the bug will be fix in 0.2-beta ?
The fix has been posted by ghz, so it is easy to fix it in the stable version 0.1 ?
If not, I will do the fix after any update of roudcube until 0.2-beta...
comment:15 Changed 5 years ago by till
- Resolution set to duplicate
- Status changed from reopened to closed
Duplicate of #1484811

the same thing with gmail and evolution
about half of header is thrown into message content