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

Comments

Projects
None yet
1 participant
@rcubetrac
Copy link

commented Feb 1, 2016

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"
        <dpc22+2@cam.ac.uk>, "David Carter @ CAM" <dpc22+3@cam.ac.uk>

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

This comment has been minimized.

Copy link
Author

commented Feb 1, 2016

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

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

@rcubetrac

This comment has been minimized.

Copy link
Author

commented Feb 1, 2016

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

later => 1.1.5

@rcubetrac

This comment has been minimized.

Copy link
Author

commented Feb 1, 2016

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

This comment has been minimized.

Copy link
Author

commented Feb 1, 2016

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

new => closed

@rcubetrac rcubetrac closed this Feb 1, 2016

@rcubetrac rcubetrac added this to the 1.1.5 milestone Mar 20, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.