[BUG] Test coverage for -DCASE_INSENSITIVE is lacking. #310

Activity

1. bcsgh

   added

   bugConfirmed bugs or reports that are very likely to be bugs.Confirmed bugs or reports that are very likely to be bugs.

   on Apr 6, 2023
2. bcsgh

   assigned

   etr

   on Apr 6, 2023
3. bcsgh

   added a commit that references this issue on Oct 27, 2023

   Fix issue etr#310, but may break existing users.

   d3c6de2
4. etr commented on Jul 23, 2026

   

   etr

   on Jul 23, 2026

   Owner

   More actions

   Update for libhttpserver 2.0 — partially addressed, but leaving this open.

   The dispatch-time impact is gone: v2's 3-tier route table is now the sole routing oracle and is consistently case-sensitive by default. It replaces the v1 registered_resources ordered map whose key comparator (http_endpoint::operator<) was the unconditionally case-insensitive one you flagged. test/integ/basic.cpp also gained #ifdef CASE_INSENSITIVE routing probes.

   However, the underlying gap this issue is about remains:

   • There's still no --enable-case-insensitive configure option and no CI lane that builds with -DCASE_INSENSITIVE, so those probes are never compiled or exercised.
   • http_endpoint::operator< is still unconditionally case-insensitive via std::toupper (now vestigial, but unchanged).

   Leaving this open to track adding a real -DCASE_INSENSITIVE build + test lane.