Ticket #2401 (closed bug: obsolete)
s2s-web-bugs
| Reported by: | mira | Owned by: | mira |
|---|---|---|---|
| Priority: | major | Milestone: | |
| Component: | uncategorized | Version: | 2.0 |
| Keywords: | Cc: | mira, dido | |
| Category: | unknown | Effort: | |
| Importance: | Ticket_group: | ||
| Estimated Number of Hours: | 0 | Add Hours to Ticket: | 0 |
| Billable?: | yes | Total Hours: | 0 |
| Analysis_owners: | mira | Design_owners: | mira |
| Imp._owners: | mira | Test_owners: | |
| Analysis_reviewers: | meddle | Changelog: | Changelog |
| Design_reviewers: | meddle | Imp._reviewers: | meddle, todor, deyan |
| Test_reviewers: | Analysis_score: | 3 | |
| Design_score: | 4 | Imp._score: | 4 |
| Test_score: | 0 |
Description (last modified by mira) (diff)
There are several bugs in the web functionality of the server.
- The registration of a username that contains some special symbols like '/' throws an exception.
- The forgotten password functionality throws an exception if the server is not correctly configured.
- The password should not be changed if the email could not be send by some reason.
- The email should be validated by a regex - ".+@.+
.[a-z]+" - If there is no specified screen name, the user should be referred to by his/her user-name (not email).
- Some message arguments are not shown like in 'The mail {0} is invalid'.
Attachments
Change History
comment:2 Changed 15 years ago by mira
- Design_owners set to mira
- Status changed from s1b_analysis_finished to s2a_design_started
comment:3 Changed 15 years ago by mira
- Status changed from s2a_design_started to s2b_design_finished
- Description modified (diff)
There should be a validation of the provided user-name in registration. It would be good idea for them to contain only save for location names symbols since they are going to be used for ResourceRefR4. The validation should be using a common method from the RequestHandlerUtil.
The getSession() method should check if the smtp host, mail or password are null and provide the user with a message. The change of the pasword should be done last after the message is send. This way if for some reason the message is not send the password will not be changed.
The email should be validated by a pattern matcher ".+@.+
.[a-z]+".
Change the getTitle method of the UserH to use the user-name of the user if there is no screen name set. This could be extracted since it is the last part of the resource ref to the user.
There are two messages that require arguments - for invalid user and invalid email. They should be added with additional argument.
comment:5 Changed 15 years ago by mira
- Owner set to mira
- Status changed from s2c_design_ok to s3a_implementation_started
comment:7 Changed 15 years ago by mira
- Status changed from s3a_implementation_started to s3b_implementation_finished
The implementation is done according to the design. There is patch file attached to this ticket.
comment:8 Changed 15 years ago by meddle
- Status changed from s3b_implementation_finished to s3c_implementation_ok
- Analysis_reviewers set to meddle
- Analysis_score changed from 0 to 3
- Cc mira, dido added
- Imp._reviewers set to meddle, todor, deyan
- Design_score changed from 0 to 4
- Design_reviewers set to meddle
- Imp._score changed from 0 to 4
Merged into the trunk at [9011]
- Fixed in the messages user-name to be username
- @dido The 'Register' button is no wrong place (see the registration page).
- Some malformed JavaDoc
4p
