gh-101936: Update the default value of fp from io.StringIO to io.BytesIO by longvhdeveloper · Pull Request #102100 · python/cpython · GitHub

Issue: HTTPError fp.read returns string instead of bytes #101936

All commit authors signed the Contributor License Agreement.

Please update the test to following codeline.

Lines 1827 to 1830 in 6f25657

    
           def test_gh_98778(self): 
        
               x = urllib.error.HTTPError("url", 405, "METHOD NOT ALLOWED", None, None) 
        
               self.assertEqual(getattr(x, "__notes__", ()), ())

The default value of ``fp`` becomes :class:`io.BytesIO` if :exc:`~urllib.error.HTTPError` is initialized without a designated `fp` parameter. Patch by Long Vo.

A Python core developer has requested some changes be made to your pull request before we can consider merging it. If you could please address their requests along with any other requests in other reviews from core developers that would be appreciated.

Once you have made the requested changes, please leave a comment on this pull request containing the phrase I have made the requested changes; please review again. I will then notify any core developers who have left a review that you're ready for them to take another look at this pull request.

Almost come :)

Suggested change

      
            The default value of fp becomes :class:io.BytesIO if
          
            :exc:~urllib.error.HTTPError is initialized without a designated fp
          
            The default value of fp becomes :class:`io.BytesIO` if
          
            :exc:`~urllib.error.HTTPError` is initialized without a designated ``fp``

Suggested change

      
                    self.assertEqual(isinstance(x.fp.read(), bytes), True)
          
                    self.assertIsInstance(x.fp.read(), bytes)

…VOxHH.rst

lgtm

Thanks @longvhdeveloper for the PR, and @corona10 for merging it 🌮🎉.. I'm working now to backport this PR to: 3.10, 3.11.
🐍🍒⛏🤖 I'm not a witch! I'm not a witch!

…o.BytesIO (pythongh-102100) (cherry picked from commit 0d4c7fc) Co-authored-by: Vo Hoang Long <vohoanglong07@gmail.com> Co-authored-by: Long Vo <long.vo@linecorp.com>

GH-102117 is a backport of this pull request to the 3.11 branch.

…o.BytesIO (pythongh-102100) (cherry picked from commit 0d4c7fc) Co-authored-by: Vo Hoang Long <vohoanglong07@gmail.com> Co-authored-by: Long Vo <long.vo@linecorp.com>

GH-102118 is a backport of this pull request to the 3.10 branch.

…io.BytesIO (gh-102100) (gh-102117) gh-101936: Update the default value of fp from io.StringIO to io.BytesIO (gh-102100) (cherry picked from commit 0d4c7fc) Co-authored-by: Long Vo <long.vo@linecorp.com>

