Bug 1099278

Summary: Changing email address produces invalid email
Product: [Retired] Zanata Reporter: Sean Flanigan <sflaniga>
Component: Component-LogicAssignee: Sean Flanigan <sflaniga>
Status: CLOSED CURRENTRELEASE QA Contact: Zanata-QA Mailling List <zanata-qa>
Severity: high Docs Contact:
Priority: unspecified    
Version: 3.4CC: djansen, zanata-bugs
Target Milestone: ---   
Target Release: 3.5   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: 3.4.0-SNAPSHOT (git-server-3.3.2-353-g76254f5) Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2015-01-26 23:27:45 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Bug Depends On:    
Bug Blocks: 1088122    

Description Sean Flanigan 2014-05-20 02:14:41 UTC
Description of problem:
  The email generated when changing email address is sent to the old email, and the activation link has no activation key (causing a 404).

Version-Release number of selected component (if applicable):
  3.4.0-SNAPSHOT (git-server-3.3.2-351-gdebfc4e) 

How reproducible:
Unknown

Steps to Reproduce:
1. Log in (I used Kerberos)
2. Visit Account Settings from the Dashboard
3. Submit a new email address
4. Log out

Actual results:
  Old email address receives an email with invalid activation link: https://zanata.example.com/account/validate_email/ 

Expected results:
  New email address receives an email with a working activation link, ie https://zanata.example.com/account/validate_email/${activationKey}

Additional info:

The 404 is because the key is missing.  If you add in a dummy key at the end of the URL, you get this message: "Invalid activation key But you can help us fix it!"

Regarding email activation, see also https://bugzilla.redhat.com/show_bug.cgi?id=1098863

Comment 1 Sean Flanigan 2014-05-20 03:53:45 UTC
https://github.com/zanata/zanata-server/pull/453

Comment 2 Damian Jansen 2014-05-20 04:56:34 UTC
Verified