-
Notifications
You must be signed in to change notification settings - Fork 30
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
Extend browser user authentication provide to handle Condition Access #361
base: integration
Are you sure you want to change the base?
Extend browser user authentication provide to handle Condition Access #361
Conversation
{ | ||
if (await page.IsVisibleAsync($"[data-test-id=\"{personaEmail}\"]")) | ||
{ | ||
logger.LogInformation($"Selecting {personaEmail}"); |
Check warning
Code scanning / CodeQL
Exposure of private information Medium test
access to local variable personaEmail
Private data returned by
access to local variable personaEmail
Private data returned by
access to local variable personaEmail
Private data returned by
access to local variable personaEmail
Minimum allowed line rate is |
{ | ||
// Default to msedge browser | ||
launchOptions.Channel = "msedge"; | ||
} |
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.
can we reset the channel to input value from the user if it is specifically requested in the yaml
{ | ||
var complete = false; | ||
|
||
var persona = testState.GetTestSuiteDefinition().Persona; |
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.
null checks for the definition, persona and email
} | ||
} | ||
|
||
if (!complete) |
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.
would it make sense to add a pause here to wait for user to attempt login
Pull Request Template
Description
Provide additional steps in the browser based authentication to handle conditional access using provider.
Checklist