Skip to content

support no_proxy excatly like CURL #522

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 3 commits into
base: main
Choose a base branch
from
Open

support no_proxy excatly like CURL #522

wants to merge 3 commits into from

Conversation

TingDaoK
Copy link
Contributor

@TingDaoK TingDaoK commented Jun 25, 2025

Issue #, if available:

Description of changes:

  • Since NO_PROXY has no standard yet. https://about.gitlab.com/blog/we-need-to-talk-no-proxy/ and likely won't have any standard in the foreseeable future (there been many discussions but none of them really driving to a conclusion.).
  • It's also very tricky about how to match the host and IP address, and how to handle concern cases and invalid cases.
  • In the end, I decided to follow exactly the curl implementation from noproxy.c at 8.14.1
  • The implementation basically replicated the CURL logic.
  • All the CIDR and host validation all followed how CURL implement it.
  • Given no standards, we are taking curl as the the standard here.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@TingDaoK TingDaoK changed the title no proxy support no_proxy excatly like CURL Jun 25, 2025
@codecov-commenter
Copy link

codecov-commenter commented Jun 25, 2025

Codecov Report

Attention: Patch coverage is 92.30769% with 11 lines in your changes missing coverage. Please review.

Project coverage is 79.74%. Comparing base (a04c10a) to head (6c6091a).

Files with missing lines Patch % Lines
source/no_proxy.c 93.33% 9 Missing ⚠️
source/proxy_connection.c 75.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #522      +/-   ##
==========================================
+ Coverage   79.62%   79.74%   +0.11%     
==========================================
  Files          27       28       +1     
  Lines       11702    11833     +131     
==========================================
+ Hits         9318     9436     +118     
- Misses       2384     2397      +13     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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

Successfully merging this pull request may close these issues.

2 participants