1
78%
master: 78%

Ran 18 Jun 2025 02:08PM UTC

Files 280

Run time 21s

Badge

Embed ▾

Committed 18 Jun 2025 01:52PM UTC coverage: 77.732% (+0.006%) from 77.726%

Job # 15734771733.1

Build Type

push

github

Committed by

web-flow

Commit Message

Added check to handle token parsing with claims without "sub". (#3287)

### Description 
In the oidc_introspection.py `func (filter *oidcIntrospectionFilter)
Request(ctx filters.FilterContext)` function get value form the Claim
map using `sub` key and then casts the value into a string, but this
would panic if no sub key is present. I added a check to ensure we only
cast values if sub key is present.

### Linked Issue 
closes #3216 

### Approach to solution
As describe in the following
[comment](https://github.com/zalando/skipper/issues/3216#issuecomment-2433498646),
I dug a bit deeper and found 2 suitable values to use if subject is not
present in token. One is the Subject value in tokenContainer struct or
UserInfo Struct. I decided to go with the tokenContainer struct. If this
is not correct please do guide me if UserInfo Struct would be better.

---------

Signed-off-by: wassafshahzad <wassafshahzad@gmail.com>

Run Details

23318 of 29998 relevant lines covered (77.73%)

102012.67 hits per line

zalando / skipper / 15734771733 / 1
78%
master: 78%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 15734771733.1

zalando / skipper / 15734771733 / 1 78% master: 78%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 15734771733.1

zalando / skipper / 15734771733 / 1
78%
master: 78%

README BADGES
x