Skip to content
This repository has been archived by the owner on Nov 1, 2023. It is now read-only.

Added support for multi tenant authentication #746

Merged
4 commits merged into from
Apr 2, 2021

Conversation

gdhuper
Copy link
Contributor

@gdhuper gdhuper commented Mar 29, 2021

Summary of the Pull Request

What is this about?

PR Checklist

  • Applies to work item: Add support for multi-tenant authentication #562
  • CLA signed. If not, go over here and sign the CLI.
  • Tests added/passed
  • Requires documentation to be updated
  • I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Info on Pull Request

The end-to-end changes needed to have onefuzz deployed with multi-tenant authentication.

Validation Steps Performed

How does someone test & validate?

@gdhuper gdhuper changed the title Added support for multi tenant authentication WIP: Added support for multi tenant authentication Mar 29, 2021
@gdhuper gdhuper changed the title WIP: Added support for multi tenant authentication Added support for multi tenant authentication Mar 29, 2021
@bmc-msft bmc-msft requested review from bmc-msft and chkeita March 29, 2021 19:00
@gdhuper gdhuper marked this pull request as ready for review March 30, 2021 19:30
@bmc-msft
Copy link
Contributor

Running integration tests.

@gdhuper gdhuper force-pushed the gdhuper/multi_tenant_auth branch from e7b40fa to aefd021 Compare April 1, 2021 04:44
@gdhuper gdhuper requested review from chkeita and bmc-msft April 1, 2021 08:37
@bmc-msft
Copy link
Contributor

bmc-msft commented Apr 1, 2021

Code LGTM. Running integration tests.

@bmc-msft
Copy link
Contributor

bmc-msft commented Apr 2, 2021

Passed integration tests.

@ghost
Copy link

ghost commented Apr 2, 2021

Hello @bmc-msft!

Because this pull request has the auto-merge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

@ghost ghost merged commit 7e5cf78 into microsoft:main Apr 2, 2021
@ghost ghost locked as resolved and limited conversation to collaborators May 2, 2021
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants