Skip to content

Commit b87e126

Browse files
authored
chore(az.sb): remove unnecessary topic subscription name sanitization (#642)
1 parent 8a10927 commit b87e126

File tree

1 file changed

+0
-17
lines changed

1 file changed

+0
-17
lines changed

src/Arcus.Messaging.Pumps.ServiceBus/Extensions/IServiceCollectionExtensions.cs

Lines changed: 0 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@
88
using Azure.Messaging.ServiceBus;
99
using Microsoft.Extensions.DependencyInjection.Extensions;
1010
using Microsoft.Extensions.Logging;
11-
using Microsoft.Extensions.Logging.Abstractions;
1211

1312
// ReSharper disable once CheckNamespace
1413
namespace Microsoft.Extensions.DependencyInjection
@@ -200,7 +199,6 @@ private static ServiceBusMessageHandlerCollection AddServiceBusMessagePump(
200199

201200
services.AddHostedService(provider =>
202201
{
203-
subscriptionName = SanitizeSubscriptionName(subscriptionName, provider);
204202
var logger = provider.GetService<ILogger<ServiceBusMessagePump>>();
205203

206204
return options.RequestedToUseSessions
@@ -214,20 +212,5 @@ private static ServiceBusMessageHandlerCollection AddServiceBusMessagePump(
214212
UseSessions = options.RequestedToUseSessions
215213
};
216214
}
217-
218-
private static string SanitizeSubscriptionName(string subscriptionName, IServiceProvider provider)
219-
{
220-
var logger =
221-
provider.GetService<ILogger<ServiceBusMessagePump>>()
222-
?? NullLogger<ServiceBusMessagePump>.Instance;
223-
224-
if (subscriptionName is { Length: > 50 })
225-
{
226-
logger.LogWarning("Azure Service Bus Topic subscription name was truncated to 50 characters");
227-
subscriptionName = subscriptionName[..50];
228-
}
229-
230-
return subscriptionName;
231-
}
232215
}
233216
}

0 commit comments

Comments
 (0)