allow case sensitive regex matches for the `MAGPIE_USER_NAME_EXTRA_REGEX` setting

Ouranosinc / Magpie

AuthN/AuthZ services

https://pavics-magpie.readthedocs.io

Apache License 2.0

1 stars 5 forks source link

allow case sensitive regex matches for the `MAGPIE_USER_NAME_EXTRA_REGEX` setting #594

Closed mishaschwartz closed 10 months ago

mishaschwartz commented 10 months ago

Ensures that the settings/environment variable MAGPIE_USER_NAME_EXTRA_REGEX is case sensitive.

Previously, the check was case insensitive meaning that it could not be used to restrict usernames based on case. For example, setting this value to ^[a-z]+$ would have permit the username "someuser" as well as "Someuser".

Now, the same regular expression will not match "Someuser" since case sensitivity in enforced.

mishaschwartz commented 10 months ago

@fmigneault The tests are failing because data.ceda.ac.uk is unresponsive right now: https://github.com/Ouranosinc/Magpie/actions/runs/6655721960/job/18098444930?pr=594

fmigneault commented 10 months ago

Validated that CEDA has planned down server until November 7. Validated that tests and other checks than links run successfully.