fix: TypeError: SocksProxyAgent is not a constructor

[Janlaywss]

What

Bug: SocksProxyAgent is not a constructor

In the documentation and code of socks-proxy-agent 7.0.0, the SocksProxyAgent class is no longer exported by default.

So when we use the default export, the error SocksProxyAgent is not a constructor will be thrown

[wraithgar]

Thanks for fixing this @Janlaywss

[wraithgar]

I forgot about tests when I clicked approve. This should fix things. You'll also need to npm run lintfix.

diff --git a/test/agent.js b/test/agent.js
index 0b721d7..f8def79 100644
--- a/test/agent.js
+++ b/test/agent.js
@@ -9,7 +9,7 @@ const agent = t.mock('../lib/agent.js', {
   agentkeepalive: MockHttp,
   'https-proxy-agent': mockHttpAgent('https-proxy'),
   'http-proxy-agent': mockHttpAgent('http-proxy'),
-  'socks-proxy-agent': mockHttpAgent('socks-proxy'),
+  'socks-proxy-agent': { SocksProxyAgent: mockHttpAgent('socks-proxy') },
 })

[Janlaywss]

I forgot about tests when I clicked approve. This should fix things. You'll also need to npm run lintfix.

diff --git a/test/agent.js b/test/agent.js

index 0b721d7..f8def79 100644

--- a/test/agent.js

+++ b/test/agent.js

@@ -9,7 +9,7 @@ const agent = t.mock('../lib/agent.js', {

   agentkeepalive: MockHttp,

   'https-proxy-agent': mockHttpAgent('https-proxy'),

   'http-proxy-agent': mockHttpAgent('http-proxy'),

-  'socks-proxy-agent': mockHttpAgent('socks-proxy'),

+  'socks-proxy-agent': { SocksProxyAgent: mockHttpAgent('socks-proxy') },

 })

Ok, thanks a lot. I'll fix it

[Janlaywss]

I forgot about tests when I clicked approve. This should fix things. You'll also need to npm run lintfix.

diff --git a/test/agent.js b/test/agent.js
index 0b721d7..f8def79 100644
--- a/test/agent.js
+++ b/test/agent.js
@@ -9,7 +9,7 @@ const agent = t.mock('../lib/agent.js', {
   agentkeepalive: MockHttp,
   'https-proxy-agent': mockHttpAgent('https-proxy'),
   'http-proxy-agent': mockHttpAgent('http-proxy'),
-  'socks-proxy-agent': mockHttpAgent('socks-proxy'),
+  'socks-proxy-agent': { SocksProxyAgent: mockHttpAgent('socks-proxy') },
 })

The test case has been modified. After modification, my local npm run lintfix all passed.

[wraithgar]

Thanks, this should go out w/ the next npm cli release.