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

Roundcube doesn't understand RFC2822 'group' addresses (foo:;) #7663

Closed
mas90 opened this issue Oct 8, 2020 · 3 comments
Closed

Roundcube doesn't understand RFC2822 'group' addresses (foo:;) #7663

mas90 opened this issue Oct 8, 2020 · 3 comments

Comments

@mas90
Copy link
Contributor

mas90 commented Oct 8, 2020

I have encountered parsing issues with To/CC headers which include a group: label as defined in RFC2822 section 3.4 (https://tools.ietf.org/html/rfc2822.html#section-3.4).

Empty group addresses

Request Tracker (RT), in our reasonably normal installation, generates mail with headers such as:

To: "AdminCc of RT Ticket #123456":;
Reply-To: rt@example.com

This 'To' header is a legal group address per RFC2822 section 3.4. Semantically, AdminCc of RT Ticket #123456 is a display-name for an empty group of addresses.

If I 'Reply All' to such an email from Roundcube, the headers in the compose window are filled as follows:

To: rt@example.com
Copy: "AdminCc of RT Ticket #123456":

Note lack of trailing ; in the pre-filled Copy field, which renders it syntactically invalid.

Then if I try to send this email, I get an 'Invalid email address' error from Roundcube.

If I try to manually re-add the missing ; character, something in the UI JavaScript removes it again as soon as I leave the field.

Non-empty group addresses

If I manually compose email in Roundcube using the group address syntax, for example:

To: "my group": foo@example.com, bar@example.com;

Roundcube mangles it, treating as "My group:" as the name associated with foo@example.com; the emitted email contains e.g.:

To: "\"my group\":" <foo@example.com>, bar@example.com
@alecpl
Copy link
Member

alecpl commented Oct 8, 2020

Groups aren't supported. We might fix the issue in a way that the group is not put into the Cc field. Anything more would be much more complicated.

@alecpl alecpl added this to the later milestone Oct 8, 2020
@mas90
Copy link
Contributor Author

mas90 commented Oct 8, 2020

Groups aren't supported. We might fix the issue in a way that the group is not put into the Cc field. Anything more would be much more complicated.

Thanks. Not putting an empty group into the Cc field would solve most of my problems.

@alecpl alecpl modified the milestones: later, 1.5-rc Dec 28, 2020
@alecpl alecpl modified the milestones: 1.5-rc, 1.5-beta Feb 6, 2021
@alecpl
Copy link
Member

alecpl commented Feb 6, 2021

Fixed.

@alecpl alecpl closed this as completed Feb 6, 2021
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

2 participants