Skip to content

fix(auth): protect bare api root path#43

Open
vernonstinebaker wants to merge 1 commit intonullclaw:mainfrom
vernonstinebaker:fix/auth-api-root
Open

fix(auth): protect bare api root path#43
vernonstinebaker wants to merge 1 commit intonullclaw:mainfrom
vernonstinebaker:fix/auth-api-root

Conversation

@vernonstinebaker
Copy link
Copy Markdown
Contributor

Summary

  • fix auth.isPublicPath() so bare /api is treated as a protected path instead of falling through as public
  • add a regression test for /api alongside the existing /api/status protected-path coverage
  • close a small but real auth boundary gap in the server auth gate, which checks isPublicPath(target) before requiring a bearer token
  • keep the change narrowly scoped to auth-path classification

Validation

  • zig build test -Dembed-ui=false -Dbuild-ui=false --summary all
  • npm --prefix ui ci --no-audit --no-fund
  • npm --prefix ui run build
  • bash tests/test_e2e.sh
  • zig fmt --check src/
  • git diff --check

Notes

  • this is a production fix plus regression test, not just a test-only change
  • the behavior is consistent with the existing intent that API paths require auth unless explicitly public (/health)
  • follow-up Phase 5 slices will continue with orchestration proxy coverage, installer failure/cleanup coverage, and service helper boundary coverage

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant