dangernoodle-io / breadboard / 25138089590

100%

push

github

jae/lower http max open sockets default (#156)

* chore(bb_http): lower BB_HTTP_MAX_OPEN_SOCKETS default 9 -> 5

The runtime clamp from PR #155 (fix: clamp max_open_sockets to LWIP_MAX_SOCKETS-3)
already enforces the safety bound at runtime. This PR aligns the Kconfig default
with the most-restrictive realistic LWIP socket pool: LWIP_MAX_SOCKETS=8 yields
a client cap of 5 (= 8 - 3 reserved). 5 is a safe lower bound that fits any
reasonable LWIP default ESP-IDF projects ship with (IDF default 10 → cap 7,
TaipanMiner's reduced 8 → cap 5). Menuconfig numbers now reflect runtime reality.

No release needed — picks up on next breadboard tag.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

* chore(bb_http): align help text with default of 5

The Kconfig default has been 5, but the help text still described it as
"Default 9". This aligns help text with reality.

The runtime clamp from #155 enforces max_open_sockets <= LWIP_MAX_SOCKETS-3,
so the menuconfig number now reflects the most-restrictive realistic LWIP
cap (LWIP=8 -> cap=5). No release needed — picks up on next bb tag.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>

336 of 336 branches covered (100.0%)

Branch coverage included in aggregate %.

527 of 527 relevant lines covered (100.0%)

487.36 hits per line