[MGNLPUR-94] ®Stamp 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 ® 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 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. |
| 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, |
| 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). |