Skip to content

gh-153822: Recreate the headers from raw form when part of signed data#154107

Open
MichalKarol wants to merge 9 commits into
python:mainfrom
MichalKarol:gh-153822-keep-raw-header-values-when-signed
Open

gh-153822: Recreate the headers from raw form when part of signed data#154107
MichalKarol wants to merge 9 commits into
python:mainfrom
MichalKarol:gh-153822-keep-raw-header-values-when-signed

Conversation

@MichalKarol

@MichalKarol MichalKarol commented Jul 19, 2026

Copy link
Copy Markdown

When Message is part of multipart/signed, the headers are geneated from the raw header value to fully reproduce folding.

@MichalKarol
MichalKarol requested a review from a team as a code owner July 19, 2026 11:26
@bedevere-app

bedevere-app Bot commented Jul 19, 2026

Copy link
Copy Markdown

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.

@python-cla-bot

python-cla-bot Bot commented Jul 19, 2026

Copy link
Copy Markdown

All commit authors signed the Contributor License Agreement.

CLA signed

@bedevere-app

bedevere-app Bot commented Jul 19, 2026

Copy link
Copy Markdown

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.

@bedevere-app

bedevere-app Bot commented Jul 19, 2026

Copy link
Copy Markdown

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.

@bedevere-app

bedevere-app Bot commented Jul 19, 2026

Copy link
Copy Markdown

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.

@zware

zware commented Jul 19, 2026

Copy link
Copy Markdown
Member

@MichalKarol I'm not qualified to actually review this, but looking at the issue I think this is worth a NEWS entry if this fixes the issue. (As Chris mentioned in person, at worst the proper reviewer will just ask that it be removed if I'm wrong and it does turn out to be superfluous.)

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

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

2 participants