Bug 504650 - If expiration is set on a message published with a ttl, don't reset it
Summary: If expiration is set on a message published with a ttl, don't reset it
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Enterprise MRG
Classification: Red Hat
Component: qpid-cpp
Version: 1.1
Hardware: All
OS: Linux
medium
low
Target Milestone: 1.1.2
: ---
Assignee: Gordon Sim
QA Contact: Jan Sarenik
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2009-06-08 17:04 UTC by Gordon Sim
Modified: 2009-06-12 17:39 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2009-06-12 17:39:44 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
Test program (1.40 KB, text/x-c++src)
2009-06-08 17:36 UTC, Gordon Sim
no flags Details
Fix (888 bytes, patch)
2009-06-08 17:40 UTC, Gordon Sim
no flags Details | Diff


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHBA-2009:1097 0 normal SHIPPED_LIVE Red Hat Enterprise MRG Messaging bug fixing update 2009-06-12 17:38:48 UTC

Description Gordon Sim 2009-06-08 17:04:20 UTC
Description of problem:

The JMS client calculates the expiration of a message itself on sending the message. The broker should not overwrite the clients value.

Version-Release number of selected component (if applicable):

Since 1.1

How reproducible:

100%

Steps to Reproduce:
1. send message with both ttl and expiration set
2. subscrbe and receive message
3. check that expiration on received message is exactly the value that was sent
  
Actual results:

The broker recalculates the expiration.

Expected results:

The expiration should be identical to the value set on sent message.

Additional info:

Comment 1 Gordon Sim 2009-06-08 17:36:47 UTC
Created attachment 346899 [details]
Test program

Fails before fix with "Expiration did not match".

Comment 2 Gordon Sim 2009-06-08 17:40:23 UTC
Created attachment 346901 [details]
Fix

Comment 3 Gordon Sim 2009-06-08 18:37:51 UTC
Fixed on trunk as r782712.

Comment 4 Gordon Sim 2009-06-09 08:23:11 UTC
Fyi: arg::durable=true in test program was supposed to be arg::autoDelete=true. Doesn't affect the core purpose of the test, but as it is it leaves queues lying around.

Comment 5 Gordon Sim 2009-06-09 08:26:22 UTC
Fixed in qpidd-0.5.752581-13.el5

Comment 6 Gordon Sim 2009-06-09 11:55:49 UTC
On RHEL4 you will need CXXFLAGS=-I/usr/include/qpid-boost as usual in order to compile the cpp test.

Comment 7 Jan Sarenik 2009-06-09 12:08:34 UTC
Reproduced on 0.5.752581-5

Verified on -13, both archs, both RHELs.

Comment 9 errata-xmlrpc 2009-06-12 17:39:44 UTC
An advisory has been issued which should help the problem
described in this bug report. This report is therefore being
closed with a resolution of ERRATA. For more information
on therefore solution and/or where to find the updated files,
please follow the link below. You may reopen this bug report
if the solution does not work for you.

http://rhn.redhat.com/errata/RHBA-2009-1097.html


Note You need to log in before you can comment on or make changes to this bug.