-
Notifications
You must be signed in to change notification settings - Fork 12
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
ID-1285 switch open api to only oauth2 security scheme #1448
Conversation
@@ -11,6 +11,7 @@ start() { | |||
docker create --name "${CONTAINER}" \ | |||
--restart "always" \ | |||
-p 40080:80 -p 50443:443 \ | |||
-e "B2C_APPLICATION_ID=${OIDC_CLIENT_ID}" \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixes local deployment to do the audience check correctly
Quality Gate passedIssues Measures |
scopes: {} | ||
tokenUrl: OAUTH_TOKEN_URL |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
replacements for OAUTH_AUTHORIZATION_URL
and OAUTH_TOKEN_URL
were implemented in workbenchLibs
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks for doing this
Ticket: https://broadworkbench.atlassian.net/browse/ID-1285
<Don't forget to include the ticket number in the PR title!>
What:
<For your reviewers' sake, please describe in a sentence or two what this PR is accomplishing (usually from the users' perspective, but not necessarily).>
Why:
<For your reviewers' sake, please describe in ~1 paragraph what the value of this PR is to our users or to ourselves.>
How:
<For your reviewers' sake, please describe in ~1 paragraph how this PR accomplishes its goal.>
<If the PR is big, please indicate where a reviewer should start reading it (i.e. which file or function).>
PR checklist