search

cornflourblue / aspnet-core-3-signup-verification-api

ASP.NET Core 3.1 - Boilerplate API with Email Sign Up, Verification, Authentication & Forgot Password
https://jasonwatmore.com/post/2020/07/06/aspnet-core-3-boilerplate-api-with-email-sign-up-verification-authentication-forgot-password
MIT License
226 stars 93 forks source link

Redundant Dollar Sign In Email Body. #3

Closed twinmind closed 3 years ago

twinmind commented 3 years ago

There is no need for dollar sign inside the string next to variable opening curly brace on these lines: https://github.com/cornflourblue/aspnet-core-3-signup-verification-api/blob/c8d1607fb70a0f73611080be7dbab412b227deca/Services/AccountService.cs#L374 https://github.com/cornflourblue/aspnet-core-3-signup-verification-api/blob/c8d1607fb70a0f73611080be7dbab412b227deca/Services/AccountService.cs#L384 It currently simply adds a dollar sign to a string content, and in case of line 384 it breaks reset password link.

cornflourblue commented 3 years ago

fix merged