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 support for OTP URIs #53

Open
robinkrahl opened this issue Jan 6, 2019 · 6 comments
Open

Add support for OTP URIs #53

robinkrahl opened this issue Jan 6, 2019 · 6 comments
Labels
extension Functionality that is related to extensions, in one form or another help wanted

Comments

@robinkrahl
Copy link
Collaborator

Google Authenticator introduced the otpauth URI scheme to share OTP data (secret, OTP algorithm, HOTP counter, TOTP time window, issuer, …). A minimal example:

otpauth:https://totp/Example:[email protected]?secret=JBSWY3DPEHPK3PXP

A complete example:

otpauth:https://totp/ACME%20Co:[email protected]?secret=HXDMVJECJJWSRB3HWIZR4IFUGFTMXBOZ&issuer=ACME%20Co&algorithm=SHA1&digits=6&period=30

We could implement this as:

I prefer option 3. The downside is that the user always has to set a name and cannot reuse the label or issuer from the URL, but as we have a very short character limit for the name, that’s probably the better solution anyway. Also, we might need a dependency for URL parsing, but I think that’s acceptable.

@d-e-s-o
Copy link
Owner

d-e-s-o commented Jan 6, 2019

Thanks for bringing this up, Robin. Does support for this feature have priority for you? While I am not opposed to options two or three but, if possible, my suggestion would be to postpone the decision until we have a better idea of the extension mechanism and can evaluate possible implementation choices in light of it.

@robinkrahl
Copy link
Collaborator Author

Yeah, we can wait with implementing this.

@d-e-s-o d-e-s-o added the extension Functionality that is related to extensions, in one form or another label Jun 8, 2019
@robinkrahl
Copy link
Collaborator Author

As we now have extension support, we should have another look at this issue. If we go for an extension I suggest this syntax: add-otp-url URL [--slot SLOT] [--name NAME].

@d-e-s-o
Copy link
Owner

d-e-s-o commented Apr 17, 2021

Yeah, I think given the way extension look, this should fit in nicely. And conceptually it should be rare enough a feature that not including it in the main application makes more sense. The only suggestion I have is naming it otp-add-url (or otp-url-add). I think that may improve discoverability because things in the OTP realm are easier recognizable when they all start with otp.

@d-e-s-o
Copy link
Owner

d-e-s-o commented Apr 30, 2022

Do you still have a use for this, @robinkrahl and would be interested to prototype? The truth is, I am not eager to implement stuff that I don't use (but open to including it), and I have never encountered such URLs in the wild (not saying they don't exist, just that I am not frequenting services that use them); so I don't see myself implementing such support.

@robinkrahl
Copy link
Collaborator Author

While raw OTP URIs are indeed rarely used, OTP QR codes are quite common and also use this URI format internally. (I think both GitHub and Gitlab use them, for instance.) So yes, I’m still interested in this feature.

There is an old prototype, nitrocli-otp-qr, that 1) takes a screenshot with imagemagick, 2) parses the QR code with zbarimg, 3) parses the URL and 4) calls nitrocli, but transforming that into a PR is not a top priority for me at the moment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
extension Functionality that is related to extensions, in one form or another help wanted
Projects
None yet
Development

No branches or pull requests

2 participants