Repository navigation
Restore Test-IsValidDn, Test-IsValidUpn and Get-ReportChain, completing migration of the former Staging/v3 candidates and bringing the supported module to 27 commands. - #13
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Replace the DN and UPN regexes with documented practical syntax policies and pipeline support.
Support escaped DN separators, multi-valued RDNs, long UPN suffixes and internationalized domains.
Preserve report-chain identity aliases, server selection and ordered property projection.
Escape LDAP assertion values, require exactly one manager and exclude that manager from report results.
Propagate AD failures without changing the caller’s error preference.
Keep ActiveDirectory optional at module import.
Update exports, documentation and regression tests.
The naming validators check the documented syntax policies, not account existence or complete AD/Entra provisioning rules.
Validation: 327 tests passed on Linux with PowerShell 7.4.13; five Windows-only registry tests were skipped. The patch applies cleanly after PR #12. AD queries are mocked; live domain integration and Windows/macOS CI verification remain pending.