nebari-dev / nebari-slurm

An opinionated open source deployment of jupyterhub based on an Slurm job scheduler.

BSD 3-Clause "New" or "Revised" License

28 stars 10 forks source link

Add workaround for user management nav-link #168

Closed viniciusdc closed 3 months ago

viniciusdc commented 3 months ago

Reference Issues or PRs

167

What does this implement/fix?

This PR includes a short-term fix, to reject all requests made to the broken link present in the theme and redirect it to /auth by using one of Traefik middlewares.

Put a x in the boxes that apply

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds a feature)
[ ] Breaking change (fix or feature that would cause existing features not to work as expected)
[ ] Documentation Update
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build related changes
[ ] Other (please describe):

Testing

[ ] Did you test the pull request locally?
[ ] Did you add new tests?

Documentation

Access-centered content checklist

Text styling

[ ] The content is written with plain language (where relevant).
[ ] If there are headers, they use the proper header tags (with only one level-one header: H1 or # in markdown).
[ ] All links describe where they link to (for example, check the Nebari website).
[ ] This content adheres to the Nebari style guides.

Non-text content

[ ] All content is represented as text (for example, images need alt text, and videos need captions or descriptive transcripts).
[ ] If there are emojis, there are not more than three in a row.
[ ] Don't use flashing GIFs or videos.
[ ] If the content were to be read as plain text, it still makes sense, and no information is missing.

Any other comments?

viniciusdc commented 3 months ago

In the gif below, we can see the redirection happening. Peek 30-07-2024 17-50 Tested locally, using vagrant and Ubuntu2204 VMs