What you would like to be added?
I’d like to add focused unit tests for helper functions in pkg/statusserver, specifically:
extractRawToken
TokenAudience
StatusUrl
I also noticed that extractRawToken currently uses exact string splitting on "Bearer <token>", which is a bit brittle for extra whitespace or lowercase bearer. If maintainers agree, I’d like to make that parsing slightly more robust while keeping the behavior simple.
Why is this needed?
These helpers are small but important for status server auth and request path generation. Adding direct unit tests would improve confidence in the status server code and make this logic easier to maintain.
What you would like to be added?
I’d like to add focused unit tests for helper functions in
pkg/statusserver, specifically:extractRawTokenTokenAudienceStatusUrlI also noticed that
extractRawTokencurrently uses exact string splitting on"Bearer <token>", which is a bit brittle for extra whitespace or lowercasebearer. If maintainers agree, I’d like to make that parsing slightly more robust while keeping the behavior simple.Why is this needed?
These helpers are small but important for status server auth and request path generation. Adding direct unit tests would improve confidence in the status server code and make this logic easier to maintain.