Skip to content
This repository has been archived by the owner on May 18, 2021. It is now read-only.

Support webauthn MFA factor #158

Closed
sharpone74 opened this issue Jun 6, 2019 · 14 comments
Closed

Support webauthn MFA factor #158

sharpone74 opened this issue Jun 6, 2019 · 14 comments

Comments

@sharpone74
Copy link

Looks like webauthn (yubikey 5c nano in this case) isn't supported. It would be great if it was.

DEBU[0022] Step: 2
INFO[0022] Requesting MFA. Please complete two-factor authentication with your second device
DEBU[0022] 00uMrQE7-7jbCWJZz61wCH5d4PKf7V8YVp2IgMb0S4
DEBU[0022] Failed to validate credentials: factor webauthn not supported
Failed to validate credentials```
@nickatsegment
Copy link
Contributor

I don't have access to a webauthn device here, so don't look at me :D Happy to accept a PR.

@sharpone74
Copy link
Author

How about if I buy you one? ;)

@nickatsegment
Copy link
Contributor

How about if I buy you one? ;)

Haha, yeah, I can't really do that; I answer only to the crystalline entity that runs Segment.

@smiller171
Copy link
Contributor

@stale
Copy link

stale bot commented Aug 24, 2019

This issue has been automatically marked stale because it has not had any activity in the last 60 days. If no further activity occurs within 7 days, it will be closed. Closed does not mean "never", just that it has no momentum to get accomplished any time soon.
See CONTRIBUTING.md for more info.

@stale stale bot added the stale label Aug 24, 2019
@nickatsegment
Copy link
Contributor

Well I actually bought a Yubikey 5c myself. I believe this would give us TouchID support as well, which would be super cool.

That said, there are no plans to implement this as yet. Happy to accept a PR though, and I can even test it now

@stale stale bot removed the stale label Aug 28, 2019
@smiller171
Copy link
Contributor

I believe this would give us TouchID support as well

Right now I'm pretty sure only Chrome can use TouchID for WebAuthn, so you'd need to open a browser for the auth request.

@stale
Copy link

stale bot commented Oct 28, 2019

This issue has been automatically marked stale because it has not had any activity in the last 60 days. If no further activity occurs within 7 days, it will be closed. Closed does not mean "never", just that it has no momentum to get accomplished any time soon.
See CONTRIBUTING.md for more info.

@stale stale bot added the stale label Oct 28, 2019
@smiller171
Copy link
Contributor

Even if it's not being tackled right now, I think it makes sense to keep this issue open

@bennumo
Copy link

bennumo commented Nov 6, 2019

Yeah, I'm happy to buy you one as well! We're looking to shift from YubiKey + Seedfile to the U2F / WebAuthN implementation in Okta. For us, this would help us to manage the growing list of YubiKeys we have. It's getting difficult to manage seed files at scale.

@nickatsegment
Copy link
Contributor

@bennumo fair enough, but like I said, no plans to implement this by maintainers at present. PRs encouraged (after #236 is lifted)

@stale
Copy link

stale bot commented Jan 5, 2020

This issue has been automatically marked stale because it has not had any activity in the last 60 days. If no further activity occurs within 7 days, it will be closed. Closed does not mean "never", just that it has no momentum to get accomplished any time soon.
See CONTRIBUTING.md for more info.

@stale stale bot added the stale label Jan 5, 2020
@smiller171
Copy link
Contributor

I really don't like this definition of closed personally. I'm more in favor of a label indicating that an issue is low-priority but that PRs are welcome

@stale stale bot removed the stale label Jan 6, 2020
@nickatsegment
Copy link
Contributor

#278

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants