Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reply All: doesn't work when To: or Cc: line is folded by Outlook/Exchange #4964

Closed
rcubetrac opened this issue Feb 1, 2016 · 4 comments
Closed

Comments

@rcubetrac
Copy link

Reported by dpc22 on 1 Feb 2016 11:58 UTC as Trac ticket #1490653

https://tools.ietf.org/html/rfc5322

2.2.3.Long Header Fields

says:

 Note: Though structured field bodies are defined in such a way
       that folding can take place between many of the lexical tokens
       (and even within some of the lexical tokens), folding SHOULD be
       limited to placing the CRLF at higher-level syntactic breaks.

Sadly it looks like Outlook/Exchange ignores this SHOULD requirement, and can generate a To: header which looks like:

To: "David Carter @ CAM"
        <[email protected]>, "David Carter @ CAM" <[email protected]>

with a CRLF+TAB sequence before the first "<".

All very well until I try to reply to such a message using Roundcube 1.0.8.

While it looks okay in the editor, Exim complains:

 unqualified recipient rejected: <"David Carter @=CAM">

when I click "send".

My suspicion would be that the CRLF has leaked somehow into the SMTP line recipient list, but I'm not quite sure where to look.

I will attach a raw RFC 5322 message as a test case. We are using Cyrus 2.4(.17) as the mailstore: I don't know if that is significant.

Migrated-From: http://trac.roundcube.net/ticket/1490653

@rcubetrac
Copy link
Author

Comment by @alecpl on 1 Feb 2016 12:51 UTC

The problem is with @ character in recipient name. Confirmed in git-master.

@rcubetrac
Copy link
Author

Milestone changed by @alecpl on 1 Feb 2016 12:51 UTC

later => 1.1.5

@rcubetrac
Copy link
Author

Comment by @alecpl on 1 Feb 2016 13:27 UTC

And it indeed involved headers folding but not in a way you suspected. Fixed in 8dc756f.

@rcubetrac
Copy link
Author

Status changed by @alecpl on 1 Feb 2016 13:27 UTC

new => closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant