Login redirects to index URL #70842

New Issue

Sybren A. Stüvel · 2019-10-15T15:36:25+02:00

Sybren A. Stüvel commented

2019-10-15 15:36:25 +02:00

Logging in brings you back to the index page at https://conference.blender.org/, instead of the page on which you logged in.

To reproduce:

Be logged out.
Visit any page, say https://conference.blender.org/2019/schedule
Click on "Login"
After logging in, you arrive at https://conference.blender.org/

Logging in brings you back to the index page at https://conference.blender.org/, instead of the page on which you logged in. To reproduce: * Be logged out. * Visit any page, say https://conference.blender.org/2019/schedule * Click on "Login" * After logging in, you arrive at https://conference.blender.org/

Sybren A. Stüvel commented

2019-10-15 15:36:25 +02:00

Added subscribers: @dr.sybren, @SemMulder, @fsiddi

Francesco Siddi commented

2019-10-15 16:04:22 +02:00

We have come across this issue in the past. Do we have a code example/best practice on how to achieve this?

For example, is there guidance on blender-id-oauth-client on how this can be implemented?
Can this be solved by using the {% url 'oauth:login' %} url?

I would be very happy if we could standardize this process.

We have come across this issue in the past. Do we have a code example/best practice on how to achieve this? For example, is there guidance on `blender-id-oauth-client` on how this can be implemented? Can this be solved by using the `{% url 'oauth:login' %}` url? I would be very happy if we could standardize this process.

Sybren A. Stüvel commented

2019-10-15 17:07:44 +02:00

blender-id-oauth-client should already do the right thing, by using the Referer HTTP header of the request to store the URL in the user's session. And indeed, {% url 'oauth:login' %} should be used instead of hard-coding /oauth/login.

`blender-id-oauth-client` should already do the right thing, by using the Referer HTTP header of the request to store the URL in the user's session. And indeed, `{% url 'oauth:login' %}` should be used instead of hard-coding `/oauth/login`.

Francesco Siddi commented

2019-10-15 17:37:37 +02:00

Changed status from 'Open' to: 'Resolved'

Francesco Siddi closed this issue

2019-10-15 17:37:37 +02:00

Francesco Siddi self-assigned this 2019-10-15 17:37:37 +02:00

Francesco Siddi commented

2019-10-15 17:37:37 +02:00

The issue appears to be fixed in f4d03e9796.

The issue appears to be fixed in f4d03e9796.

Sign in to join this conversation.