openid-certification / oidctest

THE CERTIFICATION TEST SUITE HAS BEEN MIGRATED TO A NEW SERVICE https://www.certificatinon.openid.net
Other
50 stars 15 forks source link

token_type response parameter checks #89

Closed zandbelt closed 5 years ago

zandbelt commented 6 years ago

Wrt. to the token_type parameter in the authentication response we could do 2 things:

panva commented 6 years ago

+1 for writing a test for token_type parameter existence and value

zandbelt commented 6 years ago

added a PR for this test to Roland's repo: https://github.com/rohe/oidctest/pull/108

zandbelt commented 6 years ago

we're ditching the PR with the new token_type test, see: https://github.com/rohe/oidctest/pull/108#issuecomment-417946995

I'll add some code shortly to verify (all) regular responses.

zandbelt commented 6 years ago

It appears that for flows where the access token is retrieved on the backchannel, the token_type parameter existence is already checked in pyoidc, see: https://github.com/OpenIDC/pyoidc/blob/v0.14.0/src/oic/oauth2/message.py#L1073

for Implicit/Hybrid I've added it here: https://github.com/OpenIDC/pyoidc/pull/565

zandbelt commented 6 years ago

https://github.com/OpenIDC/pyoidc/pull/565 was merged and this will be included in the next release of pyoidc > 0.14.0

zandbelt commented 5 years ago

this was deployed just now as part of OP 2.2.1 https://github.com/openid-certification/oidctest/releases/tag/v1.2.0