Skip to content

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

Closed
yihong0618 wants to merge 5 commits intopython:mainfrom
yihong0618:hy/close_issue_137942
Closed

gh-137942: urllib2 with proxy tests maybe fail on macOS#137943
yihong0618 wants to merge 5 commits intopython:mainfrom
yihong0618:hy/close_issue_137942

Conversation

@yihong0618
Copy link
Copy Markdown
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
Copy Markdown

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
Copy Markdown
Member

!buildbot macos

@bedevere-bot
Copy link
Copy Markdown

🤖 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

@yihong0618 yihong0618 closed this by deleting the head repository Dec 16, 2025
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