rejig services for better seperation

Signed-off-by: Neil South <[email protected]>

Author: neildsouth

.github/.gitversion.yml

@@ -18,13 +18,13 @@ branches:
   main:
     tag: ''
   release:
-    tag: rc
+    tag: 'rc'
   develop:
-    tag: beta
+    tag: 'beta'
   feature:
-    tag: alpha.{BranchName}
+    tag: 'alpha.{BranchName}'
   pull-request:
-    tag: pr
+    tag: 'pr'
 
 ignore:
   sha: []

doc/dependency_decisions.yml

@@ -588,6 +588,7 @@
       - 8.0.2
       - 8.0.3
       - 8.0.6
+      - 8.0.7
     :when: 2022-10-14T23:37:16.793Z
     :who: mocsharp
     :why: MIT (https://github.com/dotnet/runtime/raw/main/LICENSE.TXT)

src/InformaticsGateway/Program.cs

@@ -36,6 +36,7 @@
 using Monai.Deploy.InformaticsGateway.Services.DicomWeb;
 using Monai.Deploy.InformaticsGateway.Services.Export;
 using Monai.Deploy.InformaticsGateway.Services.Fhir;
+using Monai.Deploy.InformaticsGateway.Services.HealthLevel7;
 using Monai.Deploy.InformaticsGateway.Services.Http;
 using Monai.Deploy.InformaticsGateway.Services.Scp;
 using Monai.Deploy.InformaticsGateway.Services.Scu;
@@ -138,7 +139,6 @@ internal static IHostBuilder CreateHostBuilder(string[] args) =>
                     services.AddSingleton<IScuQueue, ScuQueue>();
                     services.AddSingleton<IMllpService, MllpService>();
 
-
                     var timeout = TimeSpan.FromSeconds(hostContext.Configuration.GetValue("InformaticsGateway:dicomWeb:clientTimeout", DicomWebConfiguration.DefaultClientTimeout));
                     services
                         .AddHttpClient("dicomweb", configure => configure.Timeout = timeout)
@@ -163,7 +163,7 @@ internal static IHostBuilder CreateHostBuilder(string[] args) =>
                     services.AddHostedService<ScuExportService>();
                     services.AddHostedService<DicomWebExportService>();
                     services.AddHostedService<PayloadNotificationService>();
-                    services.AddHostedService<MllpService>();
+                    services.AddHostedService<MllpServiceHost>();
                     services.AddHostedService<Hl7ExportService>();
 
                 })

src/InformaticsGateway/Services/Export/Hl7ExportService.cs

@@ -28,37 +28,38 @@
 using Monai.Deploy.InformaticsGateway.Configuration;
 using Monai.Deploy.InformaticsGateway.Database.Api.Repositories;
 using Monai.Deploy.InformaticsGateway.Logging;
-using Monai.Deploy.InformaticsGateway.Api.Mllp;
 using Monai.Deploy.Messaging.Common;
 using Polly;
+using Monai.Deploy.InformaticsGateway.Api.Mllp;
 
 namespace Monai.Deploy.InformaticsGateway.Services.Export
 {
     internal class Hl7ExportService : ExportServiceBase
     {
         private readonly ILogger<Hl7ExportService> _logger;
         private readonly InformaticsGatewayConfiguration _configuration;
-        private readonly IMllpService _mllpService;
 
         protected override ushort Concurrency { get; }
         public override string RoutingKey { get; }
         public override string ServiceName => "DICOM Export HL7 Service";
+        private readonly IMllpService _mllpService;
 
 
         public Hl7ExportService(
             ILogger<Hl7ExportService> logger,
             IServiceScopeFactory serviceScopeFactory,
             IOptions<InformaticsGatewayConfiguration> configuration,
-            IDicomToolkit dicomToolkit)
+            IDicomToolkit dicomToolkit,
+            IMllpService mllpService)
             : base(logger, configuration, serviceScopeFactory, dicomToolkit)
         {
             _logger = logger ?? throw new ArgumentNullException(nameof(logger));
             _configuration = configuration.Value ?? throw new ArgumentNullException(nameof(configuration));
 
-            _mllpService = serviceScopeFactory.CreateScope().ServiceProvider.GetRequiredService<IMllpService>();
             RoutingKey = $"{configuration.Value.Messaging.Topics.ExportHL7}";
             ExportCompleteTopic = $"{configuration.Value.Messaging.Topics.ExportHl7Complete}";
             Concurrency = _configuration.Dicom.Scu.MaximumNumberOfAssociations;
+            _mllpService = mllpService ?? throw new ArgumentNullException(nameof(mllpService));
         }
 
 
@@ -159,6 +160,5 @@ protected override Task<ExportRequestDataMessage> ExecuteOutputDataEngineCallbac
         {
             return Task.FromResult(exportDataRequest);
         }
-
     }
 }

