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

[proton-j] validate details for message-format value on received transfers

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Open
    • Minor
    • Resolution: Unresolved
    • proton-0.12.0
    • proton-j-future
    • proton-j
    • None

    Description

      The spec [1] notes various details for the message-format field which we dont currently enforce, such as that it must be present on the first transfer of a multi-transfer message, can be omitted on continuations, and that [if set] it is an error for the format on a contituation to differ from the first transfer.

      Previously the format was alwayss et to 0 on outgoing transfers, and never used/inspected at all on incoming transfers, with the Delivery#getMessageFormat() method always returning 0. Following PROTON-1096, the value on the first transfer for a message is used to update the message-format value returned from the Delivery object.

      [1] http://docs.oasis-open.org/amqp/core/v1.0/os/amqp-core-transport-v1.0-os.html#type-transfer

      Attachments

        Issue Links

          is blocked by

          Improvement - An improvement or enhancement to an existing feature or task. PROTON-281 Engine should handle errors by generating implicit CLOSE frame

          • Major - Major loss of function.
          • Open
          relates to

          Bug - A problem which impairs or prevents the functions of the product. PROTON-1038 message-format on received transfer is never checked (nor made available for application to check)

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Open

          New Feature - A new feature of the product, which has yet to be developed. PROTON-1096 [proton-j] enable set/get of MessageFormat on the Delivery being sent/received

          • Major - Major loss of function.
          • Closed

          Activity

            People

              Unassigned Unassigned
              robbie Robbie Gemmell
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated: