Server.ServerDiagnostics.SubscriptionDiagnosticsArray is missing in v1.4.370.12

[pc-avatar-7076]

The SubscriptionDiagnosticsArray seems to have gone missing in stack version 1.4.370.12. It is present in stack version 1.4.370.1.

To reproduce:

1. Clone the repo and git checkout 1.4.370.12
2. In Visual Studio 2022 (or 2019) open the UA Reference solution and set the ConsoleReferenceServer as the startup project (you can use global.json to define the appropriate .NET target)
3. Build and execute the ConsoleReferenceServer
4. Connect a UA Client (i.e. UA Expert) and connect to the reference server endpoint (i.e. opc.tcp://<host>:62541/Quickstarts/ReferenceServer) using no security and Anonymous authentication
5. Browse Root.Objects.Server.ServerDiagnostics and observe that the SubscriptionDiagnosticsArray is missing

Notes:

• .NET 6 and NET Core 3.1 targets both reproduced this
• The SubscriptionDiagnosticsArray node reappears after git checkout 1.4.370.1
• Diagnostics are enabled in the Quickstarts.ReferenceServer.Config.xml - <DiagnosticsEnabled>true</DiagnosticsEnabled>
• Opc.Ua.Server.DiagnosticsNodeManager.CreateSubscriptionDiagnostics seems to complete successfully. The 'diagnostic enabled' flag seems to be respected here.

UA Expert capture:

[mregen]

Hi @pc-avatar-7076 , based on this PR some privacy issues were addressed. The access to session diagnostics is only available for sessions using sign and encrypt.
In addition only the sysadmin role can see all subscriptions, otherwise only own subscriptions can be seen.

[pc-avatar-7076]

Thank you @mregen! I noticed that PR but the impact sailed over my head. Indeed, when connecting as an admin I am able to browse/subscribe the SubscriptionDiagnosticArray node:

However, when connecting as a non-admin, I would have expected from your comment to see my own subscription, but that doesn't appear to be my experience (the SubscriptionDiagnosticsArray is missing):

Looking at the PR, specifically the HasApplicationSecureAdminAccess implementation, it looks like it returns false for anonymous and non-admin users. Perhaps this is functioning as intended?

[github-actions]

This issue is stale because it has been open for 30 days with no activity.

[mregen]

@mrsuciu please check if the PR causes this issue.