dangernoodle-io / breadboard / 27045138668

100%

push

github

fix(bb_http): return 404 for unregistered URIs instead of 405 (#347)

the OPTIONS /* cors preflight wildcard caused esp-idf httpd to return
405 for any unregistered non-options request, because the wildcard
"wins" the method check even when no exact handler exists.

- add `bb_http_uri_is_registered(uri)` to `route_registry.c`: iterates
  the described-route registry, applying wildcard-prefix semantics, and
  explicitly excludes the internal OPTIONS /* and GET /* catch-alls so a
  bogus path under them still reads as unregistered
- register a custom HTTPD_405_METHOD_NOT_ALLOWED error handler in
  `bb_http_server_ensure_started` that calls the predicate: unregistered
  → send 404; genuinely wrong-method on a registered uri → keep 405
- host tests: 9 cases covering exact match, bogus path, post-only route,
  wildcard route sub-path, catch-all exclusion, query-string stripping,
  null/empty inputs

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

1386 of 1386 branches covered (100.0%)

Branch coverage included in aggregate %.

24 of 24 new or added lines in 1 file covered. (100.0%)

2348 of 2348 relevant lines covered (100.0%)

930.93 hits per line