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
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions