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

NullReferenceException when unexpected StoreType #1462

Closed
3 of 5 tasks
guillermo-ruffino opened this issue Jul 15, 2021 · 1 comment
Closed
3 of 5 tasks

NullReferenceException when unexpected StoreType #1462

guillermo-ruffino opened this issue Jul 15, 2021 · 1 comment
Labels
enhancement API or feature enhancement

Comments

@guillermo-ruffino
Copy link

Type of Issue

  • Bug
  • Enhancement
  • Compliance
  • Question
  • Help wanted

Describe the Issue
I'm migrating from .net framework to core and found out that if I use Windows as a certificate store type the CheckApplicationInstanceCertificate method throws a NullReferenceException.

This is happening because this switch

is not handling unexpected values.

The exception is happening exactly here:

I'm not sending a PR because I don't know what should be the right approach to handle this? A NotSupportedException may be, detailing what is going on.

A clear and concise description of what the issue is.

To Reproduce
Steps to reproduce the behavior:

  1. Compile Sample '...'
  2. Connect to Server '....'
  3. Browse Address space '....'
  4. Subscribe to Node '....'
  5. See error

Expected behavior
A clear and concise description of what you expected to happen.

Screenshots
If applicable, add screenshots to help explain your problem.

Log output
If applicable, add log output to help explain your problem.

Code snippets
If applicable, add a code snippet to help explain your problem.

Environment (please complete the following information):

  • OS: Windows 10
  • Development environment: Visual Studio 2019
  • Runtime: .NET 5
  • OPC Package Release Version [e.g. 1.4.363.107]
  • Component: Opc.Ua.Core
  • Server: [e.g. Reference Server]
  • Client: [e.g. UA Expert]

Additional context
Add any other context about the problem here.

@AlinMoldovean AlinMoldovean added the enhancement API or feature enhancement label Jul 20, 2021
AlinMoldovean pushed a commit that referenced this issue Jul 23, 2021
* Added default switch case for the CreateStore method; reported in issue #1462
@AlinMoldovean
Copy link
Contributor

fixed in #1465

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

No branches or pull requests

2 participants