564948 - Resetting password email needs branding update and valid URL (for custom url)

Status: RESOLVED FIXED

(Cloud Services :: Web Site - Deprecated, defect)

Description

[Tony Chung [:tchung]]

The current resetting weave password needs to be rebranded to Firefox Sync.  It also needs a valid link to be clicked within the email for custom URLs.  It currently shows: 

please click this link:  WEAVE_PASSWORD_BASE_URL?username=qastage2&key=2RQJ-6M99-NGM3-7CDF

and that means nothing to the user.

Email notification attached with custom url.

Repro:
1) install weave 1.3b5
2) login and reset password
3) verify the password email still references weave
4) repeat steps 1-2, but with a custom url and valid account (eg. staging server)
5) Verify the password email doesnt have a valid link, see above

Expected:
- custom url has a valid url.  Also, all references to Weave must change to Firefox Sync

Comment 1

[Tony Chung [:tchung]]

Attachment: reseting password email

Comment 2

[Zandr Milewski [:zandr]]

The baseurl has been reconfigured. Branding still needs to change.

Comment 3

[Tony Chung [:tchung]]

Verified fix to the baseurl.  awaiting branding changes.

Comment 4

[Tony Chung [:tchung]]

Attachment: reseting password email 2 attachment

This is still open on sync 1.3 on phx production.   i've attached a second email link from the reset password email, and clicking the link takes me to the weave branded page:

https://auth.services.mozilla.com/weave-password-reset?username=weavephone&key=ETWI-1PCG-U718-JDG4 (weave branded)

and resetting password takes me to: https://auth.services.mozilla.com/weave-password-reset (weave branded)

Please fix all weave branding to firefox sync.

Comment 5

[Zandr Milewski [:zandr]]

This is... interesting.

I think this just means that I should replace the auth.services.m.c baseurl with services.m.c

Right? mconnor?

Comment 6

[Mike Connor [:mconnor]]

Need some more bits on services, actually, to handle actually changing the password, we just have the link to do the reset request.

Comment 7

[Mike Connor [:mconnor]]

We now use all correct branding for the web bits now, the only remaining piece is "Weave" in the email itself.  Filed Bug 575905 to cover fixing the server to not hardcode this.