Uploaded image for project: 'Qpid Proton'
  1. Qpid Proton
  2. PROTON-1060

[Python Binding] API call types for some message properties do not match AMQP specification

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.12.0
    • Component/s: python-binding
    • Labels:
      None

      Description

      Several of the Python binding API calls for message properties do not use the correct types. To align them properly with the AMQP 1.0 spec, the following APIs need changing:

      Subject
      The spec for this is: <field name="subject" type="string"/>
      _get_subject() should return a unicode string, not a regular string;
      _set_subject() should accept a unicode string, not a regular string.

      Content Type
      The spec for this is: <field name="content-type" type="symbol"/>
      _get_content_type() should return a symbol, not a regular string;
      _set_content_type() should accept a symbol, not a regular string.

      Content Encoding
      The spec for this is: <field name="content-encoding" type="symbol"/>
      _get_content_encoding() should return a symbol, not a regular string;
      _set_content_encoding() should accept a symbol, not a regular string.

      Group Id
      The spec for this is: <field name="group-id" type="string"/>
      _get_group_id() should return a unicode string, not a regular string;
      _set_group_id() should accept a unicode string, not a regular string.

      Reply-To Group Id
      The spec for this is <field name="reply-to-group-id" type="string"/>
      _get_reply_to_group_id() should return a unicode string, not a regular string;
      _set_reply_to_group_id() should accept a unicode string, not a regular string.

      The unicode string pattern is already established for other message property API calls such as _get/_set_address() and _get/_set_reply_to().

        Attachments

          Activity

            People

            • Assignee:
              kpvdr Kim van der Riet
              Reporter:
              kpvdr Kim van der Riet
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: