[jira] Created: (OFBIZ-1091) Login process not reset properly after failure and hitting CLR

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|

[jira] Created: (OFBIZ-1091) Login process not reset properly after failure and hitting CLR

Nicolas Malin (Jira)
Login process not reset properly after failure and hitting CLR
--------------------------------------------------------------

                 Key: OFBIZ-1091
                 URL: https://issues.apache.org/jira/browse/OFBIZ-1091
             Project: OFBiz
          Issue Type: Bug
          Components: pos
    Affects Versions: SVN trunk
            Reporter: Wickersheimer Jeremy


When the login fails (bad user or bad password), the POS display the error message.
Then if you type something then enter, nothing seems to happen.
Then if you type CLR, the POS ask for user Id, but internally act like the previous input was taken into account, thus getting directly to a login failure (or success) without asking password.

Example:

User Id:
# 5 <enter>
Password:
# 5 <enter>
User not found.
# 1 <enter>
User not found. (the display did not change, like nothin happen, and the input display still displays "5")
# <CLR>
User Id:
# 1 <enter>
---> login success

Alternatively replace the last input by something else than "1", you will get Password incorrect.

The internal state is not reset properly after hitting CLR.


--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply | Threaded
Open this post in threaded view
|

[jira] Commented: (OFBIZ-1091) Login process not reset properly after failure and hitting CLR

Nicolas Malin (Jira)

    [ https://issues.apache.org/jira/browse/OFBIZ-1091?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12506757 ]

Wickersheimer Jeremy commented on OFBIZ-1091:
---------------------------------------------

After some investigation, in case of login error in SecurityEvents.java, the error message is displayed and the input is cleared (input.clear()) which empty the function stack.

In that state, pressing the ENTER button cause MenuEvents.addItem(pos, event); to be called causing a NPE at
if (!trans.isOpen()) {
which is hidden by an InvocationTargetException catched in ButtonEventConfig (you get to call getCause().printStackTrace() on it to get the details)

At first i thought of disabling the buttons (except CLR) to prevent that but it would be too complicated, a much simpler solution is to disable the error message and append the "Enter user ID" message, and reseting the function stack to LOGIN.

Patch is joined.



> Login process not reset properly after failure and hitting CLR
> --------------------------------------------------------------
>
>                 Key: OFBIZ-1091
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1091
>             Project: OFBiz
>          Issue Type: Bug
>          Components: pos
>    Affects Versions: SVN trunk
>            Reporter: Wickersheimer Jeremy
>
> When the login fails (bad user or bad password), the POS display the error message.
> Then if you type something then enter, nothing seems to happen.
> Then if you type CLR, the POS ask for user Id, but internally act like the previous input was taken into account, thus getting directly to a login failure (or success) without asking password.
> Example:
> User Id:
> # 5 <enter>
> Password:
> # 5 <enter>
> User not found.
> # 1 <enter>
> User not found. (the display did not change, like nothin happen, and the input display still displays "5")
> # <CLR>
> User Id:
> # 1 <enter>
> ---> login success
> Alternatively replace the last input by something else than "1", you will get Password incorrect.
> The internal state is not reset properly after hitting CLR.

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply | Threaded
Open this post in threaded view
|

[jira] Updated: (OFBIZ-1091) Login process not reset properly after failure and hitting CLR

Nicolas Malin (Jira)
In reply to this post by Nicolas Malin (Jira)

     [ https://issues.apache.org/jira/browse/OFBIZ-1091?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Wickersheimer Jeremy updated OFBIZ-1091:
----------------------------------------

    Attachment: ofbiz-1091.patch

See previous comment.

> Login process not reset properly after failure and hitting CLR
> --------------------------------------------------------------
>
>                 Key: OFBIZ-1091
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1091
>             Project: OFBiz
>          Issue Type: Bug
>          Components: pos
>    Affects Versions: SVN trunk
>            Reporter: Wickersheimer Jeremy
>         Attachments: ofbiz-1091.patch
>
>
> When the login fails (bad user or bad password), the POS display the error message.
> Then if you type something then enter, nothing seems to happen.
> Then if you type CLR, the POS ask for user Id, but internally act like the previous input was taken into account, thus getting directly to a login failure (or success) without asking password.
> Example:
> User Id:
> # 5 <enter>
> Password:
> # 5 <enter>
> User not found.
> # 1 <enter>
> User not found. (the display did not change, like nothin happen, and the input display still displays "5")
> # <CLR>
> User Id:
> # 1 <enter>
> ---> login success
> Alternatively replace the last input by something else than "1", you will get Password incorrect.
> The internal state is not reset properly after hitting CLR.

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply | Threaded
Open this post in threaded view
|

[jira] Closed: (OFBIZ-1091) Login process not reset properly after failure and hitting CLR

Nicolas Malin (Jira)
In reply to this post by Nicolas Malin (Jira)

     [ https://issues.apache.org/jira/browse/OFBIZ-1091?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jacques Le Roux closed OFBIZ-1091.
----------------------------------

       Resolution: Fixed
    Fix Version/s: Release Branch 4.0
                   SVN trunk
         Assignee: Jacques Le Roux

Thanks Jeremy,

Your patch is in trunk rev. 549972  and release rev. 549973



> Login process not reset properly after failure and hitting CLR
> --------------------------------------------------------------
>
>                 Key: OFBIZ-1091
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1091
>             Project: OFBiz
>          Issue Type: Bug
>          Components: pos
>    Affects Versions: SVN trunk
>            Reporter: Wickersheimer Jeremy
>            Assignee: Jacques Le Roux
>             Fix For: SVN trunk, Release Branch 4.0
>
>         Attachments: ofbiz-1091.patch
>
>
> When the login fails (bad user or bad password), the POS display the error message.
> Then if you type something then enter, nothing seems to happen.
> Then if you type CLR, the POS ask for user Id, but internally act like the previous input was taken into account, thus getting directly to a login failure (or success) without asking password.
> Example:
> User Id:
> # 5 <enter>
> Password:
> # 5 <enter>
> User not found.
> # 1 <enter>
> User not found. (the display did not change, like nothin happen, and the input display still displays "5")
> # <CLR>
> User Id:
> # 1 <enter>
> ---> login success
> Alternatively replace the last input by something else than "1", you will get Password incorrect.
> The internal state is not reset properly after hitting CLR.

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.