Bug 681621 - Default email address provided for rhqadmin user fails email validation
Summary: Default email address provided for rhqadmin user fails email validation
Status: CLOSED CURRENTRELEASE
Alias: None
Product: RHQ Project
Classification: Other
Component: Core UI
Version: 4.0.0.B02
Hardware: Unspecified
OS: Unspecified
medium
unspecified vote
Target Milestone: ---
: ---
Assignee: Ian Springer
QA Contact: Corey Welton
URL:
Whiteboard:
Keywords:
Depends On:
Blocks: gwt-admin-usersroles jon3
TreeView+ depends on / blocked
 
Reported: 2011-03-02 18:49 UTC by Corey Welton
Modified: 2013-08-06 00:38 UTC (History)
2 users (show)

(edit)
Clone Of:
(edit)
Last Closed:


Attachments (Terms of Use)

Description Corey Welton 2011-03-02 18:49:34 UTC
Description of problem:
The email address we provide for the rhqadmin user is 'rhqadmin@localhost'. This does not parse properly per the email address validator.  Thus, when trying to edit any properties (i.e., password) for that this user, one is unable to save the changes until something new is put in

Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.  Login as rhqadmin or another user with admin rights.
2.  Administration > Users > rhqadmin
3.  Attempt to edit various fields.
  
Actual results:
As soon as page loads, the email address is flagged as invalid.  User is then unable to save any changes until that field is change.

Expected results:
We should probably make the default email address for the rhqadmin user something that can be at least correctly parsed as a valid email address, i.e.,  'rhqadmin@example.com'

Additional info:
I don't know how often customers actually change this email address or leave it as default.  However, they most likely probably change the rhqadmin user password right off the bat.  Thus, this is reasonably high visibility.

Comment 1 Ian Springer 2011-03-03 17:08:05 UTC
Fixed by making the validation regexp accept email addresses with non-qualified hostnames, such as root@localhost - [master 4c250b2].

Comment 2 Corey Welton 2011-03-09 17:24:48 UTC
Verified.

Comment 3 Corey Welton 2011-05-24 01:16:43 UTC
Bookkeeping - closing bug - fixed in recent release.

Comment 4 Corey Welton 2011-05-24 01:16:43 UTC
Bookkeeping - closing bug - fixed in recent release.

Comment 5 Corey Welton 2011-05-24 01:16:44 UTC
Bookkeeping - closing bug - fixed in recent release.

Comment 6 Corey Welton 2011-05-24 01:16:52 UTC
Bookkeeping - closing bug - fixed in recent release.


Note You need to log in before you can comment on or make changes to this bug.