Warning: Can't synchronize with repository "(default)" (/home/git/ome.git does not appear to be a Git repository.). Look in the Trac log for more information.
Notice: In order to edit this ticket you need to be either: a Product Owner, The owner or the reporter of the ticket, or, in case of a Task not yet assigned, a team_member"

Task #13053 (new)

Opened 9 years ago

Last modified 8 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 9 years ago by jamoore

  • Milestone changed from Unscheduled to OMERO-5.2.0

At least putting it into 5.2 for review.

comment:2 Changed 8 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 8 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 8 years ago by jburel

  • Milestone changed from OMERO-5.2.2 to Permissions

comment:6 Changed 8 years ago by jburel

Note: See TracTickets for help on using tickets. You may also have a look at Agilo extensions to the ticket.

1.3.13-PRO © 2008-2011 Agilo Software all rights reserved (this page was served in: 0.81461 sec.)

We're Hiring!