Skip to content

gh-137942: urllib2 with proxy tests maybe fail on macOS #137943

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.

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

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

yihong0618
Copy link
Contributor

@yihong0618 yihong0618 commented Aug 19, 2025

Even though the test code sets NO_PROXY='' and no_proxy='' in ProxyAuthTests.setUp(), since the HTTP_PROXY environment variable is not set, getproxies_environment() returns an empty dictionary.
the proxy_bypass function falls back to using the macOS system configuration (proxy_bypass_macosx_sysconf).
maybe for the macOS system settings,
localhost is bypassed from the proxy, which causes the direct connection to fail.

@python-cla-bot
Copy link

python-cla-bot bot commented Aug 19, 2025

All commit authors signed the Contributor License Agreement.

CLA signed

@bedevere-app bedevere-app bot added awaiting review tests Tests in the Lib/test dir labels Aug 19, 2025
@bedevere-app

This comment was marked as resolved.

Signed-off-by: yihong0618 <zouzou0208@gmail.com>
Signed-off-by: yihong0618 <zouzou0208@gmail.com>
@ZeroIntensity
Copy link
Member

!buildbot macos

@bedevere-bot
Copy link

🤖 New build scheduled with the buildbot fleet by @ZeroIntensity for commit dd02380 🤖

Results will be shown at:

https://buildbot.python.org/all/#/grid?branch=refs%2Fpull%2F137943%2Fmerge

The command will test the builders whose names match following regular expression: macos

The builders matched are:

  • x86-64 MacOS Intel ASAN NoGIL PR
  • ARM64 MacOS M1 NoGIL PR
  • ARM64 macOS PR
  • x86-64 macOS PR
  • ARM64 MacOS M1 Refleaks NoGIL PR
  • x86-64 MacOS Intel NoGIL PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants