Experimenter not redirecting on missing trailing slash
mikewilli opened this issue · comments
The Experimenter API is erroneously responding with "Please login using OpenID Connect".
jlockhart Edit: Experimenter should redirect if a trailing slash is missing to a url including a trailing slash but for some reason since we updated to Django 5 it's no longer doing that on prod, but I am unable to reproduce this locally.
┆Issue is synchronized with this Jira Task
I'm unable to reproduce this either in my browser with an authenticated or unauthenticated session or from using requests from a command line. Are you still able to reproduce this @mikewilli ?
Nope, looks like it fixed itself!
Ok, this is happening again now so I'm going to reopen. Can you take a look again when you have a chance?
Ah, so what's happening here is ommitting the trailing slash / will not redirect but rather just halt on the openidc error. Let's get the redirect working again. This was probably disrupted as part of the django 5 update.
Also this would have been clearer if a URL had been included in the original bug.
Oh whoops, I sent it in the original chat message but forgot it here, sorry!
I can't seem to recreate this locally. No matter what combination of DEBUG=False/True, nginx.conf X-Forwarded-For, make up/up_prod, I can't make my local NOT redirect correctly. So I can't figure out why it's not redirecting in prod.