Skip to content

gh-123401: Fix http.cookies module to support obsolete RFC 850 date format#123405

Merged
vstinner merged 10 commits intopython:mainfrom
Konano:fix-issue-123401
Dec 11, 2024
Merged

gh-123401: Fix http.cookies module to support obsolete RFC 850 date format#123405
vstinner merged 10 commits intopython:mainfrom
Konano:fix-issue-123401

Conversation

@Konano
Copy link
Contributor

@Konano Konano commented Aug 27, 2024 •
edited
Loading

@ghost
Copy link

ghost commented Aug 27, 2024 •
edited by ghost
Loading

All commit authors signed the Contributor License Agreement.
CLA signed

@bedevere-app bedevere-app bot mentioned this pull request Aug 27, 2024
Closed
@bedevere-app
Copy link

bedevere-app bot commented Aug 27, 2024

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

Wulian233
Wulian233 reviewed Aug 28, 2024
View reviewed changes
@Konano @Wulian233
style(http/cookies): correct typo in regex comment
a373718 
Fix a minor typo in the regex comment by adding a hyphen to "double-quoted" for consistency and clarity. No functional changes to the code logic.

Co-authored-by: Wulian <1055917385@qq.com>
@Konano
Copy link
Contributor Author

Konano commented Sep 5, 2024

@zware Could you please take a look at this PR when you are free? Thanks!

picnixz
picnixz reviewed Sep 5, 2024
View reviewed changes
Konano and others added 2 commits September 6, 2024 01:32
@Konano @picnixz
Update Misc/NEWS.d/next/Library/2024-08-27-18-58-01.gh-issue-123401.t…
5cf347d 
…4-FpI.rst

Co-authored-by: Bénédikt Tran <10796600+picnixz@users.noreply.github.com>
@Konano
Include two examples from RFC 850
592da51
picnixz
picnixz reviewed Sep 5, 2024
View reviewed changes
Copy link
Member

@picnixz picnixz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

At first glance it seems correct. I'm a bit tired for more in-depth review (and even if I were to approve it, a core dev is needed for merging it) so I'll leave it at it for today.

picnixz
picnixz reviewed Sep 5, 2024
View reviewed changes
@Konano
Copy link
Contributor Author

Konano commented Sep 12, 2024

Is anyone available to follow up on this PR? Thanks!

Actually, I'm not sure who to contact...

@Wulian233
Copy link
Contributor

Wulian233 commented Sep 12, 2024

You can request for review on https://discuss.python.org/c/core-dev/23 🙂

vstinner
vstinner approved these changes Sep 18, 2024
View reviewed changes
Copy link
Member

@vstinner vstinner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks for the tests.

@vstinner
Copy link
Member

vstinner commented Dec 11, 2024

I couldn't merge the PR since some mandatory CI jobs didn't run. I clicked on [Update branch]. Let's see if it does unblock the issue.

@vstinner vstinner enabled auto-merge (squash) December 11, 2024 13:06
@vstinner vstinner merged commit 359389e into python:main Dec 11, 2024
@vstinner
Copy link
Member

vstinner commented Dec 11, 2024

Merged, thanks @Konano.

@Konano
Copy link
Contributor Author

Konano commented Dec 11, 2024

Does this PR need to be backported to other versions? @vstinner

@vstinner vstinner added the needs backport to 3.13 bugs and security fixes label Dec 11, 2024
@miss-islington-app
Copy link

miss-islington-app bot commented Dec 11, 2024

Thanks @Konano for the PR, and @vstinner for merging it 🌮🎉.. I'm working now to backport this PR to: 3.13.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Dec 11, 2024
@Konano @Wulian233
@picnixz @vstinner @miss-islington
pythongh-123401: Fix http.cookies module to support obsolete RFC 850 …
de4f22b 
…date format (pythonGH-123405)

(cherry picked from commit 359389e)

Co-authored-by: Nano <nanoapezlk@gmail.com>
Co-authored-by: Wulian <1055917385@qq.com>
Co-authored-by: Bénédikt Tran <10796600+picnixz@users.noreply.github.com>
Co-authored-by: Victor Stinner <vstinner@python.org>
@bedevere-app
Copy link

bedevere-app bot commented Dec 11, 2024

GH-127828 is a backport of this pull request to the 3.13 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.13 bugs and security fixes label Dec 11, 2024
@vstinner vstinner added the needs backport to 3.12 only security fixes label Dec 11, 2024
@miss-islington-app
Copy link

miss-islington-app bot commented Dec 11, 2024

Thanks @Konano for the PR, and @vstinner for merging it 🌮🎉.. I'm working now to backport this PR to: 3.12.
🐍🍒⛏🤖 I'm not a witch! I'm not a witch!

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Dec 11, 2024
@Konano @Wulian233
@picnixz @vstinner @miss-islington
pythongh-123401: Fix http.cookies module to support obsolete RFC 850 …
347c44b 
…date format (pythonGH-123405)

(cherry picked from commit 359389e)

Co-authored-by: Nano <nanoapezlk@gmail.com>
Co-authored-by: Wulian <1055917385@qq.com>
Co-authored-by: Bénédikt Tran <10796600+picnixz@users.noreply.github.com>
Co-authored-by: Victor Stinner <vstinner@python.org>
@bedevere-app
Copy link

bedevere-app bot commented Dec 11, 2024

GH-127829 is a backport of this pull request to the 3.12 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.12 only security fixes label Dec 11, 2024
@vstinner
Copy link
Member

vstinner commented Dec 11, 2024

Does this PR need to be backported to other versions? @vstinner

I created backports to 3.12 and 3.13 branches.

vstinner added a commit that referenced this pull request Dec 11, 2024
@miss-islington @Konano
@Wulian233 @picnixz @vstinner
[3.12] gh-123401: Fix http.cookies module to support obsolete RFC 850…
b0615a8 
… date format (GH-123405) (#127829)

gh-123401: Fix http.cookies module to support obsolete RFC 850 date format (GH-123405)
(cherry picked from commit 359389e)

Co-authored-by: Nano <nanoapezlk@gmail.com>
Co-authored-by: Wulian <1055917385@qq.com>
Co-authored-by: Bénédikt Tran <10796600+picnixz@users.noreply.github.com>
Co-authored-by: Victor Stinner <vstinner@python.org>
vstinner added a commit that referenced this pull request Dec 11, 2024
@miss-islington @Konano
@Wulian233 @picnixz @vstinner
[3.13] gh-123401: Fix http.cookies module to support obsolete RFC 850…
a94e20e 
… date format (GH-123405) (#127828)

gh-123401: Fix http.cookies module to support obsolete RFC 850 date format (GH-123405)
(cherry picked from commit 359389e)

Co-authored-by: Nano <nanoapezlk@gmail.com>
Co-authored-by: Wulian <1055917385@qq.com>
Co-authored-by: Bénédikt Tran <10796600+picnixz@users.noreply.github.com>
Co-authored-by: Victor Stinner <vstinner@python.org>
@Konano
Copy link
Contributor Author

Konano commented Dec 11, 2024

It seems everything is OK 🎉

srinivasreddy pushed a commit to srinivasreddy/cpython that referenced this pull request Jan 8, 2025
@Konano @Wulian233
@picnixz @vstinner @srinivasreddy
pythongh-123401: Fix http.cookies module to support obsolete RFC 850 …
5258f0e 
…date format (python#123405)

Co-authored-by: Wulian <1055917385@qq.com>
Co-authored-by: Bénédikt Tran <10796600+picnixz@users.noreply.github.com>
Co-authored-by: Victor Stinner <vstinner@python.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@picnixz picnixz picnixz left review comments

@vstinner vstinner vstinner approved these changes

+1 more reviewer

@Wulian233 Wulian233 Wulian233 left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@Konano @Wulian233 @vstinner @picnixz