Skip to content
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

GDS: fix Application unregistration in SQL DB for Winforms Server & fix private key storage in Pull Model GDS Client #598

Merged

Conversation

romanett
Copy link
Contributor

Proposed changes

  • GDS: fix Application unregistration in SQL DB for Winforms Server
  • fix private key creation for New Certificate in Pull Model GDS Client

Related Issues

  • Fixes #

Types of changes

  • Bugfix (non-breaking change which fixes an issue)
  • Enhancement (non-breaking change which adds functionality)
  • Test enhancement (non-breaking change to increase test coverage)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected, requires version increase of Nuget packages)
  • Documentation Update (if none of the other choices apply)

Checklist

  • I have read the CONTRIBUTING doc.
  • I have signed the CLA.
  • I ran tests locally with my changes, all passed.
  • I fixed all failing tests in the CI pipelines.
  • I fixed all introduced issues with CodeQL and LGTM.
  • I have added tests that prove my fix is effective or that my feature works and increased code coverage.
  • I have added necessary documentation (if appropriate).
  • Any dependent changes have been merged and published in downstream modules.

Further comments

… Pull Model certificate creation in GDS Client
@romanett romanett requested a review from mregen April 6, 2024 09:05
@mregen
Copy link
Contributor

mregen commented May 3, 2024

Nuget versions mismatch, build error

@romanett
Copy link
Contributor Author

romanett commented May 6, 2024

Nuget versions mismatch, build error

@mregen locally my build works fine, can you provide an error message?

@mregen
Copy link
Contributor

mregen commented May 6, 2024

see https://github.com/OPCFoundation/UA-.NETStandard-Samples/pull/603/checks?check_run_id=24548958539
updating Nuget packages means all or nothing. Unfortunately dependabot has decided to be silent on our packages.

@mregen mregen merged commit c1c3fb1 into OPCFoundation:master May 6, 2024
6 of 8 checks passed
@romanett romanett deleted the fixPrivateKey_UnregisterApplication branch May 6, 2024 14:39
@sveinfolkeson
Copy link

sveinfolkeson commented May 6, 2024

The problem with private key is still present. When I debug I can see that already when I create the certificate for the first time it seems to be OK when I watch it in the certificate store, but the statements:

                            newCert = new X509Certificate2(privateKeyPFX, string.Empty, X509KeyStorageFlags.Exportable);
                            newCert = CertificateFactory.Load(newCert, true);

Returns newCert with 'newCert.PrivateKey' threw an exception of type 'System.Security.Cryptography.CryptographicException'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unregister Application is not deleting corresponding rows in CertificateStores
3 participants