The Wayback Machine - https://web.archive.org/web/20220207164153/https://github.com/dotnet/aspnetcore/pull/40040
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

Jump to bottom

Add missing returns in EnableAuthenticator #40040

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Add missing returns in EnableAuthenticator #40040

wants to merge 2 commits into from
+2 −2

Conversation

@danirzv
Copy link

@danirzv danirzv commented Feb 7, 2022

Add missing returns in EnableAuthenticator

  • You've read the Contributor Guide and Code of Conduct.
  • You've included unit or integration tests for your change, where applicable.
  • You've included inline docs for your change, where applicable.
  • There's an open issue for the PR that you are making. If you'd like to propose a new feature or change, please open an issue to discuss the change or find an existing issue.

Description

Fixes #39909
@danirzv danirzv requested a review from Pilchie as a code owner Feb 7, 2022
@dnfadmin
Copy link

@dnfadmin dnfadmin commented Feb 7, 2022 •
edited

CLA assistant check
All CLA requirements met.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Pilchie Pilchie

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
area-identity community-contribution
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

EnableAuthenticatorModel shows page even if user was not found
2 participants
@danirzv @dnfadmin