Skip to content

Include statusText in login exception message #52

@yeeliu01

Description

@yeeliu01

Logins may fail for different reasons. For example, one case:

[Fri Apr 3 19:23:28 2020]: (ComponentName) Pyrfa: (Severity) Error: [LoginHandler::processResponse] Login Status : dataState="Suspect" streamState="Closed" statusCode="Unknown RespCode" statusText="Access Denied-already using position 170.50.166.85"

[Fri Apr 3 19:23:28 2020]: (ComponentName) Pyrfa: (Severity) Error: 102 ([Pyrfa::login] Login failed. Please check data permission. (username: yee.liu))

Or another different case:

[Sat Mar 28 01:49:22 2020]: (ComponentName) Pyrfa: (Severity) Error: [LoginHandler::processResponse] Login Status : dataState="Suspect" streamState="Closed" statusCode="Unknown RespCode" statusText="yee123, unknown to system."

[Sat Mar 28 01:49:22 2020]: (ComponentName) Pyrfa: (Severity) Error: 102 ([Pyrfa::login] Login failed. Please check data permission. (username: yee123))

It would be more informative if we can include the "statusText" as above, in the login exception thrown by pyrfa.cpp#L421.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions