[MGNLPUR-94] &regStamp in verification link is replaced by ®Stamp in hotmail Created: 14/May/13  Updated: 29/Apr/14  Resolved: 04/Apr/14

Status: Closed
Project: Magnolia Public User Registration
Component/s: registration
Affects Version/s: 1.4.2, 2.0
Fix Version/s: 1.4.6, 2.3

Type: Bug Priority: Neutral
Reporter: Marc Champion Assignee: Roman Kovařík
Resolution: Fixed Votes: 0
Labels: maintenance, support
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Template:
Acceptance criteria:
Empty
Task DoD:
[ ]* Doc/release notes changes? Comment present?
[ ]* Downstream builds green?
[ ]* Solution information and context easily available?
[ ]* Tests
[ ]* FixVersion filled and not yet released
[ ]  Architecture Decision Record (ADR)
Bug DoR:
[ ]* Steps to reproduce, expected, and actual results filled
[ ]* Affected version filled
Date of First Response:

 Description   

The &reg in the verification link sent by email is misunderstood at least by hotmail to be the ® html entity. This results in unusable verification links when sent to hotmail accounts.

Original verification link

http://www.test.ch/.magnolia/pages/validation?userId=aac0107c-94e8-426c-a0d9-2c8b9fb77eea&regStamp=1368456964532

As it's changed by hotmail:

http://www.test.ch/.magnolia/pages/validation?userId=aac0107c-94e8-426c-a0d9-2c8b9fb77eea®Stamp=1368456964532

I'm currently using an easy workaround:

if (userId.indexOf("=") > -1) {
	log.info("UserId contains = sign, splitting parameters: {}", userId);
	String splitString = userId;
	userId = splitString.substring(0, 36);
	regStamp = splitString.substring(splitString.indexOf("=") + 1, splitString.length());
}


 Comments   
Comment by Jan Haderka [ 15/May/13 ]

This is annoying, but it's really a bug in hotmail - they are overzealous when parsing entities. The full signature for entity is ® & reg ; while they ignore the terminal semicolon character.

http://www.w3schools.com/html/html_entities.asp

Comment by Marc Champion [ 15/May/13 ]

Dear Jan, thanks for your answer! Sure this is an issue with hotmail but wouldn't it be a good idea to change the parameter name to something like "stamp" to workaround this issue? We now have hundreds of complaints from hotmail users who cannot activate their accounts.

Comment by Jan Haderka [ 15/May/13 ]

Hi Mark,

sure, I was not downplaying the issue. We will for sure do something about it. The note there was just an additional explanation of the problem.

Jan

Comment by Marc Champion [ 15/May/13 ]

Thank you, Jan!

Comment by Tom Geissendoerfer [ 25/Mar/14 ]

Hi,
i send & amp ;regStamp and it's correctly iterpreted.

Comment by Roman Kovařík [ 04/Apr/14 ]

Commited on 1.4.x branch for review (port to master will be part of integration phase).

Generated at Mon Feb 12 06:43:01 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.