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

ESC3 Flowchart categorization #132

Open
techBrandon opened this issue Jun 11, 2024 · 4 comments
Open

ESC3 Flowchart categorization #132

techBrandon opened this issue Jun 11, 2024 · 4 comments
Labels
documentation Improvements or additions to documentation question Further information is requested

Comments

@techBrandon
Copy link
Contributor

The flowchart for ESC3 (https://github.com/TrimarcJake/Locksmith/blob/main/Docs/Flowcharts/ESC3.md) categorizes a group with a large "group size" as Medium but a lists a User as High. This is the only flowchart that does this. All other's categorize the large groups as a high. Thank you good sirs. You are a delight.

@SamErde
Copy link
Collaborator

SamErde commented Jun 11, 2024

Will have to review. If my memory serves me correctly, this was to align with the practice of using groups instead of individual users in ACLs. Thanks for the thoughtful review and feedback!

@techBrandon
Copy link
Contributor Author

techBrandon commented Jun 11, 2024 via email

@SamErde SamErde added documentation Improvements or additions to documentation question Further information is requested labels Jun 11, 2024
@TrimarcJake
Copy link
Owner

@techBrandon

I think easily identifiable "intentional" groups would fall under the BuiltIn/PKIAdmins group branch of the flowchart. But I see what you're saying. We should chat about this more during SMEeting.

...Also it's very likely a copy/paste error.

@TrimarcJake
Copy link
Owner

Related to #117

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants