Skip to content

Commit

Permalink
Merge pull request #4291 from Particular/john/filter
Browse files Browse the repository at this point in the history
Filter sqs fifo queue at the transport level
  • Loading branch information
jpalac authored Jul 7, 2024
2 parents 043ec61 + 383cbc9 commit 39b230d
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,6 @@ public static bool IsPlatformEndpoint(string endpointName, ThroughputSettings th
|| endpointName.Equals(throughputSettings.ServiceControlQueue, StringComparison.OrdinalIgnoreCase)
|| endpointName.EndsWith(".Timeouts", StringComparison.OrdinalIgnoreCase)
|| endpointName.EndsWith(".TimeoutsDispatcher", StringComparison.OrdinalIgnoreCase)
|| endpointName.EndsWith("-delay.fifo", StringComparison.OrdinalIgnoreCase)
|| endpointName.StartsWith($"{throughputSettings.ServiceControlQueue}.", StringComparison.OrdinalIgnoreCase)
|| endpointName.Equals(ServiceControlSettings.ServiceControlThroughputDataQueue, StringComparison.OrdinalIgnoreCase)
|| AuditThroughputCollectorHostedService.AuditQueues.Any(a => endpointName.Equals(a, StringComparison.OrdinalIgnoreCase));
Expand Down
5 changes: 4 additions & 1 deletion src/ServiceControl.Transports.SQS/AmazonSQSQuery.cs
Original file line number Diff line number Diff line change
Expand Up @@ -243,7 +243,10 @@ public override async IAsyncEnumerable<IBrokerQueue> GetQueueNames(

foreach (var queue in response.QueueUrls.Select(url => url.Split('/')[4]))
{
yield return new DefaultBrokerQueue(queue);
if (!queue.EndsWith("-delay.fifo", StringComparison.OrdinalIgnoreCase))
{
yield return new DefaultBrokerQueue(queue);
}
}

if (response.NextToken is not null)
Expand Down

0 comments on commit 39b230d

Please sign in to comment.