starred/cypht

Fork 0

mirror of https://github.com/cypht-org/cypht.git synced 2026-04-25 13:05:53 +03:00

[GH-ISSUE #377] More than 2 recipients #318

New issue

Closed

opened 2026-02-25 21:34:44 +03:00 by kerem · 7 comments

kerem commented

2026-02-25 21:34:44 +03:00

Owner

Originally created by @adrienmaloba on GitHub (Feb 20, 2020).
Original GitHub issue: https://github.com/cypht-org/cypht/issues/377

Originally assigned to: @jasonmunro on GitHub.

Enhancement

When there are more than 3 recipients, one disappears.

Originally created by @adrienmaloba on GitHub (Feb 20, 2020). Original GitHub issue: https://github.com/cypht-org/cypht/issues/377 Originally assigned to: @jasonmunro on GitHub. ## Enhancement  When there are more than 3 recipients, one disappears.   ![cypht_recipients_3_2](https://user-images.githubusercontent.com/28392444/74914489-32903700-53cb-11ea-8530-cade5f31fda2.gif)

kerem

2026-02-25 21:34:44 +03:00

closed this issue
added the
bug

smtp
labels

kerem commented

2026-02-25 21:34:44 +03:00

Author

Owner

@marclaporte commented on GitHub (Feb 20, 2020):

Good catch!

@marclaporte commented on GitHub (Feb 20, 2020): Good catch!

kerem commented

2026-02-25 21:34:44 +03:00

Author

Owner

@jasonmunro commented on GitHub (Mar 2, 2020):

Yikes! Thanks for the report. Hopefully won't be hard to track down and fix.

@jasonmunro commented on GitHub (Mar 2, 2020): Yikes! Thanks for the report. Hopefully won't be hard to track down and fix.

kerem commented

2026-02-25 21:34:44 +03:00

Author

Owner

@marclaporte commented on GitHub (Mar 13, 2020):

@jasonmunro our team will handle so you focus on bigger things :-)

@marclaporte commented on GitHub (Mar 13, 2020): @jasonmunro our team will handle so you focus on bigger things :-)

kerem commented

2026-02-25 21:34:44 +03:00

Author

Owner

@jasonmunro commented on GitHub (Apr 16, 2020):

I merged the PR for this without looking close enough and it broke address parsing. Not sure what is going on here as I can't even reproduce the original bug locally yet. I'm reverting that PR and I will see if I can trigger the issue and fix it.

@jasonmunro commented on GitHub (Apr 16, 2020): I merged the PR for this without looking close enough and it broke address parsing. Not sure what is going on here as I can't even reproduce the original bug locally yet. I'm reverting that PR and I will see if I can trigger the issue and fix it.

kerem commented

2026-02-25 21:34:44 +03:00

Author

Owner

@jasonmunro commented on GitHub (Apr 16, 2020):

Found a message that triggered this, and it would appear the issue is a trailing newline. If the last value on a header is an E-mail address, the validation fails because the newline is not stripped off, causing it to be dropped from the reply-all list.

I pushed a fix to master, would be great if someone could confirm it works for them now!

@jasonmunro commented on GitHub (Apr 16, 2020): Found a message that triggered this, and it would appear the issue is a trailing newline. If the last value on a header is an E-mail address, the validation fails because the newline is not stripped off, causing it to be dropped from the reply-all list. I pushed a fix to master, would be great if someone could confirm it works for them now!

kerem commented

2026-02-25 21:34:44 +03:00

Author

Owner

@adrienmaloba commented on GitHub (Apr 16, 2020):

I'll test for confirmation.

@adrienmaloba commented on GitHub (Apr 16, 2020): I'll test for confirmation.

kerem commented

2026-02-25 21:34:44 +03:00

Author

Owner

@marclaporte commented on GitHub (Apr 17, 2020):

On our Intranet, I updated from "1769ba9b8f" to "beea64a809" and I confirm the fix.

@marclaporte commented on GitHub (Apr 17, 2020): On our Intranet, I updated from "1769ba9b8f19f69caa01cae4a9c8af5da06367ec" to "beea64a80974445b0400be9a76935b4d9360626b" and I confirm the fix.