Login redirect from paths other than /ceda are incorrect

cedadev / archive_browser

Elasticsearch and javascript based file browser

1 stars 1 forks source link

Login redirect from paths other than /ceda are incorrect #115

Closed watucker closed 1 year ago

watucker commented 1 year ago

If you go to a data centre path other than /ceda, e.g. https://data.ceda.ac.uk/bodc/, the redirect from the login path is:

/account/signin/?r=http%3A%2F%2F127.0.0.1%3A8000%2F

This will redirect to localhost, which is not helpful.

watucker commented 1 year ago

Hi @dwest77a , this is an error that a user noticed earlier, would be good to get it fixed though it's not really causing any problems right now.

dwest77a commented 1 year ago

I have in dctheme_update a setting that disables the login/sign in option as I thought that was the plan? I can still look into this in case we need the login later?

dwest77a commented 1 year ago

The redirect url comes from request.build_absolute_uri() which for the live version at data.ceda.ac.uk I can't see why it would default to 127.0.0.1:8000.

dwest77a commented 1 year ago

No longer using login services for archive_browser so this is no longer an issue