Remove explanatory text after //nolint:goconst directives to be
consistent with the rest of the codebase. The codebase pattern is to
use just //nolint:goconst without additional comments.
This matches the style used in controller_test.go and other test files
throughout the project.
Co-authored-by: rchincha <45800463+rchincha@users.noreply.github.com>
The "sub" string in bearer_oidc.go is the standard OIDC claim name
defined by the specification and should remain as a literal.
The "test-user" string in bearer_oidc_test.go is test fixture data
that doesn't benefit from being extracted to a constant.
Both warnings are suppressed with //nolint:goconst directives.
Co-authored-by: rchincha <45800463+rchincha@users.noreply.github.com>
- Add unit tests for OIDCBearerAuthorizer
- Test configuration validation
- Test token authentication with various scenarios
- Test custom claim mapping
- Test multiple audiences support
- Test token expiry and invalid tokens
- All tests passing
Co-authored-by: rchincha <45800463+rchincha@users.noreply.github.com>
copilot-swe-agent[bot]