[issue44637] Quoting issue on header Reply-To and other address headers

Julien Castiaux added the comment: Hello there, Friendly reminder that this issue is still open and that there is a pull request ready. We continue to face the issue in production and our customers are getting upset. Can you provide us a schedule when this issue will be addressed? So that

[issue44637] Quoting issue on header Reply-To and other address headers

Julien Castiaux added the comment: Hello there, There is a pull-request on github, had to modify `_refold_parse_tree` but I could keep the diff quite small. It is properly tested and it is waiting a review :) We have a patch at work so it is *absolutely not* urgent, feel free to review it

[issue44637] Quoting issue on header Reply-To and other address headers

Alexander Mohr added the comment: btw my work-around was to set maxheaderlen=sys.maxsize, worked for AWS SES at least -- ___ Python tracker ___

[issue44637] Quoting issue on header Reply-To and other address headers

Change by Julien Castiaux : -- keywords: +patch pull_requests: +28107 stage: -> patch review pull_request: https://github.com/python/cpython/pull/29881 ___ Python tracker ___

[issue44637] Quoting issue on header Reply-To and other address headers

Change by STINNER Victor : -- nosy: -vstinner ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue44637] Quoting issue on header Reply-To and other address headers

Change by R. David Murray : -- nosy: +thehesiod title: Quoting issue on header Reply-To -> Quoting issue on header Reply-To and other address headers ___ Python tracker ___

[issue44637] Quoting issue on header Reply-To

Julien Castiaux added the comment: Hello David, Victor, Thank you for the triage, it reminds me about this issue. David, the solution I tried last month was wrong, it was breaking (for good reasons) tons of unittests. It seems to me that there is indeed no other solution than to bloat the

[issue44637] Quoting issue on header Reply-To

STINNER Victor added the comment: I change the issue type to security. The bug can be abused to send emails to the wrong email address. -- nosy: +vstinner type: behavior -> security ___ Python tracker

[issue44637] Quoting issue on header Reply-To

R. David Murray added the comment: Yes, compat32 uses a different parser and folder (the legacy ones), that have a lot of small bugs relative to the RFCs (which is why I rewrote it). -- ___ Python tracker

[issue44637] Quoting issue on header Reply-To

Julien Castiaux added the comment: Update, it works fine with the compat32 policy -- ___ Python tracker ___ ___ Python-bugs-list

[issue44637] Quoting issue on header Reply-To

Julien Castiaux added the comment: Hello David, I'm working in the same company as Baptiste and I'm trying to solve the problem. The issue is indeed related to the folding algorithm, the DBQUOTE character is lost in the parse_tree AST thus when the folding algo split the children to find a

[issue44637] Quoting issue on header Reply-To

R. David Murray added the comment: Forget what I said about my different error, I made a mistake running the test script. Interesting. If it is related to the length of the name, then the problem is most likely in the folding algorithm, specifically in what happens when the "display-name"

[issue44637] Quoting issue on header Reply-To

Baptiste added the comment: Thanks David, Here is some other tests I ran Issuing: - msg['Reply-To'] = '"foo Research Inc Foofoo BarBar on Summer Special Friday 050 days (2021-02-31" ' - msg['Reply-To'] = '"foo Research Inc Foofoo BarBar on Summer Special Friday 050 days 20210231 " '

[issue44637] Quoting issue on header Reply-To

R. David Murray added the comment: There is definitely a problem here, though I see a different problem when I run it (AttributeError: 'Group' object has no attribute 'local_part', presumably because of the ':' not getting escaped correctly). I believe it applies to any address header, not

[issue44637] Quoting issue on header Reply-To

New submission from Baptiste : Hello, When using as_string() on a Reply-To header like the following: msg['Reply-To'] = '"foo Research, Inc. Foofoo BarBar on Summer Special Friday: 0.50 days (2021-02-31)" ' The double quote disappear, which lead to wrong header value See attached file for