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.

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

DM-41998: Add support for per-user ingresses #911

Merged
merged 3 commits into from
Dec 4, 2023
Merged

DM-41998: Add support for per-user ingresses #911

merged 3 commits into from
Dec 4, 2023

Conversation

rra
Copy link
Member

@rra rra commented Dec 4, 2023

Allow ingresses to be restricted to a specific named user by setting the user attribute in the config section of a GafaelfawrIngress, or the corresponding user query parameter to the /auth route. Any other user will receive a 403 error. The scope requirements must also still be met.

Allow ingresses to be restricted to a specific named user by
setting the user attribute in the config section of a
GafaelfawrIngress, or the corresponding user query parameter to
the /auth route. Any other user will receive a 403 error. The
scope requirements must also still be met.
Document that the button to delete a token now has an ARIA label
since previously it was only an icon.
Ruff now supports formatting, so we can switch to it from Black and
use fewer tools. We still use Black for docs, since Ruff has no
equivalent to blacken-docs currently.

Start checking all the files under docs/* except for docs/conf.py,
since there's no reason not to format the Python files for
architectural diagrams.

Remove pre-commit stuff from requirements/dev.in since they're
installed via make init directly. Upgrade pip before other operations
in make init.
@rra rra merged commit 9aa91d7 into main Dec 4, 2023
5 checks passed
@rra rra deleted the tickets/DM-41998 branch December 4, 2023 23:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant