Skip to content

CURLOPT_SOCKS5_AUTH.md: an access property - #21230

Closed
bagder wants to merge 2 commits into
masterfrom
bagder/socks-auth-connect-property
Closed

bagder wants to merge 2 commits into
masterfrom
bagder/socks-auth-connect-property

Conversation

@bagder

@bagder bagder commented Apr 5, 2026

Copy link
Copy Markdown
Member

Reported-by: Cutiapreta on hackerone

Reported-by: Cutiapreta on hackerone

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the CURLOPT_SOCKS5_AUTH(3) documentation to clarify how the chosen SOCKS5 authentication method interacts with proxy connection reuse (per the reported issue).

Changes:

  • Document that the SOCKS5 authentication method is treated as an access/login-style property and does not alter the security context.
  • Clarify that libcurl may reuse proxy connections created with a different SOCKS authentication method when other reuse properties match.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/libcurl/opts/CURLOPT_SOCKS5_AUTH.md Outdated
language

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@bagder
bagder marked this pull request as ready for review April 5, 2026 15:26
@bagder bagder closed this in f3176e0 Apr 5, 2026
@bagder
bagder deleted the bagder/socks-auth-connect-property branch May 8, 2026 14:45
outcast36 pushed a commit to greearb/curl that referenced this pull request Jun 3, 2026
Reported-by: Cutiapreta on hackerone
Closes curl#21230
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants