Proxy autodetection doesn't work with websockets/SignalR

[ruben-ivre]

When using websockets (SignalR in my case) the proxy autodetection doesn't work, as in the proxy is never detected.
I think it doesn't run at all, as I added a breakpoint in ProxyDetector.OnRequestEvent and it didn't trigger.
The logs from that event doesn't appear either. websockets-proxy-autodetect.log

My guess is that something in the websockets request is different and that Resend event is never used.

For now, I'm running the proxy detectors manually and setting the proxy if so, which works perfectly fine.

[Benedicht]

at Proxies.Autodetect.ProxyDetector.Detach () [0x00001] in ProxyDetector.cs:96
at Shared.HTTPManager.set_ProxyDetector (Proxies.Autodetect.ProxyDetector value) [0x00001] in HTTPManager.cs:80
at Engage.Websockets.BestHttpSignalRClient.Connect (System.String url, System.String userToken, System.Action subscribeEventsCallbacks) [0x0004d] in BestHttpSignalRClient.cs:37

There were a call to set a new ProxyDetector from BestHttpSignalRClient.cs:37. What was the new value? A null value effectively disables proxy detection.

[ruben-ivre]

at Proxies.Autodetect.ProxyDetector.Detach () [0x00001] in ProxyDetector.cs:96
at Shared.HTTPManager.set_ProxyDetector (Proxies.Autodetect.ProxyDetector value) [0x00001] in HTTPManager.cs:80
at Engage.Websockets.BestHttpSignalRClient.Connect (System.String url, System.String userToken, System.Action subscribeEventsCallbacks) [0x0004d] in BestHttpSignalRClient.cs:37

There were a call to set a new ProxyDetector from BestHttpSignalRClient.cs:37. What was the new value? A null value effectively disables proxy detection.

BestHttpSignalRClient.cs:37 is HTTPManager.ProxyDetector = new Best.HTTP.Proxies.Autodetect.ProxyDetector();

We set it every time before connecting. We also tried doing it only once but didn't make any difference.

[Benedicht]

Hmm, i'm going to check it. However, is there a reason you set it manually? The plugin already setting it to a ProxyDetector instance in its private static constructor. You should only set it if you want to costumize or disable it.

[ruben-ivre]

Hmm, i'm going to check it. However, is there a reason you set it manually? The plugin already setting it to a ProxyDetector instance in its private static constructor. You should only set it if you want to costumize or disable it.

Oh, no particular reason, I thought it was necessary, documentation says you need to do it: https://bestdocshub.pages.dev/Shared/proxy/proxy-autodetect/

[Benedicht]

Ohh, i think i copy-pasted from the old documentation for v2, and i might enabled it by default in v3...
I'm going to update the documentation as well.