René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Sometimes sendClassical changes a msg from one thing into another thing","articleBody":"Say Alice calls sendClassical with msg = b'01' (= two bytes, first character 0 = ASCII 48, second character 1 = ASCII 49).\r\n\r\nWhen Bob calls recvClassical it receives a different msg = b'\\x01' (= one byte, value 1).\r\n\r\nThis is because of the following piece of code:\r\n\r\n    try:\r\n        to_send = bytes([int(msg)])\r\n    except (TypeError, ValueError):\r\n        to_send = bytes(msg)\r\n\r\nYou can see that:\r\n\r\n    \u003e\u003e\u003e msg = b'01'\r\n    \u003e\u003e\u003e bytes([int(msg)])\r\n    b'\\x01'\r\n\r\nThe better approach would be something along the lines of:\r\n\r\n    if isinstance(msg, bytes):\r\n        # Already provided as bytes, nothing to do\r\n    else:\r\n        # Provided as something else than bytes, convert to bytes\r\n\r\nAnd even that is not ideal, because there are still corner cases where the receiver gets something different than the sender sent.\r\n\r\nA better option could be to use pickle to serialize messages on-the-wire.\r\n\r\nI know it is \"not safe\" to unpickle messages received on-the-wire from an untrusted source, but it's probably good enough for simulations.\r\n\r\nThe pickling and unpickling could be done inside sendClassical and recvClassical, or the burden could be put on the application and sendClassical and recvClassical could insist on only getting messages of type bytes.\r\n\r\nThe former is more convenient for the application, the latter gives the application their own choice of serialization method (including the option of using something more safe than pickle).\r\n\r\n@AckslD Feel free to assign to me. I can fix the issue and generate a pull request.","author":{"url":"https://github.com/brunorijsman","@type":"Person","name":"brunorijsman"},"datePublished":"2019-12-06T14:05:53.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":2},"url":"https://github.com/59/CQC-Python/issues/59"}