Merge pull request #20 from AntAndriy7/dependabot/nuget/NetSdrClientA… #44
Annotations
11 warnings
|
Sonar Check
Node.js 20 actions are deprecated. The following actions are running on Node.js 20 and may not work as expected: actions/checkout@v4, actions/setup-dotnet@v4. Actions will be forced to run with Node.js 24 by default starting June 2nd, 2026. Node.js 20 will be removed from the runner on September 16th, 2026. Please check if updated versions of these actions are available that support Node.js 24. To opt into Node.js 24 now, set the FORCE_JAVASCRIPT_ACTIONS_TO_NODE24=true environment variable on the runner or in your workflow file. Once Node.js 24 becomes the default, you can temporarily opt out by setting ACTIONS_ALLOW_USE_UNSECURE_NODE_VERSION=true. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|
|
Sonar Check:
NetSdrClientApp/Networking/UdpClientWrapper.cs#L9
This class overrides 'GetHashCode' and should therefore also override 'Equals'.
|
|
Sonar Check:
NetSdrClientApp/Networking/UdpClientWrapper.cs#L9
Move 'UdpClientWrapper' into a named namespace.
|
|
Sonar Check:
NetSdrClientApp/Networking/TcpClientWrapper.cs#L43
Dispose '_cts' when it is no longer needed.
|
|
Sonar Check:
NetSdrClientApp/Networking/UdpClientWrapper.cs#L24
Dispose '_cts' when it is no longer needed.
|
|
Sonar Check:
NetSdrClientApp/Networking/IUdpClient.cs#L2
Move 'IUdpClient' into a named namespace.
|
|
Sonar Check:
NetSdrClientApp/Messages/NetSdrMessageHelper.cs#L10
Complete the task associated to this 'TODO' comment.
|
|
Sonar Check:
NetSdrClientApp/Networking/UdpClientWrapper.cs#L38
The variable 'ex' is declared but never used
|
|
Sonar Check:
NetSdrClientApp/Networking/TcpClientWrapper.cs#L112
The variable 'ex' is declared but never used
|
|
Sonar Check:
NetSdrClientApp/Networking/TcpClientWrapper.cs#L63
Cannot convert null literal to non-nullable reference type.
|
|
Sonar Check:
NetSdrClientApp/Networking/TcpClientWrapper.cs#L25
Non-nullable field '_cts' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the field as nullable.
|