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

Adjustment for new CIAM partition #564

Merged
merged 1 commit into from
May 24, 2023
Merged

Adjustment for new CIAM partition #564

merged 1 commit into from
May 24, 2023

Conversation

rayluo
Copy link
Collaborator

@rayluo rayluo commented May 20, 2023

Our lab switched to a new CIAM test tenant.

The two lab API were also updated:

  1. Get a test user and its test app from lab (Same API, different value in response)
  2. Get that app's authority and scope. Note: You shall NOT hardcode the test app's client id "ced7..." into your test case's source code or even configuration file. Instead, read it on-the-fly from the appId field from the response of the first API above. This way, in the future when/if the lab switch to a different client_id, your existing test automation will not break.

There is one change needed in the Confidential Client's test case. No more hardcoded scope. The scope returned from lab API 2 will work out of the box.

@rayluo rayluo force-pushed the ciam-in-new-partition branch 3 times, most recently from c3a4874 to 592c88c Compare May 20, 2023 19:58
@rayluo rayluo force-pushed the ciam-in-new-partition branch from 592c88c to 49090cb Compare May 24, 2023 06:56
@rayluo rayluo merged commit b96489d into dev May 24, 2023
@rayluo rayluo deleted the ciam-in-new-partition branch May 24, 2023 06:59
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.

1 participant