[ 
https://issues.apache.org/jira/browse/PROTON-1505?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16349451#comment-16349451
 ] 

ASF subversion and git services commented on PROTON-1505:
---------------------------------------------------------

Commit c6bfeff2a22f3e9495140df3cef046d6708cf5c1 in qpid-proton's branch 
refs/heads/master from [~astitcher]
[ https://git-wip-us.apache.org/repos/asf?p=qpid-proton.git;h=c6bfeff ]

PROTON-1505: Fix encoding and decoding of message priority
- Added missing default message header fields to xml
- Fixed decoder to set priority to default of not set rather than 0
- Fixed encoder to not set priority if it is default
- Added tests for encoding/decoding


> Message header defaults only work if no header present
> ------------------------------------------------------
>
>                 Key: PROTON-1505
>                 URL: https://issues.apache.org/jira/browse/PROTON-1505
>             Project: Qpid Proton
>          Issue Type: Bug
>          Components: proton-c
>            Reporter: Kim van der Riet
>            Assignee: Andrew Stitcher
>            Priority: Major
>              Labels: codec
>             Fix For: proton-c-0.21.0
>
>
> An error in the qpid-interop-tests JMS headers test showed that where there 
> is no message priority on the wire, the C++ and Python client APIs do not 
> return the default value as mandated by the AMQP spec. In particular, the 
> priority is returned with a value of 0 rather than the default value of 4.
> This raises two issues:
> 1. Setting the default value (in particularly for priority, where a priority 
> system may be adopted that is not default, and the missing value may need to 
> have a value other than 4). 
> 2. (C++) Gaining access to the transport headers so that there is an ability 
> to distinguish between the default and the value being actually present on 
> the wire.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to