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 Rabobank #3503

Merged
merged 1 commit into from
Nov 5, 2018
Merged

Add Rabobank #3503

merged 1 commit into from
Nov 5, 2018

Conversation

Carlgo11
Copy link
Member

Closes #3370

@Carlgo11 Carlgo11 added the add site Issue/PR adds a site to the repo. label Oct 17, 2018
@brycx
Copy link
Contributor

brycx commented Nov 3, 2018

I see no mention of 2FA in the docs? If it is the thing called Rabo Scanner, do we need an exception for this? Could be some proprietary solution.

@Carlgo11
Copy link
Member Author

Carlgo11 commented Nov 3, 2018

@brycx the 2FA method is a smartcard reader often used by banks (at least over here).
The reader queries the bankcard chip, the chip responds with a signed copy of the query and that in turn is used to generate a OTP for the account holder to authenticate with.

I don't think a exception is necessary as every user will receive a reader and card upon registering with the bank.

@brycx
Copy link
Contributor

brycx commented Nov 3, 2018

That makes sense.

@stephengroat stephengroat merged commit 5f53c22 into 2factorauth:master Nov 5, 2018
@Carlgo11 Carlgo11 deleted the add-rabobank branch February 6, 2019 03:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
add site Issue/PR adds a site to the repo.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants