Azure Notification Hub static instance or recreate about azure-notificationhubs-dotnet HOT 5 CLOSED

@tiaan-lg yes, that would be a recommended approach. Please reopen this issue if this is still a problem.

from azure-notificationhubs-dotnet.

For anybody struggling with this, we solved it by passing in a MessageHandler from the
IHttpMessageHandlerFactory this prevents it from recreating the HTTP client and starving sockets

Example code where _httpMessageHandler is a static property

// if handler is not created, then create it
if (_httpMessageHandler == null)
{
    var serviceProvider = new ServiceCollection().AddHttpClient().BuildServiceProvider();

    var httpClientFactory = serviceProvider.GetService<IHttpMessageHandlerFactory>();

    _httpMessageHandler = httpClientFactory.CreateHandler();
}

var hub = new NotificationHubClient({{Connection String}}, {{Hub Path}}, new NotificationHubClientSettings
{
    MessageHandler = _httpMessageHandler
});

from azure-notificationhubs-dotnet.

I am also experiencing issues with using the hub client whereby it is exhausting sockets, what is the best practice for creating the and reusing the NotificationHubClient

from azure-notificationhubs-dotnet.

@mpodwysocki Can you please clarify if this approach is correct or not, we are adding the following code in the Constructor of our Controller.

private NotificationHubClient _nhClient;
public NotificationsController(CMLogger logger,
                                       IConfiguration config, IHttpMessageHandlerFactory httpMessageHandler)
        {
            _logger = logger;
            _config = config;
            _nhClient = new NotificationHubClient(_config["NotificationsHub:ConnectionString"],
             _config["NotificationsHub:HubName"],
             new NotificationHubClientSettings
             {
                 OperationTimeout = TimeSpan.FromSeconds(10),
                 MessageHandler = httpMessageHandler.CreateHandler()
             })
        }

from azure-notificationhubs-dotnet.

@tiaan-lg @mpodwysocki

Example code where _httpMessageHandler is a static property

Wouldn't this static HttpMessageHandler result in an also static HttpClientHandler for the whole lifetime of the application and therefore, might cause issues with DNS changes?

If I understand the inner workings of DefaultHttpClientFactory (which implements IHttpMessageHandlerFactory) correctly, calling CreateHandler() everytime a NotificationHubClient with scoped lifetime is created, should reuse the same handler pipeline. At least withing the lifetime of ActiveHandlerTrackingEntry. This reusing should mitigate the problem of socket exhaustion.

Please correct me if I understood something wrong.

from azure-notificationhubs-dotnet.