search

vapor / auth

👤 Authentication and Authorization framework for Fluent.
53 stars 34 forks source link

Remove error thowing in `TokenAuthenticationMiddleware` #42

Closed Sorix closed 6 years ago

Sorix commented 6 years ago

Now TokenAuthenticationMiddleware authenticate user and doesn't throw error if authentication failed. Don't forget to use GuardAuthenticationMiddleware or .requireAuthenticated() to protect data.

Fix #40

0xTim commented 6 years ago

As mentioned on Discord it would be good to see a test for this - even if it is just adding it to a route group to make sure it doesn't throw

penny-coin commented 6 years ago

Hey @Sorix, you just merged a pull request, have a coin!

You now have 1 coins.

tanner0101 commented 6 years ago

(I agree a test would be nice)