PowerShell / PowerShell/PSScriptAnalyzer
AvoidUsingPlainTextForPassword docs do not cover all checked parameter names
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- C#
- Sterne
- 2.2k
- Forks
- 414
- Ø Merge
- 13 Std. 1 Min.
- Gemergte PRs (30 T.)
- 2
Beschreibung
I have a script parameter that holds a path to the encrypted credential file named $CredentialFile for which I get the warning:
Parameter '$CredentialFile' should use SecureString, otherwise this will expose sensitive information. See ConvertTo-SecureString for more information.
There is no reason for me use SecureString here and I cannot think of a better alternative to the parameter name, so it would be nice to suppress this message for this particular instance, but not to the whole file or project. Besides, the documentation for PSAvoidUsingPlainTextForPassword needs to include Credential and whatever other strings it gets triggered of, since it only mentions one word: Password.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginnen Sie mit der Regel PSAvoidUsingPlainTextForPassword und ihrem Dokumentationseintrag; im Issue ist kein Datei- oder Testpfad angegeben. Prüfen Sie, wie Parameternamen wie CredentialFile abgedeckt sind, und aktualisieren Sie anschließend die Dokumentation, um alle zutreffenden Namen sowie die unterstützte Möglichkeit zum Unterdrücken einer einzelnen Warnung zu beschreiben, sofern das dokumentierte Verhalten dies zulässt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- powershell
- Bereich
- documentation, tooling
- Issue-Typ
- Dokumentation
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100