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

Don't access keyring when not needed #54

Open
DamienCassou opened this issue Jul 31, 2018 · 6 comments
Open

Don't access keyring when not needed #54

DamienCassou opened this issue Jul 31, 2018 · 6 comments

Comments

@DamienCassou
Copy link
Contributor

When using ledger-autosync to read an existing ofx file, I don't expect ledger-autosync to access my keyring and ask me to unlock it. This annoys me because I don't use Gnome keyring and I don't want any application to use it and ask me to unlock it.

@egh
Copy link
Owner

egh commented Aug 1, 2018

Thanks for the report! I haven't been using gnome keyring with ledger-autosync so I haven't seen this. Did you need to configure it explicitly, or was it configured when you ran ofxclient?

@DamienCassou
Copy link
Contributor Author

Did you need to configure it explicitly

I haven't configured anything.

or was it configured when you ran ofxclient?

I don't use ofxclient. I get my ofx files through boobank.

@egh
Copy link
Owner

egh commented Aug 1, 2018

Great, I'll try to replicate this behavior locally.

(As an aside, I didn't know boobank could fetch OFX files. I'll have to check it out.)

@rcurtin
Copy link

rcurtin commented Sep 8, 2019

@DamienCassou @egh this seems to be a result of ofxclient. I opened an issue there: captin411/ofxclient#78

My workaround is to comment out the import keyring and keyring.get_password() lines from ofxclient/cli.py, and also set KEYRING_AVAILABLE to False always.

Hope this helps!

@egh
Copy link
Owner

egh commented Sep 8, 2019

Thank you, Ryan!

@jlesieutre
Copy link

jlesieutre commented Jan 31, 2024

For anybody still having this problem (it's still there in ofxclient), this still solves the problem as of 1/2024 except that the offending lines to comment out have moved to ofxclient/config.py

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants