Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove the -AsPlainText switch in Get-KeePassEntry and provide a function to convert a secure string into plain text from the pipeline #88

Closed
Ninjigen opened this issue Sep 30, 2016 · 2 comments
Labels

Comments

@Ninjigen
Copy link
Collaborator

It's more of a suggestion than a question, but how about instead of having a Get-KeePassEntry function having a switch to provide the password as plain text, we should provide the with a consistent PSObject having a secure string for a password, and a function which would convert the secure string as plain text ?

The syntax would be something like

$KeePassEntry.Password | ConvertFrom-KeepassPassword

plain password

@jkdba
Copy link
Member

jkdba commented Oct 3, 2016

@Ninjigen, let me think about this some. I agree that this part needs to be redone.

@Twanfox
Copy link

Twanfox commented May 25, 2017

Dragging up an old-ish question, since it bothered me and I just recently started using the module.

As a quick-fix for my own copy of 2.0.4.0, I went and modified this line so I can keep the data secure in memory at the very least. The parts in bold are my addition.

$KeePassPsObject | Add-Member -Name 'Password' -MemberType NoteProperty -Value ($_keepassItem.Strings.ReadSafe('Password') | ConvertTo-SecureString -AsPlainText -Force -ErrorAction SilentlyContinue)

@jkdba jkdba mentioned this issue Jun 25, 2018
@jkdba jkdba closed this as completed Jun 25, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants