DefaultMessageTimeToLive should only be set when not in BasicTier

tinglesoftware · Apr 19, 2021 · 1252f90 · 1252f90
1 parent da544ae
commit 1252f90
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/Tingle.EventBus.Transports.Azure.ServiceBus/AzureServiceBusTransport.cs b/src/Tingle.EventBus.Transports.Azure.ServiceBus/AzureServiceBusTransport.cs
@@ -442,16 +442,16 @@ private async Task CreateQueueIfNotExistsAsync(EventRegistration ereg, string na
                     // set the defaults for a queue here
                     Status = EntityStatus.Active,
                     EnablePartitioning = false,
-                    DefaultMessageTimeToLive = Defaults.DefaultMessageTimeToLive,
                     EnableBatchedOperations = true,
                     DeadLetteringOnMessageExpiration = true,
                     LockDuration = Defaults.LockDuration,
                     MaxDeliveryCount = Defaults.MaxDeliveryCount,
                 };
 
-                // Certain properties are not allowed in Basic Tier
+                // Certain properties are not allowed in Basic Tier or have lower limits
                 if (!await IsBasicTierAsync(cancellationToken))
                 {
+                    options.DefaultMessageTimeToLive = Defaults.DefaultMessageTimeToLive; // defaults to 14days in basic tier
                     options.RequiresDuplicateDetection = BusOptions.EnableDeduplication;
                     options.DuplicateDetectionHistoryTimeWindow = BusOptions.DuplicateDetectionDuration;
                     options.AutoDeleteOnIdle = Defaults.AutoDeleteOnIdle;