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

RFC6238 AuthenticatorService #5811

Closed
slaneyrw opened this issue Jul 26, 2018 · 2 comments
Closed

RFC6238 AuthenticatorService #5811

slaneyrw opened this issue Jul 26, 2018 · 2 comments
Labels
affected-very-few This issue impacts very few customers area-identity Includes: Identity and providers enhancement This issue represents an ask for new feature or an enhancement to an existing one severity-minor This label is used by an internal tool triaged
Milestone

Comments

@slaneyrw
Copy link

Can you please expose the internal Rfc6238AuthenticatorService ( and Base32 ), mostly so we are able to utilize it for TwoFactor authentication using the Google Authenticator class of apps.

The existing TotpSecurityStampBaseTokenProvider cannot be used as the security stamp should not be involved in the code check.

The TimeStep and interval range also needs to be customizable per token provider as the default TOTP settings are not always appropriate - effectively this forces us to re-implement ALL of the token providers.

At the moment we have to import 3rd party libraries Base32 and OptSharp to be able to provide this functionality

The PIN length consideration from #1564 should also be customizable.

Lastly, please consider converting this to a instance class, not static, so we can configure the TOTP settings within individual token providers and set the TokenOptions provider instance ( I'd love to do this solely via DI but the MS DI engine cannot support parallel named DI graphs )

@blowdart
Copy link
Contributor

@GrabYourPitchforks maybe this should move to CLR rather than just ASP.NET? Thoughts.

@aspnet-hello aspnet-hello transferred this issue from aspnet/Identity Dec 18, 2018
@aspnet-hello aspnet-hello added this to the Backlog milestone Dec 18, 2018
@aspnet-hello aspnet-hello added area-identity Includes: Identity and providers enhancement This issue represents an ask for new feature or an enhancement to an existing one labels Dec 18, 2018
@HaoK HaoK added affected-very-few This issue impacts very few customers severity-minor This label is used by an internal tool labels Oct 9, 2020 — with ASP.NET Core Issue Ranking
@mkArtakMSFT
Copy link
Member

Hi. Thanks for contacting us.
We're closing this issue as there was not much community interest in this ask for quite a while now.
You can learn more about our triage process and how we handle issues by reading our Triage Process writeup.

@mkArtakMSFT mkArtakMSFT closed this as not planned Won't fix, can't repro, duplicate, stale Nov 21, 2023
@ghost ghost locked as resolved and limited conversation to collaborators Feb 7, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
affected-very-few This issue impacts very few customers area-identity Includes: Identity and providers enhancement This issue represents an ask for new feature or an enhancement to an existing one severity-minor This label is used by an internal tool triaged
Projects
None yet
Development

No branches or pull requests

5 participants