Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 388922

Summary:	User profile email address should be validated before updating
Product:	[ECD] Orion	Reporter:	Maciej Bendkowski <maciej.bendkowski>
Component:	Client	Assignee:	Malgorzata Janczarska <malgorzata.tomczyk>
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	P3	CC:	antonm, malgorzata.tomczyk, simon_kaegi, Szymon.Brandys
Version:	unspecified
Target Milestone:	2.0 RC1
Hardware:	PC
OS:	Windows 7
Whiteboard:

Description Maciej Bendkowski

2012-09-06 06:52:56 EDT

It's possible to provide an incorrect email address in the user profile and update successfully. I suppose the server log (or the SMTP server log) will contain some errors about that event. Minor bug, but may likely occur since typos happen. Client-side form validation should handle this problem.

Comment 1 John Arthorne

2012-09-06 15:00:39 EDT

I thought it sent an actual validation email to that address before accepting the change.

Comment 2 Malgorzata Janczarska

2012-11-07 09:33:15 EST

(In reply to comment #1)
> I thought it sent an actual validation email to that address before accepting
> the change.
It does. Under the hood it stores the flag "email confirmed" that is changed after the email address is verified. Until "email confirmed" server will not send any emails to that email address. Email confirmed flag used to be shown on UI, but when new user profile was created it was not shown there. I suppose one of the solutions would be to add "unconfirmed" label next to the user email until email is confirmed. Another would be not to show email in the profile as long as it's not confirmed, but this might be confusing for the user, he might be inputing the email over and over again before he discovers he has tons of confirmation emails in his mailbox.

Comment 3 Malgorzata Janczarska

2013-02-07 07:37:37 EST

I added Email Confirmed flag to the UI and enabled displaying messages about sending email (and others). Hopefully this gives user a full knowledge about the situation.