graphql_devise icon indicating copy to clipboard operation
graphql_devise copied to clipboard

Published 20 hours ago • verified publisher icon graphql-devise

Support DTA's authorization header

Open artplan1 opened this issue 3 years ago • 1 comments
trafficstars

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

artplan1 avatar Aug 08 '22 15:08 artplan1

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

00dav00 avatar Aug 09 '22 06:08 00dav00