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

Add support for oidc extra params from api #739

Merged
merged 1 commit into from
Feb 8, 2024

Conversation

johnksv
Copy link
Contributor

@johnksv johnksv commented Feb 5, 2024

In some use cases the user want to specify extra query parameters. E.g. prompt type. This can now be done by specifying the parameters in the API. The frontend will use those parameters and send it along in the oidc flow.

Draft PR for visibility until code is ready for review.

@garronej
Copy link
Contributor

garronej commented Feb 7, 2024

Hey Jhon!

Thank you for the PR!

This is probably a parameter that we want to add at the region level alongside with the other oidc parameters.
I'll synchronise with @fcomte and @olevitt and we'll do it soon!

In some use cases the user want to specify extra query paramas.
E.g. prompt type. This can now be done by specifying a json object in the env variable

Get oidc extra query params field from api
Copy link

sonarcloud bot commented Feb 7, 2024

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

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

See analysis details on SonarCloud

@johnksv
Copy link
Contributor Author

johnksv commented Feb 7, 2024

Rebased to make changes align with upcoming changes in API (InseeFrLab/onyxia-api#378).

@johnksv johnksv changed the title Add support for specifying oidc extra query params Add support for oidc extra params from api Feb 7, 2024
@garronej
Copy link
Contributor

garronej commented Feb 7, 2024

Looking good! there's just a few litle details to adress. I'll fix them and merge tomorrow!
Thanks a lot for your contribution

@garronej garronej marked this pull request as ready for review February 8, 2024 11:49
@garronej garronej merged commit f79f041 into InseeFrLab:main Feb 8, 2024
6 checks passed
garronej added a commit that referenced this pull request Feb 9, 2024
@johnksv johnksv deleted the oidc-extra-params branch February 12, 2024 07:08
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

2 participants