search

meshtastic / firmware

Meshtastic device firmware
https://meshtastic.org
GNU General Public License v3.0
3.63k stars 909 forks source link

/api/v1/fromradio: add OPTIONS handler for CORS. #5386

Closed cpatulea closed 6 days ago

cpatulea commented 1 week ago

This avoids hitting the 404 Not Found handler, which breaks connection keep-alive, so this change fixes a big performance regression for Web Client in Chrome: https://github.com/meshtastic/firmware/issues/5385

Tested on Heltec V3.

Fixes #5385

CLAassistant commented 1 week ago

CLA assistant check
All committers have signed the CLA.

thebentern commented 1 week ago

Ah, so they get pre-flighted now? Good deal. Thanks for the fix. Once you sign the CLA, I can get this merged in.

cpatulea commented 6 days ago

Sorry, I need a bit more time for the CLA.

cpatulea commented 6 days ago

CLA is signed.