8000 [3.8] bpo-21315: Fix parsing of encoded words with missing leading ws (GH-13425) by epicfaace · Pull Request #15655 · python/cpython · GitHub
[go: up one dir, main page]

Skip to content

Conversation

@epicfaace
Copy link
Contributor
@epicfaace epicfaace commented Sep 3, 2019
  • bpo-21315: Fix parsing of encoded words with missing leading ws.

Because of missing leading whitespace, encoded word would get parsed as
unstructured token. This patch fixes that by looking for encoded words when
splitting tokens with whitespace.

Missing trailing whitespace around encoded word now register a defect
instead.

Original patch suggestion by David R. Murray on bpo-21315.
(cherry picked from commit 66c4f3f)

Co-authored-by: Abhilash Raj maxking@users.noreply.github.com
(cherry picked from commit dc20fc4)

Co-authored-by: Miss Islington (bot) 31488909+miss-islington@users.noreply.github.com

https://bugs.python.org/issue21315

Automerge-Triggered-By: @maxking

pythonGH-13425) (pythonGH-13846)

* bpo-21315: Fix parsing of encoded words with missing leading ws.

Because of missing leading whitespace, encoded word would get parsed as
unstructured token. This patch fixes that by looking for encoded words when
splitting tokens with whitespace.

Missing trailing whitespace around encoded word now register a defect
instead.

Original patch suggestion by David R. Murray on bpo-21315.
(cherry picked from commit 66c4f3f)

Co-authored-by: Abhilash Raj <maxking@users.noreply.github.com>
(cherry picked from commit dc20fc4)

Co-authored-by: Miss Islington (bot) <31488909+miss-islington@users.noreply.github.com>
@epicfaace epicfaace requested a review from a team as a code owner September 3, 2019 04:38
@maxking maxking changed the title [3.8] bpo-21315: Fix parsing of encoded words with missing leading ws… [3.8] bpo-21315: Fix parsing of encoded words with missing leading ws (GH-13425) Sep 3, 2019
@maxking
Copy link
Contributor
maxking commented Sep 3, 2019

Thanks @epicfaace for backporting this manually, not sure why we missed this. I think it was before the because it was created before the beta1 cutoff and was merged after that.

@epicfaace epicfaace deleted the backport-dc20fc4-3.8 branch September 3, 2019 19:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

0