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.

Sign up for GitHub

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

Jump to bottom

Created Global Adminstrator Control #51

Merged
merged 4 commits into from
Dec 18, 2023
Merged

Created Global Adminstrator Control #51

merged 4 commits into from
Dec 18, 2023

Conversation

alalvi00
Copy link
Contributor

@alalvi00 alalvi00 commented Dec 18, 2023 •
edited by singhgss
Loading

Overview/Summary

Added a control in our Azure CaC guardrail solution that checks if there are minimum 2 global admin accounts and all of them have MFA enabled

This PR fixes/adds/changes/removes

  1. Adds a control in our Azure CaC guardrail solution that checks if there are minimum 2 global admin accounts and all of them have MFA enabled

Breaking Changes

N/A

Testing Evidence

Please provide any testing evidence to show that your Pull Request works/fixes as described and planned (include screenshots, if appropriate).

As part of this Pull Request I have

  • Checked for duplicate Pull Requests
  • Associated it with relevant GitHub Issues
  • Ensured my code/branch is up-to-date with the latest changes in the main branch
  • Performed testing and provided evidence.
  • Updated relevant and associated documentation.
  • Ensure PowerShell module versions have been updated (manually or with the ./tools/Update-ModuleVersions.ps1 script)

@alalvi00 alalvi00 linked an issue Dec 18, 2023 that may be closed by this pull request
MFA Evaluation sans Break Glass Accounts #3
Closed
@dutt0
Copy link
Contributor

dutt0 commented Dec 18, 2023

Updated the pull request merging the main branch

@dutt0
Copy link
Contributor

dutt0 commented Dec 18, 2023

Updated module versions and zips

@dutt0 dutt0 merged commit 39291de into main Dec 18, 2023
3 checks passed
@dutt0 dutt0 deleted the mfa-ga-accnts-issue3-aalvi branch December 18, 2023 18:39
@alalvi00 alalvi00 restored the mfa-ga-accnts-issue3-aalvi branch January 11, 2024 14:42
@alalvi00 alalvi00 deleted the mfa-ga-accnts-issue3-aalvi branch January 11, 2024 14:46
dutt0 added a commit that referenced this pull request Feb 1, 2024
@alalvi00 @dutt0
Created Global Adminstrator Control (#51)
3b28475 
* Created GA control

* updated and matched module version

* updated modules

---------

Co-authored-by: Dutta <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dutt0 dutt0 dutt0 approved these changes

@singhgss singhgss singhgss approved these changes

@MathesonSho MathesonSho Awaiting requested review from MathesonSho

@amrinderssc amrinderssc Awaiting requested review from amrinderssc

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

MFA Evaluation sans Break Glass Accounts
3 participants
@alalvi00 @dutt0 @singhgss