Skip to content

fix: Add CredScan suppression for DeviceManagement.Administration v1.0 example#3630

Merged
ramsessanchez merged 1 commit into
mainfrom
fix/credscan-suppression-admin-password-example
Jun 2, 2026
Merged

fix: Add CredScan suppression for DeviceManagement.Administration v1.0 example#3630
ramsessanchez merged 1 commit into
mainfrom
fix/credscan-suppression-admin-password-example

Conversation

@ramsessanchez
Copy link
Copy Markdown
Contributor

Summary

Adds a CredScan suppression entry for the v1.0 example file:

  • \src/DeviceManagement.Administration/v1.0/examples/Update-MgDeviceManagementVirtualEndpointOnPremiseConnectionAdDomainPassword.md\

This file contains a placeholder password (\AdDomainPassword_XXXX) in its code snippet, which is not a real credential. The suppression follows the existing pattern used for similar example files (e.g., the beta counterpart under DeviceManagement.Actions).

Changes

  • Updated .azure-pipelines/config/credscan/credscan-suppressions.json\ to suppress the false positive.

…0 example

The Update-MgDeviceManagementVirtualEndpointOnPremiseConnectionAdDomainPassword.md
example file contains a placeholder password that triggers CredScan. Added it to
the suppressions list to match the existing pattern for similar example files.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@ramsessanchez ramsessanchez requested a review from a team as a code owner June 1, 2026 22:13
@ramsessanchez ramsessanchez merged commit f7cf43b into main Jun 2, 2026
8 checks passed
@ramsessanchez ramsessanchez deleted the fix/credscan-suppression-admin-password-example branch June 2, 2026 07:56
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.

2 participants