Support DTA's authorization header

graphql-devise / graphql_devise

GraphQL interface on top devise_token_auth

MIT License

200 stars 39 forks source link

Support DTA's authorization header #234

Open artplan1 opened 2 years ago

artplan1 commented 2 years ago

Authorization header support was implemented in https://github.com/lynndylanhurley/devise_token_auth/pull/1534

This works in edge DTA, but seems like can't be merged until graphql_devise and DTA are properly synced

00dav00 commented 2 years ago

Hi @artplan1 this is an interesting addition! I think we need to think about backward compatibility since this header is not going to be present in all DTA versions.

Also, adding/editing some specs to tests the new header is working as expected would be handy here:

new header being returned on login
new header being used to authenticate
new header cleaned up/invalidated correctly on logout