Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

req.auth cleanup #2111

Merged
merged 2 commits into from Dec 6, 2019
Merged

req.auth cleanup #2111

merged 2 commits into from Dec 6, 2019

Conversation

@tanner0101
Copy link
Member

tanner0101 commented Dec 5, 2019

Related to https://github.com/vapor/jwt-kit/pull/1/files and vapor/jwt#110

Puts request authentication methods behind req.auth helper method.

Adds Request as argument to all request authenticators.

@tanner0101 tanner0101 added this to In Progress in Vapor 4 via automation Dec 5, 2019
@tanner0101 tanner0101 merged commit 9ecb1a2 into master Dec 6, 2019
2 checks passed
2 checks passed
xenial
Details
bionic
Details
Vapor 4 automation moved this from In Progress to Done Dec 6, 2019
@tanner0101 tanner0101 deleted the tn-req-auth branch Dec 6, 2019
kaishin added a commit to kaishin/vapor that referenced this pull request Jan 9, 2020
* req.auth cleanup

* fix tests + enable test discovery
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Vapor 4
  
Done
Linked issues

Successfully merging this pull request may close these issues.

None yet

1 participant
You can’t perform that action at this time.