search

devcartel / pyrfa

Open sourced Python API for Refinitiv (Thomson Reuters) Enterprise Platform.
http://devcartel.com/pyrfa
MIT License
50 stars 15 forks source link

Include statusText in login exception message #52

Open yeeliu01 opened 4 years ago

yeeliu01 commented 4 years ago

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.