Skip to content

CURLINFO_USED_PROXY: return a bool whether the proxy was used#12719

Closed
bagder wants to merge 2 commits intomasterfrom
bagder/was-proxy
Closed

CURLINFO_USED_PROXY: return a bool whether the proxy was used#12719
bagder wants to merge 2 commits intomasterfrom
bagder/was-proxy

Conversation

@bagder
Copy link
Copy Markdown
Member

@bagder bagder commented Jan 16, 2024 •
edited
Loading

Idea: #12669

  • - test case 536 verifies
  • - %{proxy_used} for -w

@bagder bagder added feature-window A merge of this requires an open feature window connecting & proxies libcurl API labels Jan 16, 2024
genotrance
genotrance reviewed Jan 23, 2024
View reviewed changes
Comment thread include/curl/curl.h Outdated
@bagder bagder marked this pull request as ready for review January 23, 2024 16:54
@github-actions github-actions bot added the tests label Feb 21, 2024
@bagder bagder changed the title CURLINFO_WAS_PROXY: return a bool whether the proxy was used CURLINFO_USED_PROXY: return a bool whether the proxy was used Feb 21, 2024
@bagder
CURLINFO_USED_PROXY: return bool whether the proxy was used
dcc6bf3 
Adds test536 to verify

Closes #12719
@bagder
write-out: add '%{proxy_used}'
13d77f5 
Returns 1 if the previous transfer used a proxy, otherwise 0. Useful to
for example determine if a `NOPROXY` pattern matched the hostname or
not.

Extended test 970 and 972
@bagder bagder closed this in cc04c73 Feb 22, 2024
@bagder bagder deleted the bagder/was-proxy branch February 22, 2024 07:39
vszakats added a commit to vszakats/curl that referenced this pull request Mar 20, 2026
@vszakats
getinfo: initalize PureInfo field used_proxy
9b2a8ce 
Follow-up to cc04c73 curl#12719
@vszakats vszakats mentioned this pull request Mar 20, 2026
Closed
vszakats added a commit that referenced this pull request Mar 20, 2026
@vszakats
getinfo: initialize PureInfo field used_proxy
2cdb220 
Found by Codex Security

Follow-up to cc04c73 #12719

Closes #21020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@genotrance genotrance genotrance left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

cmdline tool connecting & proxies feature-window A merge of this requires an open feature window libcurl API tests

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@bagder @genotrance