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

Add optional -usr command line parameter. #6

Closed
wants to merge 2 commits into from

Conversation

giuscri
Copy link

@giuscri giuscri commented Jun 4, 2017

Useful in cases where you have multiple accounts for the same service but you don't want to use the same password for each. This does not break compatibility with older versions of gokey, since when -usr is not specified passwords are generated as before.

Giuseppe added 2 commits June 4, 2017 12:03
Useful in cases where you have multiple accounts
for the same service but you don't want to use
the same password for each. This does not break
compatibility with older versions of gokey,
since when `-usr` is not specified passwords
are generated as before.
@ignatk
Copy link
Contributor

ignatk commented Jun 6, 2017

I'm not entirely sure about it. While the case is definitely useful, this opens a path to a bunch of similar cases: what if we want to differentiate on OS groups as well, or service path, or hostname or whatever etc...

The bottomline is that tool has to be as generic as possible. That's why we just provide realm, but it is up to the caller to identify the "composition" of the realm.

@giuscri
Copy link
Author

giuscri commented Jun 11, 2017

You're right. I'm going to close this pull request then. (:

@giuscri giuscri closed this Jun 11, 2017
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.

None yet

2 participants