src/InformaticsGateway/Services/HealthLevel7/MllpService.cs

@@ -1,267 +1,28 @@
-/*
- * Copyright 2022-2023 MONAI Consortium
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-using System;
-using System.Collections.Concurrent;
-using System.Collections.Generic;
-using System.IO.Abstractions;
-using System.Linq;
+using System;
 using System.Net;
 using System.Net.Sockets;
 using System.Text;
 using System.Threading;
 using System.Threading.Tasks;
-using Ardalis.GuardClauses;
 using HL7.Dotnetcore;
-using Microsoft.Extensions.DependencyInjection;
-using Microsoft.Extensions.Hosting;
 using Microsoft.Extensions.Logging;
 using Microsoft.Extensions.Options;
-using Monai.Deploy.InformaticsGateway.Api.PlugIns;
-using Monai.Deploy.InformaticsGateway.Api.Rest;
-using Monai.Deploy.InformaticsGateway.Api.Storage;
-using Monai.Deploy.InformaticsGateway.Common;
+using Monai.Deploy.InformaticsGateway.Api.Mllp;
 using Monai.Deploy.InformaticsGateway.Configuration;
-using Monai.Deploy.InformaticsGateway.Database.Api.Repositories;
 using Monai.Deploy.InformaticsGateway.Logging;
-using Monai.Deploy.InformaticsGateway.Services.Common;
-using Monai.Deploy.InformaticsGateway.Services.Connectors;
-using Monai.Deploy.InformaticsGateway.Services.HealthLevel7;
-using Monai.Deploy.InformaticsGateway.Services.Storage;
-using Monai.Deploy.Messaging.Events;
 
-namespace Monai.Deploy.InformaticsGateway.Api.Mllp
+namespace Monai.Deploy.InformaticsGateway.Services.HealthLevel7
 {
-    internal sealed class MllpService : IMllpService, IHostedService, IDisposable, IMonaiService
+    internal class MllpService : IMllpService
     {
-        private const int SOCKET_OPERATION_CANCELLED = 125;
-        private bool _disposedValue;
-        private readonly ITcpListener _tcpListener;
-        private readonly IMllpClientFactory _mllpClientFactory;
-        private readonly IObjectUploadQueue _uploadQueue;
-        private readonly IPayloadAssembler _payloadAssembler;
-        private readonly IFileSystem _fileSystem;
-        private readonly ILoggerFactory _logginFactory;
-        private readonly ILogger<MllpService> _logger;
-        private readonly IOptions<InformaticsGatewayConfiguration> _configuration;
-        private readonly IStorageInfoProvider _storageInfoProvider;
-        private readonly ConcurrentDictionary<Guid, IMllpClient> _activeTasks;
-        private readonly IMllpExtract _mIIpExtract;
-        private readonly IInputHL7DataPlugInEngine _inputHL7DataPlugInEngine;
-        private readonly IHl7ApplicationConfigRepository _hl7ApplicationConfigRepository;
-        private DateTime _lastConfigRead = new(2000, 1, 1, 0, 0, 0, DateTimeKind.Utc);
-
-        public int ActiveConnections
-        {
-            get
-            {
-                return _activeTasks.Count;
-            }
-        }
-
-        public ServiceStatus Status { get; set; } = ServiceStatus.Unknown;
-
-        public string ServiceName => "HL7 Service";
-
-        public MllpService(IServiceScopeFactory serviceScopeFactory, IOptions<InformaticsGatewayConfiguration> configuration)
-        {
-            ArgumentNullException.ThrowIfNull(serviceScopeFactory, nameof(serviceScopeFactory));
-
-            _configuration = configuration ?? throw new ArgumentNullException(nameof(configuration));
-
-            var serviceScope = serviceScopeFactory.CreateScope();
-            _logginFactory = serviceScope.ServiceProvider.GetService<ILoggerFactory>() ?? throw new ServiceNotFoundException(nameof(ILoggerFactory));
-            _logger = _logginFactory.CreateLogger<MllpService>();
-            var tcpListenerFactory = serviceScope.ServiceProvider.GetService<ITcpListenerFactory>() ?? throw new ServiceNotFoundException(nameof(ITcpListenerFactory));
-            _tcpListener = tcpListenerFactory.CreateTcpListener(System.Net.IPAddress.Any, _configuration.Value.Hl7.Port);
-            _mllpClientFactory = serviceScope.ServiceProvider.GetService<IMllpClientFactory>() ?? throw new ServiceNotFoundException(nameof(IMllpClientFactory));
-            _uploadQueue = serviceScope.ServiceProvider.GetService<IObjectUploadQueue>() ?? throw new ServiceNotFoundException(nameof(IObjectUploadQueue));
-            _payloadAssembler = serviceScope.ServiceProvider.GetService<IPayloadAssembler>() ?? throw new ServiceNotFoundException(nameof(IPayloadAssembler));
-            _fileSystem = serviceScope.ServiceProvider.GetService<IFileSystem>() ?? throw new ServiceNotFoundException(nameof(IFileSystem));
-            _storageInfoProvider = serviceScope.ServiceProvider.GetService<IStorageInfoProvider>() ?? throw new ServiceNotFoundException(nameof(IStorageInfoProvider));
-            _mIIpExtract = serviceScope.ServiceProvider.GetService<IMllpExtract>() ?? throw new ServiceNotFoundException(nameof(IMllpExtract));
-            _activeTasks = new ConcurrentDictionary<Guid, IMllpClient>();
-            _inputHL7DataPlugInEngine = serviceScope.ServiceProvider.GetService<IInputHL7DataPlugInEngine>() ?? throw new ServiceNotFoundException(nameof(IInputHL7DataPlugInEngine));
-            _hl7ApplicationConfigRepository = serviceScope.ServiceProvider.GetService<IHl7ApplicationConfigRepository>() ?? throw new ServiceNotFoundException(nameof(IHl7ApplicationConfigRepository));
-        }
-
-        public Task StartAsync(CancellationToken cancellationToken)
-        {
-            var task = Task.Run(async () =>
-            {
-                _tcpListener.Start();
-                await BackgroundProcessing(cancellationToken).ConfigureAwait(true);
-            }, CancellationToken.None);
-
-            Status = ServiceStatus.Running;
-            _logger.ServiceRunning(ServiceName);
-            _logger.Hl7ListeningOnPort(_configuration.Value.Hl7.Port);
-
-            if (task.IsCompleted)
-                return task;
-            return Task.CompletedTask;
-        }
-
-        public Task StopAsync(CancellationToken cancellationToken)
-        {
-            _logger.ServiceStopping(ServiceName);
-            _tcpListener.Stop();
-            Status = ServiceStatus.Stopped;
-            return Task.CompletedTask;
-        }
-
-        private async Task BackgroundProcessing(CancellationToken cancellationToken)
-        {
-            while (!cancellationToken.IsCancellationRequested)
-            {
-                IMllpClient? mllpClient = null;
-                try
-                {
-                    WaitUntilAvailable(_configuration.Value.Hl7.MaximumNumberOfConnections);
-                    var client = await _tcpListener.AcceptTcpClientAsync(cancellationToken).ConfigureAwait(false);
-                    _logger.ClientConnected();
-
-                    if (!_storageInfoProvider.HasSpaceAvailableToStore)
-                    {
-                        _logger.Hl7DisconnectedDueToLowStorageSpace(_storageInfoProvider.AvailableFreeSpace);
-                        client.Close();
-                        await Task.Delay(5000, cancellationToken).ConfigureAwait(false);
-                        continue;
-                    }
-
-                    mllpClient = _mllpClientFactory.CreateClient(client, _configuration.Value.Hl7, _logginFactory.CreateLogger<MllpClient>());
-                    _ = mllpClient.Start(OnDisconnect, cancellationToken);
-                    _activeTasks.TryAdd(mllpClient.ClientId, mllpClient);
-                }
-                catch (System.Net.Sockets.SocketException ex)
-                {
-                    _logger.Hl7SocketException(ex.Message);
-
-                    if (mllpClient is not null)
-                    {
-                        mllpClient.Dispose();
-                        _activeTasks.Remove(mllpClient.ClientId, out _);
-                    }
-
-                    if (ex.ErrorCode == SOCKET_OPERATION_CANCELLED)
-                    {
-                        break;
-                    }
-                }
-                catch (Exception ex)
-                {
-                    _logger.ServiceInvalidOrCancelled(ServiceName, ex);
-                }
-            }
-            Status = ServiceStatus.Cancelled;
-            _logger.ServiceCancelled(ServiceName);
-        }
-
-        private async Task OnDisconnect(IMllpClient client, MllpClientResult result)
-        {
-            Guard.Against.Null(client, nameof(client));
-            Guard.Against.Null(result, nameof(result));
-
-            await ConfigurePlugInEngine().ConfigureAwait(false);
-
-            try
-            {
-                foreach (var message in result.Messages)
-                {
-                    var newMessage = message;
-                    var hl7Filemetadata = new Hl7FileStorageMetadata(client.ClientId.ToString(), DataService.HL7, client.ClientIp);
-                    var configItem = await _mIIpExtract.GetConfigItem(message).ConfigureAwait(false);
-                    if (configItem is not null)
-                    {
-                        await _inputHL7DataPlugInEngine.ExecutePlugInsAsync(message, hl7Filemetadata, configItem).ConfigureAwait(false);
-                        newMessage = await _mIIpExtract.ExtractInfo(hl7Filemetadata, message, configItem).ConfigureAwait(false);
-
-                        _logger.HL7MessageAfterPluginProcessing(newMessage.HL7Message, hl7Filemetadata.CorrelationId);
-                    }
-                    _logger.Hl7MessageReceieved(newMessage.HL7Message);
-                    await hl7Filemetadata.SetDataStream(newMessage.HL7Message, _configuration.Value.Storage.TemporaryDataStorage, _fileSystem, _configuration.Value.Storage.LocalTemporaryStoragePath).ConfigureAwait(false);
-                    var payloadId = await _payloadAssembler.Queue(client.ClientId.ToString(), hl7Filemetadata, new DataOrigin { DataService = DataService.HL7, Source = client.ClientIp, Destination = FileStorageMetadata.IpAddress() }).ConfigureAwait(false);
-                    hl7Filemetadata.PayloadId ??= payloadId.ToString();
-                    _uploadQueue.Queue(hl7Filemetadata);
-                }
-            }
-            catch (Exception ex)
-            {
-                _logger.ErrorHandlingHl7Results(ex);
-            }
-            finally
-            {
-                _activeTasks.Remove(client.ClientId, out _);
-                _logger.Hl7ClientRemoved(client.ClientId);
-                client.Dispose();
-            }
-        }
 
-        private async Task ConfigurePlugInEngine()
-        {
-            var configs = await _hl7ApplicationConfigRepository.GetAllAsync().ConfigureAwait(false);
-            if (configs is not null && configs.Count is not 0 && configs.Max(c => c.LastModified) > _lastConfigRead)
-            {
-                var pluginAssemblies = new List<string>();
-                foreach (var config in configs.Where(p => p.PlugInAssemblies?.Count > 0))
-                {
-                    try
-                    {
-                        pluginAssemblies.AddRange(config.PlugInAssemblies.Where(p => string.IsNullOrWhiteSpace(p) is false && pluginAssemblies.Contains(p) is false));
-                    }
-                    catch (Exception ex)
-                    {
-                        _logger.HL7PluginLoadingExceptions(ex);
-                    }
-                }
-                if (pluginAssemblies.Count is not 0)
-                {
-                    _inputHL7DataPlugInEngine.Configure(pluginAssemblies);
-                }
-            }
-            _lastConfigRead = DateTime.UtcNow;
-        }
-
-        private void WaitUntilAvailable(int maximumNumberOfConnections)
-        {
-            var count = 0;
-            while (ActiveConnections >= maximumNumberOfConnections)
-            {
-                if (++count % 25 == 1)
-                {
-                    _logger.MaxedOutHl7Connections(maximumNumberOfConnections);
-                }
-                Thread.Sleep(100);
-            }
-        }
+        private readonly ILogger<MllpService> _logger;
+        private readonly InformaticsGatewayConfiguration _configuration;
 
-        private void Dispose(bool disposing)
+        public MllpService(ILogger<MllpService> logger, IOptions<InformaticsGatewayConfiguration> configuration)
         {
-            if (!_disposedValue)
-            {
-                if (disposing)
-                {
-                    foreach (var client in _activeTasks.Values)
-                    {
-                        client.Dispose();
-                    }
-                }
-
-                _disposedValue = true;
-            }
+            _logger = logger ?? throw new ArgumentNullException(nameof(logger));
+            _configuration = configuration?.Value ?? throw new ArgumentNullException(nameof(configuration)); ;
         }
 
         public async Task SendMllp(IPAddress address, int port, string hl7Message, CancellationToken cancellationToken)
@@ -312,7 +73,7 @@ private async Task WriteMessage(byte[] sendMessageByteBuffer, IPAddress address,
         private async Task EnsureAck(NetworkStream networkStream)
         {
             using var s_cts = new CancellationTokenSource();
-            s_cts.CancelAfter(_configuration.Value.Hl7.ClientTimeoutMilliseconds);
+            s_cts.CancelAfter(_configuration.Hl7.ClientTimeoutMilliseconds);
             var buffer = new byte[2048];
 
             // get the SentHl7Message
@@ -344,12 +105,5 @@ private async Task EnsureAck(NetworkStream networkStream)
             }
             throw new Hl7SendException("ACK message contains no ACK!");
         }
-
-        public void Dispose()
-        {
-            // Do not change this code. Put cleanup code in 'Dispose(bool disposing)' method
-            Dispose(disposing: true);
-            GC.SuppressFinalize(this);
-        }
     }
 }