OIDC : extra params

[olevitt]

See InseeFrLab/onyxia#739

[johnksv]

Should this field be added to Region.OIDCConfiguration as well? Such that it can be returned in ConfigurationController L52.
Also, regarding the name. Do you think extraParams is OK, or should it be emphasised that it is extraQueryParams (or is that a frontend detail)?

[garronej]

@johnksv @olevitt I also would prefer oidc.extra-query-params or oidc.extra-url-redirect-query-params.
About Region.OIDCConfiguration I think it would be overkill. Most of the time all client are on the same realm so there's SSO, it's the Onyxia oidc client that will initiate the redirect.

[olevitt]

I think we can do the same as for other OIDC configuration : make this parameter optionnal for additional OIDCConfiguration so that it's only needed once but can still be overriden in case it's needed

[garronej]

@olevitt ok works for me I guess I just need to update the doc

[sonarcloud]

Quality Gate passed

Issues
0 New issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud