Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Basic Auth is broken on sandboxes #388

Closed
robrap opened this issue Aug 9, 2023 · 6 comments
Closed

Basic Auth is broken on sandboxes #388

robrap opened this issue Aug 9, 2023 · 6 comments
Labels
escalate-to-psre Create a PSRE ticket for this issue esre

Comments

@robrap
Copy link
Contributor

robrap commented Aug 9, 2023

It seems sandboxes are broken with Basic Auth is enabled in Jenkins (which is the default). The problem seems to be that the subsequent requests from the MFE have the header 'Authorization: Basic xxxxxxx' included, and these return an invalid username/password error. I am unable to log in via the AuthN MFE.

Is it possible that this used to work by not passing this header on to Django? Other?

I'm escalating this to SRE because I think this is owned by SRE.

Note: This replaces https://2u-internal.atlassian.net/browse/DOS-3917 which I will close.
@robrap robrap added this to Arch-BOM Aug 9, 2023
@robrap robrap converted this from a draft issue Aug 9, 2023
@robrap robrap added the escalate-to-psre Create a PSRE ticket for this issue label Aug 9, 2023
@github-actions
Copy link

github-actions bot commented Aug 9, 2023

This is now https://2u-internal.atlassian.net/browse/PSRE-2704

@jmbowman jmbowman added the esre label Aug 9, 2023
@jmbowman jmbowman moved this to Blocked in Arch-BOM Aug 9, 2023
@jmbowman
Copy link

jmbowman commented Aug 9, 2023

Marking as blocked while we wait for PSRE to address.

@robrap
Copy link
Contributor Author

robrap commented Aug 15, 2023

https://github.com/edx/sandbox-internal/pull/315 is an alternative solution. I will also note this on the PSRE ticket.

@robrap robrap closed this as completed Aug 16, 2023
@github-project-automation github-project-automation bot moved this from Blocked to Done in Arch-BOM Aug 16, 2023
@robrap
Copy link
Contributor Author

robrap commented Aug 16, 2023

Dropping BasicAuthentication from sandboxes as a default class fixed the issue. Closing this ticket.

@robrap
Copy link
Contributor Author

robrap commented Aug 22, 2023

@rgraber: This issue is fixed. I commented as such on the PSRE ticket, but Adam still had a long comment on there, so I didn't want to just mark as Done. Should we follow-up or just forget about it?

@rgraber
Copy link
Contributor

rgraber commented Aug 23, 2023

Given that the solution turned out to be pretty easy it seems like, I think we can ignore it. Reading his comment, it looks like he was more concerned with what to do if it turned out to be hard.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
escalate-to-psre Create a PSRE ticket for this issue esre
Projects
Arch-BOM
Archived in project
Milestone
No milestone
Development

No branches or pull requests
3 participants
@jmbowman @rgraber @robrap