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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create a `requireAuth` wrapper for `verifyToken`, `populateUser`, `restrictToAuth` #118

Closed
ekryski opened this Issue Mar 24, 2016 · 3 comments

Comments

Projects
None yet
3 participants
@ekryski
Copy link
Member

ekryski commented Mar 24, 2016

Saving key strokes is what we like doing. 馃槃

@ekryski

This comment has been minimized.

Copy link
Member Author

ekryski commented Mar 24, 2016

Dependent on #109 getting merged.

@ekryski ekryski added the Task label Mar 26, 2016

@marshallswain

This comment has been minimized.

Copy link
Member

marshallswain commented Apr 3, 2016

Do we want to create a general way to allow people to created nested hooks?

@daffl daffl self-assigned this May 18, 2016

@daffl daffl added ready and removed Discussion labels May 19, 2016

@ekryski ekryski added Discussion and removed ready labels May 21, 2016

@ekryski

This comment has been minimized.

Copy link
Member Author

ekryski commented Oct 26, 2016

Closing as those hooks have been re-shuffled into the new authentication chain so you won't need to repeat them.

@ekryski ekryski closed this Oct 26, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can鈥檛 perform that action at this time.
You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session.