OPCFoundation / UA-.NETStandard

OPC Unified Architecture .NET Standard
Other
1.97k stars 950 forks source link

[Server] Fix: Return full CertificateChain after Certificate Update #2855

Closed romanett closed 2 days ago

romanett commented 5 days ago

Proposed changes

Fix the CertificateTypesProvider to also return the correct chain after a CertificateUpdate occured.

Related Issues

Types of changes

Checklist

codecov[bot] commented 5 days ago

Codecov Report

Attention: Patch coverage is 78.04878% with 9 lines in your changes missing coverage. Please review.

Project coverage is 55.30%. Comparing base (ce33d63) to head (bef4079). Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
.../Security/Certificates/CertificateTypesProvider.cs 78.57% 1 Missing and 2 partials :warning:
...a.Server/Configuration/ConfigurationNodeManager.cs 81.81% 2 Missing :warning:
...ndings.Https/Stack/Https/HttpsTransportListener.cs 0.00% 2 Missing :warning:
Stack/Opc.Ua.Core/Stack/Server/ServerBase.cs 85.71% 0 Missing and 1 partial :warning:
....Ua.Core/Stack/Tcp/UaSCBinaryChannel.Asymmetric.cs 50.00% 0 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #2855 +/- ## ========================================== - Coverage 55.70% 55.30% -0.41% ========================================== Files 352 352 Lines 67332 67342 +10 Branches 13806 13805 -1 ========================================== - Hits 37506 37241 -265 - Misses 25792 26019 +227 - Partials 4034 4082 +48 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.