Skip to content

Conversation

@sizzlemctwizzle
Copy link
Member

Works for me locally. I'm not sure if it will work in production. It uses the "Referer" header and I'm not sure if Nodejitsu handles that properly.

sizzlemctwizzle added a commit that referenced this pull request Mar 18, 2015
Redirect the user to their previous page after login/logout.
@sizzlemctwizzle sizzlemctwizzle merged commit d94380e into master Mar 18, 2015
@sizzlemctwizzle
Copy link
Member Author

Only one way to know. Deploying...

@sizzlemctwizzle
Copy link
Member Author

Tested and it apparently works!

@Martii
Copy link
Member

Martii commented Mar 18, 2015

Nice addition. :)

@Martii Martii added enhancement Something we do have implemented already but needs improvement upon to the best of knowledge. CODE Some other Code related issue and it should clearly describe what it is affecting in a comment. labels Mar 18, 2015
@Martii
Copy link
Member

Martii commented Mar 18, 2015

Btw testing if referer is blocked momentarily to see what happens.

@Martii
Copy link
Member

Martii commented Mar 18, 2015

Blocked referer header redirects to / ... so perfect.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 19, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

CODE Some other Code related issue and it should clearly describe what it is affecting in a comment. enhancement Something we do have implemented already but needs improvement upon to the best of knowledge.

Development

Successfully merging this pull request may close these issues.

3 participants