Skip to content

[3.10] bpo-41402: Fix email ContentManager calling .encode() on bytes (GH-21631)#27686

Merged
miss-islington merged 1 commit intopython:3.10from
miss-islington:backport-b33186b-3.10
Aug 9, 2021
Merged

[3.10] bpo-41402: Fix email ContentManager calling .encode() on bytes (GH-21631)#27686
miss-islington merged 1 commit intopython:3.10from
miss-islington:backport-b33186b-3.10

Conversation

@miss-islington
Copy link
Copy Markdown
Contributor

@miss-islington miss-islington commented Aug 9, 2021 •
edited by bedevere-bot
Loading

(cherry picked from commit b33186b)

Co-authored-by: Johannes Reiff mail@jreiff.de

https://bugs.python.org/issue41402

@joreiff @miss-islington
bpo-41402: Fix email ContentManager calling .encode() on bytes (pytho…
fa6b884 
…nGH-21631)

(cherry picked from commit b33186b)

Co-authored-by: Johannes Reiff <mail@jreiff.de>
@miss-islington miss-islington requested a review from a team as a code owner August 9, 2021 16:45
@miss-islington
Copy link
Copy Markdown
Contributor Author

miss-islington commented Aug 9, 2021

@joreiff and @ambv: Status check is done, and it's a success ✅ .

@bedevere-bot bedevere-bot mentioned this pull request Aug 9, 2021
Merged
@miss-islington
Copy link
Copy Markdown
Contributor Author

miss-islington commented Aug 9, 2021

@joreiff and @ambv: Status check is done, and it's a success ❌ .

@miss-islington
Copy link
Copy Markdown
Contributor Author

miss-islington commented Aug 9, 2021

@joreiff and @ambv: Status check is done, and it's a success ✅ .

1 similar comment
@miss-islington
Copy link
Copy Markdown
Contributor Author

miss-islington commented Aug 9, 2021

@joreiff and @ambv: Status check is done, and it's a success ✅ .

@ambv
Copy link
Copy Markdown
Contributor

ambv commented Aug 9, 2021

@pablogsal, one branch in set_bytes_content in email tried to errorneously further encode bytes() instead of decoding them.

@miss-islington miss-islington merged commit a3185da into python:3.10 Aug 9, 2021
@miss-islington miss-islington deleted the backport-b33186b-3.10 branch August 9, 2021 23:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pablogsal pablogsal pablogsal approved these changes

Assignees

@pablogsal pablogsal

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@miss-islington @ambv @pablogsal @the-knights-who-say-ni @bedevere-bot @joreiff