dotnet / dotnet/aspnetcore

Expose UserManager<T>.GetChangeEmailTokenPurpose()

Open
#12,609 7 comments 0 reactions 0 assignees View on GitHub
affected-very-few area-identity enhancement severity-nice-to-have
Dominant language
C#
Stars
38.4k
Forks
10.9k
Avg merge
2d 5h
Merged PRs (30d)
276

Description

`UserManager` In 2.1 exposed the "purpose" constants, eg. [ConfirmEmailTokenPurpose](https://github.com/aspnet/Identity/blob/release/2.1/src/Core/UserManager.cs#L39).

We relied on this in order to use `userManager.VerifyUserTokenAsync()`.

Now those constants are gone, and there is a [`GetChangeEmailTokenPurpose()`](https://github.com/aspnet/AspNetCore/blob/master/src/Identity/Extensions.Core/src/UserManager.cs#L2472) method in its place, which is `protected`.

Please make it `public`, otherwise the verify function is unusable.

(Subclassing just adds an unnecessary mess as this class already conceals a lot of complexity and should be usable as is.)

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.