…io.BytesIO (gh-102100) (#102118) gh-101936: Update the default value of fp from io.StringIO to io.BytesIO (gh-102100) (cherry picked from commit 0d4c7fc) Co-authored-by: Long Vo <long.vo@linecorp.com>

* main: (76 commits) Fix syntax error in struct doc example (python#102160) pythongh-99108: Import MD5 and SHA1 from HACL* (python#102089) pythonGH-101777: `queue.rst`: use 2 spaces after a period to be consistent. (python#102143) Few coverage nitpicks for the cmath module (python#102067) pythonGH-100982: Restrict `FOR_ITER_RANGE` to a single instruction to allow instrumentation. (pythonGH-101985) pythongh-102135: Update turtle docs to rename wikipedia demo to rosette (python#102137) pythongh-99942: python.pc on android/cygwin should link to libpython per configure.ac (pythonGH-100356) pythongh-95672 fix typo SkitTest to SkipTest (pythongh-102119) pythongh-101936: Update the default value of fp from io.StringIO to io.BytesIO (pythongh-102100) pythongh-102008: simplify test_except_star by using sys.exception() instead of sys.exc_info() (python#102009) pythongh-101903: Remove obsolete undefs for previously removed macros Py_EnterRecursiveCall and Py_LeaveRecursiveCall (python#101923) pythongh-100556: Improve clarity of `or` docs (python#100589) pythongh-101777: Make `PriorityQueue` docs slightly clearer (python#102026) pythongh-101965: Fix usage of Py_EnterRecursiveCall return value in _bisectmodule.c (pythonGH-101966) pythongh-101578: Amend exception docs (python#102057) pythongh-101961 fileinput.hookcompressed should not set the encoding value for the binary mode (pythongh-102068) pythongh-102056: Fix a few bugs in error handling of exception printing code (python#102078) pythongh-102011: use sys.exception() instead of sys.exc_info() in docs where possible (python#102012) pythongh-101566: Sync with zipp 3.14. (pythonGH-102018) pythonGH-99818: improve the documentation for zipfile.Path and Traversable (pythonGH-101589) ...

…o.BytesIO (pythongh-102100) Co-authored-by: Long Vo <long.vo@linecorp.com>

HTTPError fp.read returns string instead of bytes

5042916

bedevere-bot added the awaiting review label Feb 21, 2023

bedevere-bot mentioned this pull request Feb 21, 2023

HTTPError fp.read returns string instead of bytes #101936

Closed

corona10 self-assigned this Feb 21, 2023

corona10 self-requested a review February 21, 2023 07:33

corona10 added needs backport to 3.10 only security fixes needs backport to 3.11 only security fixes labels Feb 21, 2023

corona10 requested changes Feb 21, 2023

View reviewed changes

bedevere-bot added awaiting changes and removed awaiting review labels Feb 21, 2023

corona10 changed the title gh-101936: HTTPError fp.read returns string instead of bytes gh-101936: Update the default value of fp from io.StringIO to io.BytesIO Feb 21, 2023

update unit test

3553029

corona10 requested changes Feb 21, 2023

View reviewed changes

longvhdeveloper added 3 commits February 21, 2023 13:41

Update Misc/News.d

1de16a7

update unit test

d5b74b6

Update Misc/NEWS.d

b02433e

corona10 reviewed Feb 21, 2023

View reviewed changes

Comment thread Misc/NEWS.d/next/Library/2023-02-21-07-15-41.gh-issue-101936.QVOxHH.rst Outdated

Update Misc/NEWS.d/next/Library/2023-02-21-07-15-41.gh-issue-101936.Q…

9cc1605

…VOxHH.rst

corona10 reviewed Feb 21, 2023

View reviewed changes

Comment thread Misc/NEWS.d/next/Library/2023-02-21-07-15-41.gh-issue-101936.QVOxHH.rst Outdated

Update Misc/NEWS.d/next/Library/2023-02-21-07-15-41.gh-issue-101936.Q…

b1c8dea

…VOxHH.rst

corona10 approved these changes Feb 21, 2023

View reviewed changes

bedevere-bot added awaiting merge and removed awaiting changes labels Feb 21, 2023

corona10 merged commit 0d4c7fc into python:main Feb 21, 2023

bedevere-bot removed the awaiting merge label Feb 21, 2023

bedevere-bot removed the needs backport to 3.11 only security fixes label Feb 21, 2023

bedevere-bot removed the needs backport to 3.10 only security fixes label Feb 21, 2023

python-sidebar pushed a commit to python-sidebar/Python-Documentation-Fork-With-TOC that referenced this pull request Sep 1, 2024

pythongh-101936 : Update the default value of fp from io.StringIO to i…

563bb72

…o.BytesIO (pythongh-102100) Co-authored-by: Long Vo <long.vo@linecorp.com>

python-sidebar pushed a commit to python-sidebar/Python-Documentation-Fork-With-TOC that referenced this pull request Sep 1, 2024

pythongh-101936 : Update the default value of fp from io.StringIO to i…

4f88406

…o.BytesIO (pythongh-102100) Co-authored-by: Long Vo <long.vo@linecorp.com>

JelleZijlstra pushed a commit to JelleZijlstra/cpython that referenced this pull request Sep 10, 2024

pythongh-101936 : Update the default value of fp from io.StringIO to i…

c1a279c

…o.BytesIO (pythongh-102100) Co-authored-by: Long Vo <long.vo@linecorp.com>

	def test_gh_98778(self):
	x = urllib.error.HTTPError("url", 405, "METHOD NOT ALLOWED", None, None)
	self.assertEqual(getattr(x, "__notes__", ()), ())

		@@ -0,0 +1 @@
		HTTPError fp.read returns string instead of bytes

		The default value of fp becomes :class:io.BytesIO if
		:exc:~urllib.error.HTTPError is initialized without a designated fp

	self.assertEqual(isinstance(x.fp.read(), bytes), True)
	self.assertIsInstance(x.fp.read(), bytes)

gh-101936: Update the default value of fp from io.StringIO to io.BytesIO by longvhdeveloper · Pull Request #102100 · python/cpython · GitHub

Uh oh!

Uh oh!

gh-101936: Update the default value of fp from io.StringIO to io.BytesIO - #102100

gh-101936: Update the default value of fp from io.StringIO to io.BytesIO#102100corona10 merged 7 commits intopython:mainpython/cpython:mainfrom longvhdeveloper:gh-101936longvhdeveloper/cpython:gh-101936Copy head branch name to clipboard

Conversation

longvhdeveloper commented Feb 21, 2023 • edited by bedevere-bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

ghost commented Feb 21, 2023 • edited by ghost Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

corona10 left a comment

Choose a reason for hiding this comment

Uh oh!

corona10 Feb 21, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

bedevere-bot commented Feb 21, 2023

Uh oh!

corona10 left a comment

Choose a reason for hiding this comment

Uh oh!

corona10 Feb 21, 2023

Choose a reason for hiding this comment

Uh oh!

corona10 Feb 21, 2023

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

corona10 left a comment

Choose a reason for hiding this comment

Uh oh!

miss-islington commented Feb 21, 2023

Uh oh!

bedevere-bot commented Feb 21, 2023

Uh oh!

bedevere-bot commented Feb 21, 2023

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

gh-101936: Update the default value of fp from io.StringIO to io.BytesIO#102100
corona10 merged 7 commits into
python:mainfrom
longvhdeveloper:gh-101936

longvhdeveloper commented Feb 21, 2023 •
edited by bedevere-bot
Loading

ghost commented Feb 21, 2023 •
edited by ghost
Loading

corona10 Feb 21, 2023 •
edited
Loading