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

Integration with Password Managers #83

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

bxsx
Copy link
Contributor

@bxsx bxsx commented Aug 7, 2022

Summary

This PR integrates garmin-connect-export with password managers. Currently supported managers:

A newly added option --passmanager PASSWORD_MANAGER enables integration for the selected manager.

Pitch

Garmin Connect requires login and password for authentication. These credentials are passed to the app as a plain text via program arguments such as --username and --password. This can lead to potential leakage of credentials in the event of unsecured storage (like in plain text files).

This integration allows you to get the credentials directly from Password Manager and pass them to the application.

Requirements

  • Password Manager CLI executable must be in $PATH
  • garmin-connect-export :-)

Usage

Use --passmanager PASSWORD_MANAGER to look up garmin.com item for credentials.

Examples

python gcexport.py --passmanager BitWarden

Notes

--passmanager option cannot be mixed with --username or --password.

@bxsx bxsx mentioned this pull request Aug 7, 2022
@bxsx
Copy link
Contributor Author

bxsx commented Mar 16, 2024

@pe-st any plans to merge it? I can update the branch and resolve conflicts..

@pe-st
Copy link
Owner

pe-st commented Mar 24, 2024

Hello @bxsx, I will not be able to test your PR (not using these 2 password managers, and the ones I uses don't have CLI (yet)), but if it's helpful I'm sure other people will like it. So yes, go ahead :-)

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

Successfully merging this pull request may close these issues.

2 participants