Update AppInsightsProducerEventsHandler.cs (#2) · agoda-com/KafkaFlow.ApplicationInsights@95a514a

The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-dotnet@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/

Non-nullable property 'EndpointAddress' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.

Dereference of a possibly null reference.

'DependencyTelemetry.DependencyKind' is obsolete: 'Use Type'

'DependencyTelemetry.DependencyTypeName' is obsolete: 'Renamed to Type'

'DependencyTelemetry.DependencyKind' is obsolete: 'Use Type'

'DependencyTelemetry.DependencyTypeName' is obsolete: 'Renamed to Type'

Remove this commented out code. (https://rules.sonarsource.com/csharp/RSPEC-125)

Fix this implementation of 'IDisposable' to conform to the dispose pattern. (https://rules.sonarsource.com/csharp/RSPEC-3881)

Prevent the use of Thread.Sleep (https://agoda-com.github.io/standards-c-sharp/async/avoid-blocking.html)

Do not use 'Thread.Sleep()' in a test. (https://rules.sonarsource.com/csharp/RSPEC-2925)

Provide feedback