Skip to content

fix(server): include auth challenge on typed 401#26455

Merged
kitlangton merged 1 commit into
devfrom
kit/httpapi-auth-challenge-header
May 9, 2026
Merged

fix(server): include auth challenge on typed 401#26455
kitlangton merged 1 commit into
devfrom
kit/httpapi-auth-challenge-header

Conversation

@kitlangton
Copy link
Copy Markdown
Contributor

Summary

  • Add WWW-Authenticate to typed HttpApi basic-auth failures.
  • Cover missing and invalid credentials with the authorization middleware test.

Test

  • bun test --timeout 5000 test/server/httpapi-authorization.test.ts test/server/httpapi-instance.test.ts test/server/httpapi-sync.test.ts
  • bun typecheck

@kitlangton kitlangton force-pushed the kit/test-rescue-instance-bootstrap branch 9 times, most recently from d26a17b to 5e288fa Compare May 9, 2026 03:49
Base automatically changed from kit/test-rescue-instance-bootstrap to dev May 9, 2026 03:53
@kitlangton kitlangton force-pushed the kit/httpapi-auth-challenge-header branch from 0bde4fc to fc45533 Compare May 9, 2026 04:12
@kitlangton kitlangton marked this pull request as ready for review May 9, 2026 04:15
@kitlangton kitlangton enabled auto-merge (squash) May 9, 2026 04:15
@kitlangton kitlangton disabled auto-merge May 9, 2026 04:15
@kitlangton kitlangton merged commit 8cbc43f into dev May 9, 2026
6 of 9 checks passed
@kitlangton kitlangton deleted the kit/httpapi-auth-challenge-header branch May 9, 2026 04:15
katosun2 pushed a commit to katosun2/opencode that referenced this pull request May 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant