X Tutup
Skip to content

For HTTP/1.1 requests, enable keep-alive by default.#7

Merged
thebentern merged 1 commit intomeshtastic:masterfrom
cpatulea:master
Jan 22, 2026
Merged

For HTTP/1.1 requests, enable keep-alive by default.#7
thebentern merged 1 commit intomeshtastic:masterfrom
cpatulea:master

Conversation

@cpatulea
Copy link
Copy Markdown

Keep-alive became the default in 1.1:
https://developer.mozilla.org/en-US/docs/Web/HTTP/Reference/Headers/Connection

This is confirmed in Python HTTP server:
https://github.com/python/cpython/blob/38f0114ff1f8d07b151639ff30b4409814422520/Lib/http/server.py#L367

This will improve connection reuse with Chrome, which does not send explicit 'Connection: keep-alive' and just relies on the server default.

Tested on ESP32 Heltec v3.

Keep-alive became the default in 1.1:
https://developer.mozilla.org/en-US/docs/Web/HTTP/Reference/Headers/Connection

This is confirmed in Python HTTP server:
https://github.com/python/cpython/blob/38f0114ff1f8d07b151639ff30b4409814422520/Lib/http/server.py#L367

This will improve connection reuse with Chrome, which does not send explicit
'Connection: keep-alive' and just relies on the server default.

Tested on ESP32 Heltec v3.
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Jan 15, 2026

CLA assistant check
All committers have signed the CLA.

@cpatulea
Copy link
Copy Markdown
Author

CLA is now signed

Copy link
Copy Markdown

@esev esev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@thebentern thebentern merged commit b0f3960 into meshtastic:master Jan 22, 2026
1 check passed
@cpatulea
Copy link
Copy Markdown
Author

@thebentern thanks for merging. I noticed that the firmware repo uses renovate to bump the esp32_https_server version. Is that something I need to trigger manually? or it happens automatically on some schedule? I tried to figure this out myself, I found the renovate.json file but I'm still not sure how it is triggered.

@cpatulea
Copy link
Copy Markdown
Author

Looks like it got merged yesterday meshtastic/firmware#9393 - thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

X Tutup