Skip to content

feat(auth): implement auth error specification compliance#348

Merged
ngjunsiang merged 2 commits intoweeklyfrom
refactor/auth-error-spec
Feb 9, 2026
Merged

feat(auth): implement auth error specification compliance#348
ngjunsiang merged 2 commits intoweeklyfrom
refactor/auth-error-spec

Conversation

@nycomp
Copy link
Contributor

@nycomp nycomp commented Feb 9, 2026

Summary

Implements Campus auth error envelope format while maintaining OAuth 2.0 compliance (RFC 6749).

Changes

  • Add envelope_format parameter to OAuthError.to_dict() for dual format support
  • Add 12 AUTH_* error constants to ErrorConstant enum
  • Update handle_token_error() to use Campus envelope format
  • Add validate_types() function that returns FieldError list
  • Add comprehensive error response tests (11 new test cases)
  • Create OpenAPI spec with error response schemas
  • Add oauth_error properties to Python client APIError

Reference

🤖 Generated with Claude Code

ngjunsiang and others added 2 commits February 9, 2026 19:04
Implement Campus auth error envelope format while maintaining OAuth 2.0
compliance (RFC 6749).

Changes:
- Add envelope_format parameter to OAuthError.to_dict() for dual format support
- Add 12 AUTH_* error constants to ErrorConstant enum
- Update handle_token_error() to use Campus envelope format
- Add validate_types() function that returns FieldError list
- Add comprehensive error response tests (11 new test cases)
- Create OpenAPI spec with error response schemas
- Add oauth_error properties to Python client APIError

Reference: campus/auth/docs/auth-error-spec.md

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Remove api-error-spec-refactor-plan.md and auth-error-spec-refactor-plan.md.
All technical details are now captured in the specifications, code docstrings,
OpenAPI specs, and test files.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@ngjunsiang ngjunsiang merged commit 59593dd into weekly Feb 9, 2026
4 checks passed
@ngjunsiang ngjunsiang deleted the refactor/auth-error-spec branch February 9, 2026 11:17
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.

2 participants

Comments