Skip to content

Commit a8cac42

Browse files
etrclaude
andcommitted
TASK-054: mark all action items done and set Status: Done
Check all six action-item checkboxes in the TASK-054 section of v2-deferred-backlog-plan.md and add **Status:** Done, reflecting that the auth_handler_ptr optional migration is fully implemented. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent db36c0f commit a8cac42

1 file changed

Lines changed: 8 additions & 6 deletions

File tree

specs/tasks/v2-deferred-backlog-plan.md

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -107,22 +107,22 @@ TODO comment marking this migration already lives at
107107
`src/httpserver/create_webserver.hpp:92-99`.
108108

109109
**Action Items:**
110-
- [ ] Define `auth_handler_ptr` as
110+
- [x] Define `auth_handler_ptr` as
111111
`std::function<std::optional<http_response>(const http_request&)>` in
112112
`create_webserver.hpp`. Keep the old typedef alias for one transitional
113113
build with a `[[deprecated]]` attribute pointing at the new shape.
114-
- [ ] Update the auth short-circuit path in `webserver_dispatch.cpp` to
114+
- [x] Update the auth short-circuit path in `webserver_dispatch.cpp` to
115115
consume `optional<http_response>` instead of dereferencing a
116116
`shared_ptr`.
117-
- [ ] Update the documented v1 alias surface (`auth_handler` setter) to
117+
- [x] Update the documented v1 alias surface (`auth_handler` setter) to
118118
internally adapt v1 callers that still produce a `shared_ptr` (free
119119
function `compat::adapt_legacy_auth(...)`), with a `[[deprecated]]`
120120
warning.
121-
- [ ] Update `examples/centralized_authentication.cpp` to return
121+
- [x] Update `examples/centralized_authentication.cpp` to return
122122
`std::optional<http_response>` directly (drop `std::make_shared`).
123-
- [ ] Update Doxygen, README "Centralized authentication" section, and
123+
- [x] Update Doxygen, README "Centralized authentication" section, and
124124
RELEASE_NOTES.md "Migration notes" with the new signature.
125-
- [ ] Remove the TODO comment at `create_webserver.hpp:92-99`.
125+
- [x] Remove the TODO comment at `create_webserver.hpp:92-99`.
126126

127127
**Dependencies:**
128128
- Blocked by: TASK-046 (by-value response cutover, Done)
@@ -141,6 +141,8 @@ TODO comment marking this migration already lives at
141141
**Related Findings:** task-036 #38, task-030 #18
142142
**Related Decisions:** DR-009, PRD-RSP-REQ-007
143143

144+
**Status:** Done
145+
144146
---
145147

146148
## TASK-055 — DR-009 revision: default error body must not surface `e.what()`

0 commit comments

Comments
 (0)