Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork33.3k
gh-86650: get_obs_local_part() fails to handle empty local part#24669
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
gh-86650: get_obs_local_part() fails to handle empty local part#24669
Uh oh!
There was an error while loading.Please reload this page.
Conversation
This PR is stale because it has been open for 30 days with no activity. |
spacefreak86 commentedSep 6, 2021
What exactly is missing for this PR to get applied? It would be great because this bug is affecting my productive system and I have to monkey-patch the email library by myself. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Ran 1654 tests in 6.561s
OK (skipped=1)
Looks ok.
lesleyxyz commentedFeb 27, 2023
I am getting this bug in a production environment. How is this PR not yet reviewed after- all this time? How can we notify the right people to review this? |
@python/email-team |
mdavranche commentedMar 4, 2023
Same here, got this on my production environment, preventing me to upgrade from Python 3.7.3 (bug not present) to a more recent version (bug confirmed in 3.8, 3.9, 3.10, 3.11). Easy to trigger: |
lesleyxyz commentedMay 25, 2023
Is there an ETA for this please? |
fsc-eriker commentedFeb 9, 2024 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
I have a competing PR which does not simply silently ignore the problem, but instead raises a defect. My proposed implementation returns a partial message-id which is perhaps less than ideal. |
Thank you for your contribution,@ZackerySpytz, but I think that it is not a correct solution. Invalid Message-ID should not be silently truncated. The parsing code should raise HeaderParseError, this will allow to preserve its literal value. See#117934. |
Uh oh!
There was an error while loading.Please reload this page.
https://bugs.python.org/issue42484