Skip to content

Adding TRR for Azure Forged SAML#26

Open
vanvleeta wants to merge 7 commits into
tired-labs:mainfrom
vanvleeta:trr_entra_forge_saml
Open

Adding TRR for Azure Forged SAML#26
vanvleeta wants to merge 7 commits into
tired-labs:mainfrom
vanvleeta:trr_entra_forge_saml

Conversation

@vanvleeta
Copy link
Copy Markdown
Contributor

No description provided.

@vanvleeta vanvleeta requested a review from a team May 13, 2026 22:21
Copy link
Copy Markdown
Contributor

@mjwhitta mjwhitta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bit distracted by family stuff, but didn't want to forget. I'll finish reviewing later.

Comment thread reports/trr0000/azr/README.md Outdated
Copy link
Copy Markdown

@ineffectivecoder ineffectivecoder left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Other than that parenthesis(Today I learned/remembered singular of parentheses) it looks good to me.

dumpst3rfir3
dumpst3rfir3 previously approved these changes May 22, 2026
Copy link
Copy Markdown
Contributor

@dumpst3rfir3 dumpst3rfir3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great to me. Great work, as always. I had a few minor, nit-picky suggestions (and a couple of spelling corrections, I think).

Comment thread reports/trr0000/azr/README.md Outdated
Comment thread reports/trr0000/azr/README.md
- none: validation applied to no domains (the state prior to the Dec 2025
update)

Currently this can only be done via the Graph API:
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What privileges are needed to do this? I assume high level (Global admin, etc.)?

Comment thread reports/trr0000/azr/README.md Outdated
Comment thread reports/trr0000/azr/README.md Outdated
Comment thread reports/trr0000/azr/README.md Outdated
Comment thread reports/trr0000/azr/README.md Outdated
Comment thread reports/trr0000/azr/README.md Outdated
Comment thread reports/trr0000/azr/README.md Outdated
Co-authored-by: Alex Garcia <149016698+dumpst3rfir3@users.noreply.github.com>
Co-authored-by: vanvleeta <vanvleet@gmail.com>
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.

4 participants