msgpack incompatibility with encoding field

Bug #1864811 reported by Nicolas Belouin
20
This bug affects 2 people
Affects Status Importance Assigned to Milestone
oslo.privsep
Fix Released
Undecided
Ben Nemec

Bug Description

msgpack in its 1.0.0 version dropped support for the encoding field.
A change similar to https://opendev.org/openstack/oslo.serialization/commit/d5855294a891b91a6c87977128b19b9bb0588f87 is necessary to ensure compatibility with newer version of msgpack.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to oslo.privsep (master)

Fix proposed to branch: master
Review: https://review.opendev.org/713496

Changed in oslo.privsep:
assignee: nobody → Ben Nemec (bnemec)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to oslo.privsep (master)

Reviewed: https://review.opendev.org/713496
Committed: https://git.openstack.org/cgit/openstack/oslo.privsep/commit/?id=f19765c683ce847f8d0ae0499d9e67c41114c0b7
Submitter: Zuul
Branch: master

commit f19765c683ce847f8d0ae0499d9e67c41114c0b7
Author: Ben Nemec <email address hidden>
Date: Tue Mar 17 17:28:58 2020 +0000

    Make compatible with msgpack 1.0.0

    There are a couple of things that changed in msgpack 1.0.0 that were
    breaking oslo.privsep:

    1) The encoding parameter to Unpacker was removed. This has been
       deprecated for a while in favor of the `raw` parameter[0], so this
       change switches to using raw.

    2) The strict_map_key parameter default was changed from False to
       True.[1] I haven't found an explanation of why this was done, but
       we can explicitly set it False to maintain the previous behavior.

    Change-Id: Ia97ecf965d807f12524d5b6602446934b5813ce6
    Closes-Bug: 1855914
    Closes-Bug: 1864811
    0: https://msgpack-python.readthedocs.io/en/latest/api.html#msgpack.Unpacker
    1: https://github.com/msgpack/msgpack-python/pull/392/commits/6e1d12c0a2072572890cbb124b94cb08b582dd2d

Changed in oslo.privsep:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/oslo.privsep 2.1.0

This issue was fixed in the openstack/oslo.privsep 2.1.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to oslo.privsep (stable/train)

Fix proposed to branch: stable/train
Review: https://review.opendev.org/757033

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to oslo.privsep (stable/train)

Reviewed: https://review.opendev.org/757033
Committed: https://git.openstack.org/cgit/openstack/oslo.privsep/commit/?id=ced0e7b33b617fbfd9137cf2917ab279092dbd7f
Submitter: Zuul
Branch: stable/train

commit ced0e7b33b617fbfd9137cf2917ab279092dbd7f
Author: Ben Nemec <email address hidden>
Date: Tue Mar 17 17:28:58 2020 +0000

    Make compatible with msgpack 1.0.0

    There are a couple of things that changed in msgpack 1.0.0 that were
    breaking oslo.privsep:

    1) The encoding parameter to Unpacker was removed. This has been
       deprecated for a while in favor of the `raw` parameter[0], so this
       change switches to using raw.

    2) The strict_map_key parameter default was changed from False to
       True.[1] I haven't found an explanation of why this was done, but
       we can explicitly set it False to maintain the previous behavior.

    Closes-Bug: 1855914
    Closes-Bug: 1864811
    0: https://msgpack-python.readthedocs.io/en/latest/api.html#msgpack.Unpacker
    1: https://github.com/msgpack/msgpack-python/pull/392/commits/6e1d12c0a2072572890cbb124b94cb08b582dd2d

    Change-Id: Ia97ecf965d807f12524d5b6602446934b5813ce6
    Closes-Bug: 1899140
    (cherry picked from commit f19765c683ce847f8d0ae0499d9e67c41114c0b7)

tags: added: in-stable-train
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.