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

multiple auth schemes #1157

Merged
merged 10 commits into from
Apr 30, 2021
Merged

multiple auth schemes #1157

merged 10 commits into from
Apr 30, 2021

Conversation

jennyf19
Copy link
Collaborator

No description provided.

@jennyf19 jennyf19 requested a review from jmprieur April 23, 2021 03:32
@jennyf19 jennyf19 changed the base branch from master to authSchemes April 29, 2021 16:18
@jennyf19 jennyf19 marked this pull request as ready for review April 29, 2021 16:33
@jennyf19 jennyf19 merged commit 105179d into authSchemes Apr 30, 2021
jennyf19 added a commit that referenced this pull request May 19, 2021
* multiple auth schemes (#1157)

* initial commit for auth schemes

* update tests part 1

* add basic web app for testing

* updates to dev app

* more updates to dev app and downstreamapi

* update from merge conflict

* update app integration tests

* few more updates

* merge options partial idea (#1162)

* merge options partial idea

* fix tests

* fix warnings

* fix validate options

* fix credscan (#1172)

* fix policheck

* more updates to mergedoptions (#1177)

* another possiblity for merged options

* fix issue w/post configure

* fix tests

* fix a few more tests

* fix warnings

* fix integration tests

* update merged options

* fix warnings

* fix push

* add scheme to graph call, clean up unused method, add randomness to d… (#1180)

* add scheme to graph call, clean up unused method, add randomness to dev ap

* fix warnings

* PR feedback & fix sign-out

* add auth scheme integration tests (#1189)

* add auth scheme integration tests

* few more updates

* few more updates

* Adding the merged options generator

* Adding generated methods for merged options

* Removing code that isn't necessary

* Taking into account the JwtBearer options
(actually the authority) into the Microsoft
identity options if needed.

* Adding the Merged options generator to
the solution

* few more updates

Co-authored-by: Jean-Marc Prieur <jmprieur@microsoft.com>

* fix merge conflict

* clean-up, pr feedback

Co-authored-by: Jean-Marc Prieur <jmprieur@microsoft.com>
@jennyf19 jennyf19 deleted the jennyf/authScheme branch May 31, 2021 20:58
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