Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
DRIVERS-2672 Add OIDC machine workflow spec. #1471
DRIVERS-2672 Add OIDC machine workflow spec. #1471
Changes from 39 commits
fbc3d72
f01c678
62130cf
d09ae6c
67c9794
b7f750c
a333353
aa1ada7
93d9b27
46afc8d
2c0e07a
becadd5
5b0665e
88de5b9
e9972de
17fd8a5
cf5455e
b349eb2
1fb61b6
be7f515
48f9fd4
f841d13
9523a84
98eec93
f8c3b4b
1ec045c
82a20d4
9ff52c0
4e1a5f5
7f79ecd
697bed6
cd1470d
9ef2650
bb09469
f2135be
9b6bd5f
11357db
357cad9
09554a6
37102c5
03a0f8b
95395e2
f41bb37
6b07e16
353fa83
4203a6d
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
Large diffs are not rendered by default.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
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.
PROVIDER_NAME and callbacks are mutual exclusive. Callback parameter probably has to be removed from this test.
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.
You're correct. The
callback
legacy test case parameter was never documented and is not supported by most drivers, so I intended to remove it from all test cases here, but accidentally left that one in. I've updated that test case to assert that providing a password causes a validation error, which matches the spec.