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

MonitoredItem2.OnReportEvent Ignores Session in ISystemContext During Notification Process #2777

Open
1 of 5 tasks
Filippo-Oliva-ABB opened this issue Sep 27, 2024 · 0 comments · May be fixed by #2779
Open
1 of 5 tasks
Labels
bug A bug was identified and should be fixed. server

Comments

@Filippo-Oliva-ABB
Copy link

Filippo-Oliva-ABB commented Sep 27, 2024

Type of issue

  • Bug
  • Enhancement
  • Compliance
  • Question
  • Help wanted

Current Behavior

The MonitoredNode2.OnReportEvent method does not take into account the Session included in the passed ISystemContext during the notification process. Currently, when a Session is included, the notification occurs on each connected session without checking the specific session passed through the ISystemContext.

Expected Behavior

The MonitoredNode2.OnReportEvent method should only notify the Session specified in the ISystemContext, if any, otherwise all the connected session.

Steps To Reproduce

  1. Single OPC UA server providing a single “NodeManager” that Creates and exposes the whole address space.
  2. Two clients (Client1 and Client2) connected to the server, both subscribed to events of the same node
  3. Override the ConditionRefresh method of the NodeManager trying to notify some event to the subscribed node by calling the ReportEvent method exposed by that node passing as the first parameter a copy of the SystemContext that includes the OperationContext got by the ConditionRefresh method.

Environment

- OS: Windows 10
- Environment: Visual Studio 2019 16.11.33
- Runtime: .NET Framework 4.8
- Nuget Version: 1.5.364.36
- Component: Opc.Ua.Server
- Server: Quickstarts.ConsoleReferenceServer
- Client: UA Expert
Filippo-Oliva-ABB added a commit to Filippo-Oliva-ABB/UA-.NETStandard-PR that referenced this issue Sep 27, 2024
…sion in ISystemContext During Notification Process
@mregen mregen added bug A bug was identified and should be fixed. server labels Sep 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug A bug was identified and should be fixed. server
Projects
None yet
2 participants