search

nazarewk / keyring_pass

Password Store (pass) backend for python's keyring
MIT License
19 stars 5 forks source link

Invalid directory created when key-prefix set to '' --- works with . (dot) #4

Closed matthewsht closed 3 years ago

matthewsht commented 3 years ago

Got a chance to test new version v0.7.0 of keyring_pass.

The config file (~/.config/python_keyring/keyringrc.cfg) contains [pass] key-prefix=.

With key-prefix=. the password ends up at local/monkey -- which is correct.

According to the documentation, key-prefix='' should also have worked but did not - I got a directory named ''/local/monkey at the top of tree which was not intended:

pass Password Store ├── '' │ └── local │ └── monkey │ └── foo OTHER ENTRIES ELIDED

nazarewk commented 3 years ago

ini/cfg files do not have a concept of quoted string, everything after the equal sign is a string, you can try writing simply key-prefix= without anything else in the line.

matthewsht commented 3 years ago

Darn - that was it. Blank works fine - I was being too literal.

Sorry for the issue report.

-- Hunter Matthews Scientific System Engineer [C] NIH/National Human Genome Research Institute @.**@.> | (919) 491-2124

On Sep 1, 2021, at 3:04 AM, Krzysztof Nazarewski @.**@.>> wrote:

ini/cfg files do not have a concept of string, everything after the equal sign is a string, you can try writing simply key-prefix= without anything else in the line.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/nazarewk/keyring_pass/issues/4#issuecomment-909983441, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ATFEQQZCNSBONPPVE25MPNLT7XF6XANCNFSM5DFJQVRA. Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

matthewsht commented 3 years ago

Mistakenly reported NOTABUG - closing.

Thank you for quick reply.