Remove the unused Auth() method from internal/middleware/middleware.go.
This method only logged "AUTH: before request" and passed through to the next handler — it performed no actual authentication. It was never referenced anywhere in the codebase; authentication is handled per-handler via requireAuth in the handlers package.
Remove the unused `Auth()` method from `internal/middleware/middleware.go`.
This method only logged "AUTH: before request" and passed through to the next handler — it performed no actual authentication. It was never referenced anywhere in the codebase; authentication is handled per-handler via `requireAuth` in the handlers package.
closes #38
<!-- session: agent:sdlc-manager:subagent:629a7621-ec4b-49af-b7e8-03141664d682 -->
The Auth() method in internal/middleware/middleware.go only logged and
passed through without performing any actual authentication. It was
never referenced anywhere in the codebase — authentication is handled
per-handler via requireAuth in the handlers package.
closes#38
Auth() method (14 lines) fully removed from internal/middleware/middleware.go
No references to Auth() remain anywhere in the codebase (grep confirmed)
All imports still used by remaining methods (CORS, Logging, Metrics, MetricsAuth)
No other files modified — single file, deletion only
No changes to Makefile, linter config, or tests
docker build . passes (fmt-check, lint, test, build all green)
README unaffected (no mention of Auth() middleware)
Rebased onto main (was 1 commit behind due to #64).
Labeling merge-ready and assigning to @sneak for final merge.
## Review: PASS ✅
Clean, minimal removal of dead code.
**Verified:**
- `Auth()` method (14 lines) fully removed from `internal/middleware/middleware.go`
- No references to `Auth()` remain anywhere in the codebase (`grep` confirmed)
- All imports still used by remaining methods (`CORS`, `Logging`, `Metrics`, `MetricsAuth`)
- No other files modified — single file, deletion only
- No changes to Makefile, linter config, or tests
- `docker build .` passes (fmt-check, lint, test, build all green)
- README unaffected (no mention of `Auth()` middleware)
Rebased onto `main` (was 1 commit behind due to [#64](https://git.eeqj.de/sneak/chat/pulls/64)).
Labeling `merge-ready` and assigning to @sneak for final merge.
<!-- session: agent:sdlc-manager:subagent:498189b6-365c-4853-bbd7-0c15dd7cc4d6 -->
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Remove the unused
Auth()method frominternal/middleware/middleware.go.This method only logged "AUTH: before request" and passed through to the next handler — it performed no actual authentication. It was never referenced anywhere in the codebase; authentication is handled per-handler via
requireAuthin the handlers package.closes #38
15d85d169eto18750f107aReview: PASS ✅
Clean, minimal removal of dead code.
Verified:
Auth()method (14 lines) fully removed frominternal/middleware/middleware.goAuth()remain anywhere in the codebase (grepconfirmed)CORS,Logging,Metrics,MetricsAuth)docker build .passes (fmt-check, lint, test, build all green)Auth()middleware)Rebased onto
main(was 1 commit behind due to #64).Labeling
merge-readyand assigning to @sneak for final merge.