Thanks to visit codestin.com
Credit goes to github.com

Skip to content

bpo-39826: add getConnection() hook to logging HTTPHandler #18745

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

Jump to bottom
Merged
merged 4 commits into from
Mar 4, 2020
Merged

bpo-39826: add getConnection() hook to logging HTTPHandler #18745

merged 4 commits into from
Mar 4, 2020

Conversation

l0rb
Copy link
Contributor

@l0rb l0rb commented Mar 2, 2020 •
edited by bedevere-bot
Loading

This PR enables it easily extend HTTPHandler to provide a custom connection, for example to use a proxy, without having to reimplement emit.

https://bugs.python.org/issue39826

@l0rb l0rb requested a review from vsajip as a code owner March 2, 2020 15:14
@python python deleted a comment from codecov bot Mar 3, 2020
@vsajip vsajip added the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Mar 4, 2020
@bedevere-bot
Copy link

🤖 New build scheduled with the buildbot fleet by @vsajip for commit 94b52e3 🤖

If you want to schedule another build, you need to add the ":hammer: test-with-buildbots" label again.

@bedevere-bot bedevere-bot removed the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Mar 4, 2020
@vsajip vsajip merged commit 22a9a54 into python:master Mar 4, 2020
sthagen added a commit to sthagen/python-cpython that referenced this pull request Mar 4, 2020
@sthagen
Merge pull request #299 from python/master
a9f0237 
bpo-39826: add getConnection() hook to logging HTTPHandler (pythonGH-18745)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vsajip vsajip vsajip approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@l0rb @bedevere-bot @vsajip @the-knights-who-say-ni