[3.14] gh-127146: Emscripten: Fix test_open_undecodable_uri by setting `-sTEXTDECODER=2` (GH-136624) by miss-islington · Pull Request #136631 · python/cpython · GitHub
Skip to content

[3.14] gh-127146: Emscripten: Fix test_open_undecodable_uri by setting -sTEXTDECODER=2 (GH-136624) - #136631

Merged
freakboy3742 merged 1 commit into
python:3.14from
miss-islington:backport-283b050-3.14
Jul 14, 2025
Merged

[3.14] gh-127146: Emscripten: Fix test_open_undecodable_uri by setting -sTEXTDECODER=2 (GH-136624)#136631
freakboy3742 merged 1 commit into
python:3.14from
miss-islington:backport-283b050-3.14

Conversation

@miss-islington

@miss-islington miss-islington commented Jul 13, 2025

Copy link
Copy Markdown
Contributor

Removes the JS text decoder fallback and gets rid of the bugs due to the differences
in behavior on invalid utf8 strings. See emscripten-core/emscripten#24690.
(cherry picked from commit 283b050)

Co-authored-by: Hood Chatham roberthoodchatham@gmail.com

… `-sTEXTDECODER=2` (pythonGH-136624)

Removes the JS text decoder fallback and gets rid of the bugs due to the differences
in behavior on invalid utf8 strings. See emscripten-core/emscripten#24690.
(cherry picked from commit 283b050)

Co-authored-by: Hood Chatham <roberthoodchatham@gmail.com>
@freakboy3742

Copy link
Copy Markdown
Contributor

@bedevere-bot

Copy link
Copy Markdown

@freakboy3742
freakboy3742 merged commit 2618180 into python:3.14 Jul 14, 2025
kumaraditya303 pushed a commit to miss-islington/cpython that referenced this pull request Sep 9, 2025
…setting `-sTEXTDECODER=2` (pythonGH-136624) (python#136631)

Removes the JS text decoder fallback and gets rid of the bugs due to the differences
in behavior on invalid utf8 strings. See emscripten-core/emscripten#24690.
(cherry picked from commit 283b050)

Co-authored-by: Hood Chatham <roberthoodchatham@gmail.com>
@miss-islington
miss-islington deleted the backport-283b050-3.14 branch January 2, 2026 17:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants