Skip to content

Expanded comment about normalizing email to lowercase #368

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.

Sign up for GitHub

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

Merged
ProgrammerAL merged 1 commit into from
Jul 1, 2026
Merged

Expanded comment about normalizing email to lowercase #368

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 9 additions & 3 deletions ...ityServer/v8/UserManagement/PasswordRegistration/PasswordRegistration/StringExtensions.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,14 @@ namespace PasswordRegistration;

internal static class StringExtensions
{
/// <summary>
/// Takes an email string and puts it into the way it's stored in the database
/// </summary>
/// Duende User Management respects the email RFC, where the part before the @ is case-sensitive
/// and the domain itself is not.
///
/// aaa@aaa.com and AaA@aaa.com are different email addresses.
///
/// In this sample, we are normalizing the email address so that the sample app treats
/// these 2 example emails as the same.
///
/// While it differs from the RFC, it's what users expect.
public static string NormalizeEmail(this string email) => email.ToLowerInvariant();
}
Loading