Change the connection string parser inside the ConnectionSettings class to prevent reinventing the wheel #102

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

benspeth merged 7 commits into EcovadisCode:master from DigitalMushrooms:change-connectionstring-parser-inside-connectionsettings

Nov 25, 2025

Merged

Change the connection string parser inside the ConnectionSettings class to prevent reinventing the wheel #102

Restoring two constructors in ConnectionSettings

Annotations

13 warnings

src/Ev.ServiceBus/Reception/MessageReceptionHandler.cs#L82

Possible null reference argument for parameter 'clientType' in 'void ServiceBusMeter.RecordDeliveryCount(int value, string clientType, string resourceId, string? payloadTypeId)'.

src/Ev.ServiceBus/TextJsonPayloadSerializer.cs#L28

Possible null reference return.

src/Ev.ServiceBus.Prometheus/ServicebusPrometheusExporter.cs#L50

Argument of type 'string?[]' cannot be used for parameter 'labelValues' of type 'string[]' in 'Child Collector<Child>.WithLabels(params string[] labelValues)' due to differences in the nullability of reference types.

src/Ev.ServiceBus/Diagnostics/ServiceBusMeter.cs#L13

Code should not contain trailing whitespace (https://github.com/DotNetAnalyzers/StyleCopAnalyzers/blob/master/documentation/SA1028.md)

src/Ev.ServiceBus/Dispatch/DispatchSender.cs#L195

Non-nullable property 'Sender' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.

src/Ev.ServiceBus/Dispatch/DispatchSender.cs#L194

Non-nullable property 'ResourceId' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.

src/Ev.ServiceBus/Dispatch/DispatchSender.cs#L192

Non-nullable property 'Messages' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.

src/Ev.ServiceBus/Reception/MessageReceptionHandler.cs#L83

Possible null reference argument for parameter 'resourceId' in 'void ServiceBusMeter.RecordDeliveryCount(int value, string clientType, string resourceId, string? payloadTypeId)'.

src/Ev.ServiceBus/Reception/MessageReceptionHandler.cs#L82

Possible null reference argument for parameter 'clientType' in 'void ServiceBusMeter.RecordDeliveryCount(int value, string clientType, string resourceId, string? payloadTypeId)'.

src/Ev.ServiceBus/TextJsonPayloadSerializer.cs#L28

Possible null reference return.

.NET 6.0 is no longer supported and will not receive security updates in the future. Please refer to https://aka.ms/dotnet-core-support for more information about the .NET support policy.

.NET 5.0 is no longer supported and will not receive security updates in the future. Please refer to https://aka.ms/dotnet-core-support for more information about the .NET support policy.

.NET Core 3.1 is no longer supported and will not receive security updates in the future. Please refer to https://aka.ms/dotnet-core-support for more information about the .NET support policy.

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Change the connection string parser inside the ConnectionSettings class to prevent reinventing the wheel #102

Uh oh!

Uh oh!

Change the connection string parser inside the ConnectionSettings class to prevent reinventing the wheel #102

Uh oh!

Annotations

The logs for this run have expired and are no longer available.

Re-running jobs...