FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

gh-130132: add an explicit close() call to urlopen examples without with by Mr-Sunglasses · Pull Request #130280 · python/cpython · GitHub

  1. python
  2. cpython

Repository navigation

gh-130132: add an explicit close() call to urlopen examples without with #130280

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
picnixz merged 11 commits into from
Mar 18, 2025
Conversation 25 Commits 11 Checks 0 Files changed
Merged

gh-130132: add an explicit close() call to urlopen examples without with #130280

Changes from all commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
acafffd
gh-130132: add an explicit close() call to urlopen examples without with
Mr-Sunglasses Feb 18, 2025
3c00f5d
gh-130132: add an explicit close() in Use of Basic HTTP Authentication
Mr-Sunglasses Feb 18, 2025
379b075
gh-130132: add an explicit close() in Adding HTTP headers
Mr-Sunglasses Feb 18, 2025
c9fbdb0
fix: use context manager in urllib.request examples
Mr-Sunglasses Feb 20, 2025
05d0a4e
fix: spaces in the docs,
Mr-Sunglasses Feb 25, 2025
9f1344a
fix: spaces in the docs.
Mr-Sunglasses Feb 25, 2025
90332d1
Merge branch 'main' into fix/130132
Mr-Sunglasses Mar 3, 2025
97e6b18
docs: Add context manager in examples of `urllib.request`
Mr-Sunglasses Mar 9, 2025
098ba79
Merge branch 'fix/130132' of github.com:Mr-Sunglasses/cpython into fi…
Mr-Sunglasses Mar 9, 2025
e56cc3e
Merge branch 'main' into fix/130132
Mr-Sunglasses Mar 9, 2025
3825537
use `f` as the `as` variable
picnixz Mar 18, 2025
File filter

Filter by extension

Filter by extension .rst  (1) All 1 file type selected
Viewed files
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Unified
Split
Hide whitespace
Diff view
Unified
Split
Hide whitespace
18 changes: 13 additions & 5 deletions Doc/library/urllib.request.rst
Show comments
View file
Open in desktop
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Original file line number Diff line number Diff line change
Expand Up @@ -1249,7 +1249,10 @@ It is also possible to achieve the same result without using the

>>> import urllib.request
>>> f = urllib.request.urlopen('http://www.python.org/')
>>> print(f.read(100).decode('utf-8'))
>>> try:
Comment thread
sobolevn marked this conversation as resolved.
... print(f.read(100).decode('utf-8'))
... finally:
... f.close()
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"
"http://www.w3.org/TR/xhtml1/DTD/xhtm

Expand Down Expand Up @@ -1294,7 +1297,8 @@ Use of Basic HTTP Authentication::
opener = urllib.request.build_opener(auth_handler)
# ...and install it globally so it can be used with urlopen.
urllib.request.install_opener(opener)
urllib.request.urlopen('http://www.example.com/login.html')
with urllib.request.urlopen('http://www.example.com/login.html') as f:
print(f.read().decode('utf-8'))

:func:`build_opener` provides many handlers by default, including a
:class:`ProxyHandler`. By default, :class:`ProxyHandler` uses the environment
Expand All @@ -1312,7 +1316,8 @@ programmatically supplied proxy URLs, and adds proxy authorization support with

opener = urllib.request.build_opener(proxy_handler, proxy_auth_handler)
# This time, rather than install the OpenerDirector, we use it directly:
opener.open('http://www.example.com/login.html')
with opener.open('http://www.example.com/login.html') as f:
print(f.read().decode('utf-8'))

Adding HTTP headers:

Expand All @@ -1323,15 +1328,18 @@ Use the *headers* argument to the :class:`Request` constructor, or::
req.add_header('Referer', 'http://www.python.org/')
# Customize the default User-Agent header value:
req.add_header('User-Agent', 'urllib-example/0.1 (Contact: . . .)')
r = urllib.request.urlopen(req)
with urllib.request.urlopen(req) as f:
print(f.read().decode('utf-8'))


:class:`OpenerDirector` automatically adds a :mailheader:`User-Agent` header to
every :class:`Request`. To change this::

import urllib.request
opener = urllib.request.build_opener()
opener.addheaders = [('User-agent', 'Mozilla/5.0')]
opener.open('http://www.example.com/')
with opener.open('http://www.example.com/') as f:
print(f.read().decode('utf-8'))

Also, remember that a few standard headers (:mailheader:`Content-Length`,
:mailheader:`Content-Type` and :mailheader:`Host`)
Expand Down
Back | FazBrowse Home | New Git URL