Skip to content

Update powershell code to use APIs that take SecureString when they become available in .NET Core #3228

Closed

Description

PowerShell Core currently uses some APIs that only take plain password text, because the overloads that take SecureString is not available in .NET Core. But once they become available, we need to update our code to switch to the latter.

Instances: (not a complete list)
Constructor of NetworkCredential at Credential.cs
Constructor of X509Certificate2 in CertificateCommands.cs
ProcessStartInfo.PasswordInClearText at [Process.cs]

Feel free to update this list when you spot one.

Activity

  1. added this to the milestone on Feb 28, 2017
  2. SteveL-MSFT commented on Feb 28, 2017

    @SteveL-MSFT
    Member

    Related #3198
    #1654

    CoreFx PR exposing SecureString dotnet/corefx#13362

  3. mirichmo commented on Mar 2, 2017

    @mirichmo
    Member

    Should this be part of #3235?

  4. daxian-dbw commented on Mar 2, 2017

    @daxian-dbw
    MemberAuthor

    This one is related to security. I thought it probably would have higher priority and thus kept them separate.

  5. daxian-dbw commented on Apr 17, 2017

    @daxian-dbw
    MemberAuthor

    Fixed via #3556

  6. iSazonov commented on Apr 18, 2017

    @iSazonov
    Collaborator

    Joey Aiello (@joeyaiello) Please clarify - should we remove "Waiting-NetStandard20" if an Issue was fixed?
    Now (after moving to .Net Core 2.0 in #3556) could we reuse the Label to mark Issues where we are waiting external bug fixes in CoreFX and CoreCLR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

Resolution-FixedThe issue is fixed.WG-Enginecore PowerShell engine, interpreter, and runtimeWaiting - DotNetCorewaiting on a fix/change in .NET

Type

No type

Projects

No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions