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

FYI: under x2go experiences 1-2 min import time #420

Closed
yarikoptic opened this issue Jan 27, 2020 · 3 comments
Closed

FYI: under x2go experiences 1-2 min import time #420

yarikoptic opened this issue Jan 27, 2020 · 3 comments

Comments

@yarikoptic
Copy link

$> time python -c 'import keyring'
Gkr-Message: 22:13:50.844: secret service operation failed: Did not receive a reply. Possible causes include: the remote application did not send a reply, the message bus security policy blocked the reply, the reply timeout expired, or the network connection was broken.
python -c 'import keyring'  0.11s user 0.04s system 0% cpu 50.197 total
  • keyring installed via debian package 17.1.1-1
  • Relates to Improve import times #403 (on import times) -- IMHO keyring must not load any backend merely upon import. Should load it (ideally the one "best" for the system or specified, not all) only whenever its API started to be used and require connection.
  • decided to file an issue just to let you know and didn't want to "pollute" Improve import times #403, so feel free to close unless you want to convert it to some action (e.g. "add a reasonable timeout for allowing backend to load")
@takluyver
Copy link
Contributor

This is hopefully improved from version 21.5.0, following PR #474. Now, on import, keyring only checks whether the secrets & kwallet services are available, rather than trying to talk to them. If the code uses keyring, it may still see a delay when it does that, but fingers crossed that's better than a long wait on import.

@adfnekc
Copy link

adfnekc commented Dec 11, 2020

see more in pypa/pip#8485

@jaraco
Copy link
Owner

jaraco commented Dec 22, 2020

This should be fixed with the latest release due to #480.

@jaraco jaraco closed this as completed Dec 22, 2020
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