Issue 13297: xmlrpc.client could accept bytes for input and output (original) (raw)

Created on 2011-10-30 23:41 by flox, last changed 2022-04-11 14:57 by admin. This issue is now closed.

Files
File name Uploaded Description Edit
issue13297_xmlrpc_bytes.diff flox,2011-10-30 23:44 review
issue13297_xmlrpc_bytes_v2.diff flox,2011-11-09 23:40 review
issue13297_xmlrpc_bytes_v3.diff flox,2011-11-10 18:17 review
issue13297_xmlrpc_bytes_v4.diff flox,2011-11-12 15:59 review
Messages (8)
msg146666 - (view) Author: Florent Xicluna (flox) * (Python committer) Date: 2011-10-30 23:41
Since Python 3 makes clear the distinction between bytes and string, there is no more reason to use the xmlrpc.client.Binary wrapper for binary objects. The xmlrpc library may deal with bytes and bytearray normally. To support backward compatibility, the ServerProxy will have a keyword argument "use_bytes" similar to the "use_datetime" from issue #1120353.
msg147382 - (view) Author: Florent Xicluna (flox) * (Python committer) Date: 2011-11-09 23:39
Updated with documentation.
msg147383 - (view) Author: Florent Xicluna (flox) * (Python committer) Date: 2011-11-09 23:40
Ooops wrong patch.
msg147399 - (view) Author: Florent Xicluna (flox) * (Python committer) Date: 2011-11-10 08:31
Maybe a flag "use_builtin_types" is preferred? It should implies both `bytes` and `datetime.datetime` are used for presenting data received. I don't see a use case where we would like to have `bytes` for binary data but keep `xmlrpc.client.DateTime` for date objects. Becomes obsolete, if this change is done: - xmlrpc.client.Binary - xmlrpc.client.DateTime - use_datetime flag of `ServerProxy` class and `loads` function I'm not sure it's necessary to deprecate things here.
msg147406 - (view) Author: Florent Xicluna (flox) * (Python committer) Date: 2011-11-10 18:17
This patch v3 implements the proposal of use_builtin_types flag. In this case the use_datetime flag becomes obsolete. Please comment.
msg147473 - (view) Author: Éric Araujo (eric.araujo) * (Python committer) Date: 2011-11-12 11:18
I made comments on Rietveld but there was a glitch, I’m not sure the email was sent.
msg147507 - (view) Author: Florent Xicluna (flox) * (Python committer) Date: 2011-11-12 15:59
Thank you for your comments. Uploaded a new version.
msg147701 - (view) Author: Roundup Robot (python-dev) (Python triager) Date: 2011-11-15 19:54
New changeset 0175883d9513 by Florent Xicluna in branch 'default': Closes #13297: use bytes type to send and receive binary data through XMLRPC. http://hg.python.org/cpython/rev/0175883d9513
History
Date User Action Args
2022-04-11 14:57:23 admin set github: 57506
2011-11-15 19:54:17 python-dev set status: open -> closednosy: + python-devmessages: + resolution: fixedstage: patch review -> resolved
2011-11-12 15:59:50 flox set files: + issue13297_xmlrpc_bytes_v4.diffmessages: +
2011-11-12 11🔞28 eric.araujo set nosy: + eric.araujomessages: +
2011-11-10 18:17:09 flox set files: + issue13297_xmlrpc_bytes_v3.diffmessages: +
2011-11-10 08:49:50 flox link issue2979 dependencies
2011-11-10 08:31:38 flox set messages: +
2011-11-09 23:40:47 flox set files: + issue13297_xmlrpc_bytes_v2.diffmessages: +
2011-11-09 23:40:15 flox set files: - issue13291_xmlrpc_v2.diff
2011-11-09 23:39:30 flox set files: + issue13291_xmlrpc_v2.diffnosy: + fdrake, effbot, skip.montanaromessages: +
2011-10-30 23:44:36 flox set files: + issue13297_xmlrpc_bytes.diffkeywords: + patchstage: patch review
2011-10-30 23:41:39 flox create