Skip to content
This repository has been archived by the owner on Feb 27, 2023. It is now read-only.

Support setting optional JWT headers #126

Closed
ekristen opened this issue Nov 21, 2016 · 4 comments
Closed

Support setting optional JWT headers #126

ekristen opened this issue Nov 21, 2016 · 4 comments

Comments

@ekristen
Copy link

See RFC -- https://tools.ietf.org/html/draft-ietf-oauth-json-web-token-32#section-5

While the typ header is optional it is good practice when creating JWT tokens to set typ to JWT (see https://tools.ietf.org/html/draft-ietf-oauth-json-web-token-32#section-3.1)

@shaxbee
Copy link
Contributor

shaxbee commented Nov 22, 2016

I initially didn't implement it as it wasn't required.
Will add the necessary tpe header shortly.

On Tue, Nov 22, 2016, 6:23 AM Erik Kristensen notifications@github.com
wrote:

See RFC --
https://tools.ietf.org/html/draft-ietf-oauth-json-web-token-32#section-5

While the typ header is optional it is good practice when creating JWT
tokens to set typ to JWT (see
https://tools.ietf.org/html/draft-ietf-oauth-json-web-token-32#section-3.1
)


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#126, or mute the thread
https://github.com/notifications/unsubscribe-auth/AE_mfTeWeYwt9vLbE8JvBP6AwEd3ta3fks5rAhnTgaJpZM4K4xa0
.

@ekristen
Copy link
Author

Would be nice to support all the other headers that are acceptable in a JWS which is used by JWT https://tools.ietf.org/html/draft-ietf-jose-json-web-signature-41#page-11

@shaxbee
Copy link
Contributor

shaxbee commented Nov 29, 2016 via email

@mitar
Copy link

mitar commented Jun 3, 2021

Ping on this?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

6 participants