[ARTEMIS-3947] Expiry settings not merging properly

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.24.0
• Fix Version/s: 2.25.0
• Assignee: Justin Bertram
• Reporter: Justin Bertram
• Created: 20/Aug/22 02:42
• Updated: 05/Oct/22 08:54
• Resolved: 05/Oct/22 08:54

Description

Setting expiry-delay in <address-setting match="#"> doesn't work if we set another property aside from expiry-delay and don't set expiry-delay in a specific match.

Test 1 - Fails

   <address-setting match="#">
      <expiry-delay>40000</expiry-delay>
   </address-setting>
   <address-setting match="myAddress">      
      <max-size-bytes>12428800</max-size-bytes>
   </address-setting>

Result:  When I send a message to a queue on myAddress using the default expiration time (0 meaning don't expire), the broker doesn't change the expiration time to 40 seconds. Messages never expire.

Test 2 - OK

   <address-setting match="#">
      <expiry-delay>40000</expiry-delay>
   </address-setting>
   <address-setting match="myAddress">
      <expiry-delay>60000</expiry-delay>
   </address-setting>

Result:  When I send a message to a queue on myAddress using the default expiration time (0 meaning don't expire) the broker changes the expiration time to 60 seconds.

Test 3 - OK

   <address-setting match="#">
      <expiry-delay>40000</expiry-delay>
  </address-setting>

Result:  When I send a message to a queue on myAddress using the default expiration time (0 meaning don't expire) the broker changes the expiration time to 40 seconds.

Issue Links

• links to: GitHub Pull Request #4182