Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refresh Token header definition is missing #204

Open
fmarino-ipzs opened this issue Jan 15, 2024 · 1 comment
Open

Refresh Token header definition is missing #204

fmarino-ipzs opened this issue Jan 15, 2024 · 1 comment
Assignees
Labels
documentation Improvements or additions to documentation

Comments

@fmarino-ipzs
Copy link
Collaborator

In the documentation, only the body of the RT is defined. We have to define the header as well. In particular the typ parameter.

@fmarino-ipzs fmarino-ipzs added the documentation Improvements or additions to documentation label Jan 15, 2024
@peppelinux
Copy link
Member

peppelinux commented Jan 15, 2024

In OAuth 2.0, Refresh Tokens are not JWTs (JSON Web Tokens) and therefore do not have a JWT header. Refresh Tokens are opaque strings that are issued by the authorization server and can be used to obtain a new access token when the current one expires. They are typically long strings of characters that do not contain any meaningful information to the client.

However, the implementation that uses JWTs as access and refresh tokens in OpenID Conenct, as the italian one, the typ (type) header parameter in the JWT would typically be set to JWT. This is defined in the JWT specification, RFC 7519.

other cases are JWT that have specific typ value registered in IANA and defined in IETF/OpenID specs.
Access Tokens have their typ value, according to https://datatracker.ietf.org/doc/html/rfc9068, while refresh token doesn't.

if we want to define the typ value for the refresh token we should propose an IETF draft. Otherwise we can omit the typ value or set it to JWT allowing missing typ value, since when missing the default value is implicitly set to JWT.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
Development

No branches or pull requests

3 participants