Skip to content
This repository has been archived by the owner on Oct 13, 2021. It is now read-only.

Add plug RequireLogin #202

Closed
paulcsmith opened this issue Apr 23, 2016 · 0 comments
Closed

Add plug RequireLogin #202

paulcsmith opened this issue Apr 23, 2016 · 0 comments
Assignees

Comments

@paulcsmith
Copy link
Contributor

We fetch the current user, but do not require login. We should add a plug for requiring login on authenticated pages. https://github.com/thoughtbot/constable-api/blob/master/web/router.ex#L13

Right now if someone isn't logged in the page 500s. Instead it should redirect to the sign in with a flash message

@paulcsmith paulcsmith changed the title Add plug RequireLoging Add plug RequireLogin Apr 23, 2016
@paulcsmith paulcsmith added this to the Use Phoenix generated HTML milestone Apr 25, 2016
@joshuaclayton joshuaclayton self-assigned this Apr 28, 2016
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

2 participants