mcordell / grape_token_auth

Token auth for grape apps
MIT License
52 stars 19 forks source link

Modify session api to used `authed_with_token` data key #44

Closed mcordell closed 8 years ago

mcordell commented 8 years ago

Change the session API so that it doesn't set the authentication headers within the sign in end point. Instead, use the flag on Authorizer Data, to indicate that the user was not authenticated with a token, which causes the headers to be created within the middleware.

Fix one for #39

There is a double request for uniqueness because validation is checked twice. Unclear what else should change.

qd3v commented 8 years ago

Hi! Sorry, just got back to project, found your update. It seems this is really fixing the problem: w/o: with: with Thanks! Also, can you please take a look at #43 , I spent several hours to find it and fix :)