Skip to content
This repository has been archived by the owner on Dec 14, 2017. It is now read-only.

Adding a sample custom user service with FIDO U2F #313

Open
wants to merge 8 commits into
base: dev
Choose a base branch
from
Open

Adding a sample custom user service with FIDO U2F #313

wants to merge 8 commits into from

Conversation

ReeceWilliams7
Copy link

I've recently starting looking into FIDO U2F and how this might be used as an additional option for 2FA within our (my company's) existing implementation of IdentityServer3.

I came across the following thread on the issue tracker: IdentityServer/IdentityServer3#2121 and thought it would be a good idea to create a new sample that shows how this can achieved so that others can get an idea (as well as helping me learn about U2F at the same time!).

The project was initiated by creating a copy of the existing CustomUserService, so generally speaking it follows a similar structure to that.

For the actual U2F implementation, I've used the OSS library https://github.com/brucedog/u2flib, including some of the view/controller functionality provided within the demo site there.

It only works on Google Chrome at this time as that's the only browser that supports FIDO U2F currently, and I've tested it with the Yubikey Neo - unfortunately that's the only device I have at my disposal.

I've provided a readme file with some of the above detail in also.

Hope you find this useful!

@dnfclas
Copy link

dnfclas commented Oct 9, 2016

Hi @ReeceWilliams7, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla2.dotnetfoundation.org.

TTYL, DNFBOT;

@dnfclas
Copy link

dnfclas commented Oct 9, 2016

@ReeceWilliams7, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.

Thanks, DNFBOT;

@ghost ghost removed the cla-signed label Dec 7, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants