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

Some Linux docker distributions do not support Brainpool, e.g. Mariner #2902

Closed
1 of 5 tasks
mregen opened this issue Dec 11, 2024 · 1 comment
Closed
1 of 5 tasks

Comments

@mregen
Copy link
Contributor

mregen commented Dec 11, 2024

Type of issue

  • Bug
  • Enhancement
  • Compliance
  • Question
  • Help wanted

Current Behavior

See failing tests in a mariner docker container...

Failed CreateSelfSignedForECDsaDefaultTest(brainpoolP256r1-SHA256) [< 1 ms]
  Error Message:
   System.PlatformNotSupportedException : The specified curve '1.3.36.3.3.2.8.1.1.7' or its parameters are not valid for this platform.
  Stack Trace:
     at System.Security.Cryptography.ECOpenSsl.GenerateKey(ECCurve curve)
   at System.Security.Cryptography.ECDsa.Create(ECCurve curve)
   at Opc.Ua.Security.Certificates.CertificateBuilder.CreateForECDsa() in /__w/1/s/Libraries/Opc.Ua.Security.Certificates/X509Certificate/CertificateBuilder.cs:line 191
   at Opc.Ua.Security.Certificates.Tests.CertificateTestsForECDsa.CreateSelfSignedForECDsaDefaultTest(ECCurveHashPair eccurveHashPair) in /__w/1/s/Tests/Opc.Ua.Security.Certificates.Tests/CertificateTestsForECDsa.cs:line 118
   at InvokeStub_CertificateTestsForECDsa.CreateSelfSignedForECDsaDefaultTest(Object, Span`1)
   at System.Reflection.MethodBaseInvoker.InvokeWithOneArg(Object obj, BindingFlags invokeAttr, Binder binder, Object[] parameters, CultureInfo culture)

Expected Behavior

Tests and Opc.Ua.Core gracefully detects that specific curves are unsupported, so these profiles are disabled.

Steps To Reproduce

No response

Environment

- OS:
- Environment:
- Runtime:
- Nuget Version:
- Component:
- Server:
- Client:

Anything else?

No response

@mregen
Copy link
Contributor Author

mregen commented Dec 17, 2024

fixed with #2903

@mregen mregen closed this as completed Dec 17, 2024
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

No branches or pull requests

1 participant