SignalK / SensESP

Universal Signal K sensor framework for the ESP32 platform
https://signalk.org/SensESP/
Apache License 2.0
154 stars 81 forks source link

Remove set_test_auth_on_each_connect builder member function #492

Closed mairas closed 2 years ago

mairas commented 2 years ago

This was an optimization that, when turned off, would make the websocket connection logic bypass testing the authorization token on reconnects. Since this is very much a corner case and made the already hairy connection logic even more complex, I think it's better to remove it.

ba58smith commented 2 years ago

I sure don't need it!

mairas commented 2 years ago

Tested by having a test program connect to the server and then shutting down and restarting the server process. Would seem to work fine.