Skip to content
This repository has been archived by the owner on Nov 16, 2020. It is now read-only.

TokenAuthenticationMiddleware still throws an Abort (is not optional) #40

Closed
andrewtheis opened this issue May 10, 2018 · 0 comments
Closed

Comments

@andrewtheis
Copy link

andrewtheis commented May 10, 2018

In 2.0.0-RC.4, work was done to allow multiple authentication middlewares to exist for a route optionally. However, for TokenAuthenticationMiddleware the following line still throws an Abort:

let token = try req.requireAuthenticated(A.TokenType.self)

This should be try? so TokenAuthenticationMiddleware works optionally as intended.

@andrewtheis andrewtheis changed the title TokenAuthenticationMiddleware still requires a token TokenAuthenticationMiddleware still throws an Abort (is not optional) May 10, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant