dnewsholme / PasswordState-Management

Powershell Module For Management of Passwordstate. (Powershell Core Compatible)
GNU General Public License v3.0
39 stars 14 forks source link

Update PasswordResult Class for compatibility with PasswordState v9 #135

Closed eizedev closed 3 years ago

eizedev commented 3 years ago

Hi @dnewsholme

In PasswordState v9 the passwords api method will return two new fields (GenericFieldInfo and OTP) that are currently not referenced in the PasswordResult class. So the retrieval of Passwords after updating to v9 are not possible.

I have included these to properties to the class (Type Array), now it is working again. This should not break compatibility with older version of passwordstate.

Reference OTP + GenericFieldInfo: #132

eizedev commented 3 years ago

@dnewsholme

Your appveyor pipelines files because the pester module will be updated to v5: https://ci.appveyor.com/project/dnewsholme/passwordstate-management/builds/38111737

I think you should install Pester with the latest v4 version (Install-Module -Name Pester -RequiredVersion 4.10.1) in the pipeline until the code ist changed to v5.

dnewsholme commented 3 years ago

@eizedev i've updated master with a new appveyor.yml which will install pester 4.10.1 if you pull that down your build should work again.

eizedev commented 3 years ago

@dnewsholme Thanks! Build succeeded :)

dnewsholme commented 3 years ago

Thanks for the contribution again @eizedev :)