Skip to content

[pull] dev from KelvinTegelaar:dev#15

Merged
pull[bot] merged 11 commits intoTecharyJames:devfrom
KelvinTegelaar:dev
Apr 5, 2026
Merged

[pull] dev from KelvinTegelaar:dev#15
pull[bot] merged 11 commits intoTecharyJames:devfrom
KelvinTegelaar:dev

Conversation

@pull
Copy link
Copy Markdown

@pull pull bot commented Apr 5, 2026

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

kris6673 and others added 11 commits April 3, 2026 20:23
…undant put thing so we fix not doing many graph call things
- Updated log messages to clarify when settings are already configured correctly for various standards.
- Enhanced error handling and logging for remediation processes across multiple standards.
# Summary

Replaces all `-user $request.headers.'x-ms-client-principal'` calls with
`-Headers $Request.Headers` in `Invoke-ExecUpdateDriftDeviation` for
consistent logging behavior.

# Description

`Write-LogMessage` handles IDP detection, AAD API client resolution,
and IP address capture from `x-forwarded-for` when given the full
`-Headers` object. Passing `-user` with the raw base64 principal skips
all of that. All 9 `Write-LogMessage` calls in the file now use
`-Headers $Request.Headers` to match the convention used elsewhere.
…ng it (#1984)

Enhance logging messages to clarify the status of settings and improve
error handling during remediation processes across various standards.
This should remove some unnecessary graph calls
@pull pull bot locked and limited conversation to collaborators Apr 5, 2026
@pull pull bot added the ⤵️ pull label Apr 5, 2026
@pull pull bot merged commit 77684ea into TecharyJames:dev Apr 5, 2026
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.

3 participants