-
Notifications
You must be signed in to change notification settings - Fork 16
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
Credential lookup failure crashes s3 crawler #46
Comments
[ I know nothing about the keyring package and very little about DataLad's credential handling. I'd guess @yarikoptic will be able to give a more informed response. ]
It seems like more work would be needed to get keyring working in that environment: https://keyring.readthedocs.io/en/latest/index.html#using-keyring-on-headless-linux-systems. Perhaps a workaround would be to specify the credentials through an environment variable? Also, could you provide output from |
thanks for asking and sorry for the pains -- we should improve all this keyring handling and documentation. we indeed use keyring module with the default to choose the "most appropriate credential store" for the system. E.g. if it is a linux with Gnome3 , you could find "Passwords and Keys" application and see your keystore in there, probably locked. Upon initial invocation keyring should initiate its unlock, you unlock, and subsequent call should work out... On a remote server, if no X forwarding was setup for your session, it should have used some alternative secure storage backend, it should have worked :-/ What I did, awhile back, to avoid messing with autoselection etc on a remote server is to follow keyring docs to establish simple file based keyring (see e.g. https://pypi.org/project/keyring/#config-file-path). Here is a walkthrough:
$> python -c "import keyring.util.platform_; print(keyring.util.platform_.config_root())"
/home/yoh/.local/share/python_keyring
$> cat /home/yoh/.local/share/python_keyring/keyringrc.cfg
[backend]
default-keyring=keyrings.alt.file.PlaintextKeyring and then when you try, it should generate file like |
Trying this:
Leads to a crash:
Full traceback
This is running on a server that has no desktop environment installed.
The text was updated successfully, but these errors were encountered: