Task #13053 (new)
Opened 4 years ago
Last modified 3 years ago
Bug: Password Reset is not an Error
| Reported by: | khgillen | Owned by: | atarkowska |
|---|---|---|---|
| Priority: | minor | Milestone: | Permissions |
| Component: | Web | Version: | n.a. |
| Keywords: | logging, password | Cc: | ux@… |
| Resources: | n.a. | Referenced By: | n.a. |
| References: | n.a. | Remaining Time: | n.a. |
| Sprint: | n.a. |
Description
The password reset dialogue in OMERO.web 5.1.4 is currently triggering ERROR entries in the OMEROweb.log.
Case 1: The username does not exist in OMERO - the error message is "No such experimenter".
2015-10-06 10:51:05,515 ERROR [ omeroweb.webadmin.views] (proc.03838) forgotten_password:345 object #0 (::omero::cmd::ERR)
{
category = ::omero::cmd::ResetPasswordRequest
name = unknown-user
parameters =
{
key = ApiUsageException
value = No such experimenter: {userid}
}
}
The user should be informed, certainly, but this is not an ERROR state of OMERO - it's working as it should.
2: A user trying a password reset with an email address that doesn't match the entry in their OMERO user account, this is generating an ERROR entry in the OMEROweb.log - once again, this is a normal function of OMERO and should not be producing ERROR log entries.
ERROR: demo OMEROweb:
2015-10-06 11:10:19,308 ERROR [ omeroweb.webadmin.views] (proc.03839) forgotten_password:345 object #0 (::omero::cmd::ERR)
{
category = ::omero::cmd::ResetPasswordRequest
name = not-match
parameters =
{
key = ApiUsageException
value = Email address does not match.
}
}
Change History (6)
comment:1 Changed 4 years ago by jamoore
- Milestone changed from Unscheduled to OMERO-5.2.0
comment:2 Changed 4 years ago by jburel
- Milestone changed from OMERO-5.2.1 to OMERO-5.2.2
Milestone OMERO-5.2.1 deleted
comment:3 Changed 4 years ago by jburel
- Milestone changed from OMERO-5.2.2 to OMERO-5.2.1
Milestone OMERO-5.2.2 deleted
comment:4 Changed 3 years ago by jburel
- Milestone changed from OMERO-5.2.2 to Permissions
comment:5 Changed 3 years ago by jburel
comment:6 Changed 3 years ago by jburel
on the same topic see also https://trac.openmicroscopy.org/ome/ticket/13028
At least putting it into 5.2 for review.