Skip to content
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.

Sign up for GitHub

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

fix: set socks proxy socket family #94

Merged
merged 7 commits into from
Apr 1, 2024
Merged

fix: set socks proxy socket family #94

Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
c7388bc
fix: set socks proxy socket family
lukekarrys Mar 31, 2024
a0fd357
Use hostAddress for ipv6 socks
lukekarrys Apr 1, 2024
46c89b5
Request to localhost for ipv6 socks
lukekarrys Apr 1, 2024
833e9b1
Fix ipv6 test for socks in node16
lukekarrys Apr 1, 2024
457d215
Add some todos
lukekarrys Apr 1, 2024
25d8de4
Update socks-proxy-agent
lukekarrys Apr 1, 2024
efbe5f0
Figure out why node 16 socks test failed for ipv6
lukekarrys Apr 1, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Request to localhost for ipv6 socks
  • Loading branch information
@lukekarrys
lukekarrys committed Apr 1, 2024
commit 46c89b50292987519652c5916688ec90ba57b84e
2 changes: 1 addition & 1 deletion test/fixtures/server.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,7 @@ class Server extends EventEmitter {

get hostAddress () {
const ipv6 = net.isIPv6(this.#host)
const host = `${ipv6 ? '[' : ''}${this.#host}${ipv6 ? ']' : ''}`
const host = ipv6 ? 'localhost' : this.#host
return `${this.#protocol}//${this.#auth}${host}:${this.#port}`
}

Expand Down
Loading