Issue 36033: logging.makeLogRecord should update "rv" using a dict defined with bytes instead of strings (original) (raw)

Created on 2019-02-19 10:46 by ralonsoh, last changed 2022-04-11 14:59 by admin. This issue is now closed.

Messages (4)
msg335925 - (view) Author: Rodolfo Alonso (ralonsoh) Date: 2019-02-19 10:46
When using Pycharm to execute unit/functional tests (OpenStack development), oslo_privsep.daemon._ClientChannel.out_of_band retrieves the record from the input arguments. Pycharm arguments are stored in a dictionary using bytes instead of strings. When logging.makeLogRecord tries to update "rv", the value in "dict" parameter (using bytes) doesn't update "rv" values.
msg335989 - (view) Author: Karthikeyan Singaravelan (xtreak) * (Python committer) Date: 2019-02-19 17:41
Sorry, I don't get the report. Can you please add a short reproducer script with a description of the current behavior and the expected behavior?
msg335999 - (view) Author: Vinay Sajip (vinay.sajip) * (Python committer) Date: 2019-02-19 19:04
If the arguments are retrieved from a byte source, why can't they be converted to strings before passing to logging? This may be an issue for OpenStack. It's not logging's job to convert bytes passed to APIs that expect strings.
msg346055 - (view) Author: Vinay Sajip (vinay.sajip) * (Python committer) Date: 2019-06-19 14:53
Closing, as no further feedback from issue reporter. Feel free to reopen if you have a good response to my last comment.
History
Date User Action Args
2022-04-11 14:59:11 admin set github: 80214
2019-06-19 14:53:30 vinay.sajip set status: open -> closedresolution: not a bugmessages: + stage: resolved
2019-02-19 19:04:09 vinay.sajip set messages: +
2019-02-19 17:41:38 xtreak set nosy: + xtreakmessages: +
2019-02-19 16:01:29 SilentGhost set nosy: + vinay.sajipcomponents: + Library (Lib)
2019-02-19 10:46:00 ralonsoh create