Skip to content

[3.11] gh-145599, CVE 2026-3644: Reject control characters in http.cookies.Morsel.update() (GH-145600)#146026

Open
miss-islington wants to merge 2 commits into
python:3.11from
miss-islington:backport-57e88c1-3.11
Open

[3.11] gh-145599, CVE 2026-3644: Reject control characters in http.cookies.Morsel.update() (GH-145600)#146026
miss-islington wants to merge 2 commits into
python:3.11from
miss-islington:backport-57e88c1-3.11

Conversation

@miss-islington
Copy link
Copy Markdown
Contributor

@miss-islington miss-islington commented Mar 16, 2026

Reject control characters in http.cookies.Morsel.update() and http.cookies.BaseCookie.js_output.
(cherry picked from commit 57e88c1)

Co-authored-by: Stan Ulbrych 89152624+StanFromIreland@users.noreply.github.com
Co-authored-by: Victor Stinner vstinner@python.org
Co-authored-by: Victor Stinner victor.stinner@gmail.com

…okies.Morsel.update()` (pythonGH-145600)

Reject control characters in `http.cookies.Morsel.update()` and `http.cookies.BaseCookie.js_output`.
(cherry picked from commit 57e88c1)

Co-authored-by: Stan Ulbrych <89152624+StanFromIreland@users.noreply.github.com>
Co-authored-by: Victor Stinner <vstinner@python.org>
Co-authored-by: Victor Stinner <victor.stinner@gmail.com>
Copy link
Copy Markdown
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.

Comment thread Misc/NEWS.d/next/Security/2026-03-06-17-03-38.gh-issue-145599.kchwZV.rst Outdated
…kchwZV.rst

Co-authored-by: Stan Ulbrych <89152624+StanFromIreland@users.noreply.github.com>
@elboulangero
Copy link
Copy Markdown

Hello, I'm doing some backport work for Debian, and wondering if this one is going to be merged, or if there's some reason why it shouldn't. From the surface it looks like a fairly simple patch.

I don't want to rush anyone, just asking for advice. Thanks!

@StanFromIreland
Copy link
Copy Markdown
Member

It'll be merged before the next release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants