Skip to content
This repository was archived by the owner on Oct 10, 2026. It is now read-only.

Document synthetic credential test exception - #21

Merged
cmendesvdl merged 1 commit into
mainfrom
security/securestring-analyzer-review
Oct 10, 2026
Merged

cmendesvdl merged 1 commit into
mainfrom
security/securestring-analyzer-review

Conversation

@cmendesvdl

Copy link
Copy Markdown
Collaborator

Summary

Adds a narrowly scoped SuppressMessageAttribute justification for PSAvoidUsingConvertToSecureStringWithPlainText in tests/Invoke-CMApplyDriverPackage.Tests.ps1. The value is a fixed synthetic test-only credential.

No runtime behavior changes. Mirrors the fix already merged in vartaxe/DriverAutomationTool#20.

Validation

  • File parses cleanly.
  • PSScriptAnalyzer: 0 findings for the targeted rule.
  • Fork regression harness passed.

Add a narrowly scoped analyzer justification for the fixed test-only PSCredential value.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@cmendesvdl
cmendesvdl merged commit 0f34556 into main Oct 10, 2026
1 check passed
@cmendesvdl
cmendesvdl deleted the security/securestring-analyzer-review branch October 10, 2026 08:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants