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

Why not authProtect the destroy handler? #8

Closed
golliher opened this issue Nov 15, 2016 · 2 comments
Closed

Why not authProtect the destroy handler? #8

golliher opened this issue Nov 15, 2016 · 2 comments

Comments

@golliher
Copy link
Contributor

I am excited by the potential of authboss, so I have been studying your documentation and the example in order to learn.

I was curious is there is a reason the destroy handler doesn't use authProtect? I tried adding the protection and it seems to work ok, but since I'm still learning I thought I'd ask.

@aarondl
Copy link
Member

aarondl commented Nov 15, 2016

Yeah, there's no reason for it not to be. Feel free to PR that since you already have the code :)

@golliher
Copy link
Contributor Author

Done.

@aarondl aarondl closed this as completed Dec 28, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants