We removed the assumption of a leading slash from the paths.
Unfortunately, we missed a couple of places the leading slash should have been removed when this change was made.
Set up a verify email with a destination path starting with a leading slash. Initiate a login on the verify form. Click the email link to complete. "The website encountered an unexpected error" will be displayed.
The VerifyEmailRedirectTest class sets up a Verify Email entity for testing. Unfortunately, it assumes a leading slash will be added, so doesn't contain one.
In the Verifier class, the verifyAndLogin() function concatenates the specified redirect path to internal:/. Unfortunately, if the destination path starts with a "/", this is an invalid redirect.
path and redirect_path, when setting up the the VerifyEmail entityVerifier::verifyAndLogin(), there are two places where 'internal:/' is used. This should be changed to 'internal:'None.
None.
None.
Active
1.0
Code
Hii,
I have done the changes as per your requirements. Thanks!
I have tested locally, and the tests are all passing.
Moving to RTBC.
I've merged this change.
Great to get this fixed. Thanks for your effort!