mirror of
https://github.com/OPCFoundation/UA-.NETStandard.git
synced 2026-10-20 16:52:07 +08:00
a81d981755
# Description
Pure code-style sweep across the repo — no behavioral changes. Drives
`dotnet format` info-level diagnostics down on a large slice of the
codebase (Stack, Libraries, Tests, Tools).
Six commits, applied bottom-up in safety order so each batch leaves the
build green:
1. **`style: fix whitespace and final-newline formatting (WHITESPACE,
FINALNEWLINE)`**
197 hits across ~30 files; pure mechanical cleanup.
2. **`style: apply dotnet format style fixes (IDE rules)`**
IDE-rule batch via `dotnet format style --severity info` —
`var`/explicit type, simplify names, collection expressions, expression
bodies, unused usings, etc.
3. **`style: apply dotnet format analyzers fixes (RCS blank-line
formatting)`**
Roslynator formatting (RCS0xxx) blank-line and punctuation rules.
4. **`style: convert comments to doc comments and inline local vars
(RCS1181, RCS1124)`**
24 files. RCS1181 promotes `// summary`-style comments above
public/internal members to `/// <summary>` doc comments; RCS1124 inlines
locals used exactly once. Generated `Tools/` files were reverted to keep
the source-generator outputs deterministic.
5. **`docs(agent): correct --diagnostics flag form and verify Roslynator
4.x code fixers`**
Updates `.github/agents/opc-ua-codestyle-enforcer.agent.md` with two
corrections discovered while running this sweep:
- `dotnet format --diagnostics "RULE1 RULE2"` (single quoted,
space-separated) silently fixes nothing on `dotnet format` 10.x — it
parses as one unknown ID and exits 0. The correct form is `--diagnostics
RULE1 --diagnostics RULE2 …`, repeated per rule.
- Roslynator 4.15.0 **does** ship code fixers for
RCS1078/1118/1124/1140/1142/1181/1221/1260 (the previous "no auto-fixer"
list was outdated). RCS1140/1142 default to `Hidden` so need `--severity
info` to surface and fire; RCS1078 requires
`roslynator_empty_string_style = field` (already set in
`.editorconfig`). Only NUnit4002/2046/2010 genuinely lack auto-fixers in
this stack.
6. **`style: format new code from master merge (dotnet format sweep)`**
After merging master into the branch (commit `e3ebc8742`), re-ran the
same three-phase format sweep on the 151 .cs files brought in by the
merge, scoped per project via repeated `--include`. 81 files changed;
same rule families as commits 1–4.
- Hand-fixed `Tests/Opc.Ua.MigrationAnalyzer.Tests/AnalyzerHarness.cs`
for `RCS1174` (redundant `async`/`await` removal) because the auto-fixer
didn't apply for that specific case.
- Reverted files that the formatter would break:
- `Libraries/Opc.Ua.Server/Server/ServerInternalData.cs` — multi-TFM
analysis disagreement injected `<<<<<<<` / `=======` / `>>>>>>>`
markers.
-
`Tools/Opc.Ua.MigrationAnalyzer.Core/{Core/Stack/Client/ChannelBase.cs,
Core/Stack/Types/IUserIdentityTokenHandler.cs,
Client/Session/Session.cs}` — IDE0005 removed `using
Opc.Ua.Security.Certificates` that IS used (the files are 1.5.378
reference fixtures for the analyzer).
- `Tests/Opc.Ua.Gds.Tests/PushTest.cs` — IDE0002 over-simplified
`OpcUa.ObjectTypeIds` to `ObjectTypeIds`, which then resolves to a
conflicting type.
### What this PR does **not** touch
- No behavioral or semantic logic changes.
- No `.editorconfig` rule promotions (rules stay at their current
severities; this is a backlog-burndown pass, not a rule tightening).
- Generated source under `Tools/` is intentionally excluded.
- Higher-risk rule families (`RCS1090`/`CA2007` `ConfigureAwait`,
`RCS1249` null-forgiving cleanup, `IDE0270`/`IDE0059`) are left for
follow-up sweeps because they need per-TFM build verification or
semantic review.
## Related Issues
_No tracking issue — this is a non-functional style sweep and the
agent-doc correction is a follow-up to the same work. Happy to open one
if the maintainers prefer._
## Checklist
- [ ] I have signed the
[CLA](https://opcfoundation.org/license/cla/ContributorLicenseAgreementv1.0.pdf)
and read the
[CONTRIBUTING](https://github.com/OPCFoundation/UA-.NETStandard/blob/master/CONTRIBUTING.md)
doc.
- [x] I have added tests that prove my fix is effective or that my
feature works and increased code coverage. _(N/A — style-only, no
behavioral change)_
- [x] I have added all necessary documentation. _(agent doc updated with
the two corrections)_
- [x] I have verified that my changes do not introduce (new) build or
analyzer warnings. _(verified clean on `net10.0` and `net48`; only
pre-existing `NETSDK1005`/`NETSDK1207` failures on Tools/AOT projects)_
- [ ] I ran **all** tests locally using the **UA.slnx** solution against
at least .net **framework** and .net **10**, and all passed. _(build
verified on net10 and net48 per batch; full test suite not yet run
locally — relying on CI)_
- [ ] I fixed **all** failing and flaky tests in the CI pipelines and
**all** CodeQL warnings.
- [ ] I have addressed **all** PR feedback received.
---------
Co-authored-by: agent <agent@local>
939 lines
37 KiB
C#
939 lines
37 KiB
C#
/* ========================================================================
|
|
* Copyright (c) 2005-2025 The OPC Foundation, Inc. All rights reserved.
|
|
*
|
|
* OPC Foundation MIT License 1.00
|
|
*
|
|
* Permission is hereby granted, free of charge, to any person
|
|
* obtaining a copy of this software and associated documentation
|
|
* files (the "Software"), to deal in the Software without
|
|
* restriction, including without limitation the rights to use,
|
|
* copy, modify, merge, publish, distribute, sublicense, and/or sell
|
|
* copies of the Software, and to permit persons to whom the
|
|
* Software is furnished to do so, subject to the following
|
|
* conditions:
|
|
*
|
|
* The above copyright notice and this permission notice shall be
|
|
* included in all copies or substantial portions of the Software.
|
|
* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
|
|
* EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES
|
|
* OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
|
|
* NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT
|
|
* HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY,
|
|
* WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
|
|
* FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
|
|
* OTHER DEALINGS IN THE SOFTWARE.
|
|
*
|
|
* The complete license agreement can be found here:
|
|
* http://opcfoundation.org/License/MIT/1.00/
|
|
* ======================================================================*/
|
|
|
|
using System;
|
|
using System.Collections.Concurrent;
|
|
using System.Collections.Generic;
|
|
using System.Threading;
|
|
using System.Threading.Channels;
|
|
using System.Threading.Tasks;
|
|
using Microsoft.Extensions.Logging;
|
|
|
|
namespace Opc.Ua.Server
|
|
{
|
|
/// <summary>
|
|
/// Stores the current set of MonitoredItems for a Node.
|
|
/// </summary>
|
|
/// <remarks>
|
|
/// An instance of this object is created the first time a MonitoredItem is
|
|
/// created for any attribute of a Node. The object is deleted when the last
|
|
/// MonitoredItem is deleted.
|
|
/// </remarks>
|
|
public class MonitoredNode2 : IDisposable
|
|
{
|
|
private const int k_defaultChannelCapacity = 4096;
|
|
|
|
/// <summary>
|
|
/// Initializes a new instance of the <see cref="MonitoredNode2"/> class.
|
|
/// </summary>
|
|
/// <param name="nodeManager">The node manager.</param>
|
|
/// <param name="server">The server.</param>
|
|
/// <param name="node">The node.</param>
|
|
/// <param name="enableMultipleEventConsumers">
|
|
/// When <c>true</c>, enables dynamic scaling of consumer tasks based on
|
|
/// the number of event monitored items. The Server node
|
|
/// (<see cref="ObjectIds.Server"/>) always opts in automatically.
|
|
/// </param>
|
|
public MonitoredNode2(
|
|
IAsyncNodeManager nodeManager,
|
|
IServerInternal server,
|
|
NodeState node,
|
|
bool enableMultipleEventConsumers = false)
|
|
: this(nodeManager, server, node, enableMultipleEventConsumers, timeProvider: null)
|
|
{
|
|
}
|
|
|
|
/// <summary>
|
|
/// Initializes a new instance of the <see cref="MonitoredNode2"/> class
|
|
/// with an explicit <see cref="TimeProvider"/>.
|
|
/// </summary>
|
|
/// <param name="nodeManager">The node manager.</param>
|
|
/// <param name="server">The server.</param>
|
|
/// <param name="node">The node.</param>
|
|
/// <param name="enableMultipleEventConsumers">
|
|
/// When <c>true</c>, enables dynamic scaling of consumer tasks.
|
|
/// </param>
|
|
/// <param name="timeProvider">
|
|
/// Optional <see cref="TimeProvider"/> used for the operation-context
|
|
/// cache lifetime and for wall-clock <c>SourceTimestamp</c> /
|
|
/// <c>ServerTimestamp</c> values produced inside this node. When
|
|
/// <c>null</c>, the time provider exposed by the server (via
|
|
/// <see cref="ITimeProviderProvider"/>) is used, falling back to
|
|
/// <see cref="TimeProvider.System"/>.
|
|
/// </param>
|
|
public MonitoredNode2(
|
|
IAsyncNodeManager nodeManager,
|
|
IServerInternal server,
|
|
NodeState node,
|
|
bool enableMultipleEventConsumers,
|
|
TimeProvider? timeProvider)
|
|
{
|
|
NodeManager = nodeManager ?? throw new ArgumentNullException(nameof(nodeManager));
|
|
m_server = server ?? throw new ArgumentNullException(nameof(server));
|
|
Node = node ?? throw new ArgumentNullException(nameof(node));
|
|
m_timeProvider = timeProvider
|
|
?? (server as ITimeProviderProvider)?.TimeProvider
|
|
?? TimeProvider.System;
|
|
m_logger = server.Telemetry?.CreateLogger<MonitoredNode2>();
|
|
m_useMultipleConsumers = enableMultipleEventConsumers || node.NodeId == ObjectIds.Server;
|
|
m_channel = Channel.CreateBounded<INodeNotification>(new BoundedChannelOptions(k_defaultChannelCapacity)
|
|
{
|
|
SingleReader = !m_useMultipleConsumers,
|
|
FullMode = BoundedChannelFullMode.Wait,
|
|
AllowSynchronousContinuations = false
|
|
});
|
|
m_consumerCts = new CancellationTokenSource();
|
|
m_consumerTask = Task.Run(() => ProcessChannelAsync(m_consumerCts.Token));
|
|
|
|
if (m_useMultipleConsumers)
|
|
{
|
|
m_additionalConsumers = [];
|
|
}
|
|
}
|
|
|
|
/// <summary>
|
|
/// Gets or sets the NodeManager which the MonitoredNode belongs to.
|
|
/// </summary>
|
|
public IAsyncNodeManager NodeManager { get; set; }
|
|
|
|
/// <summary>
|
|
/// Gets or sets the Node being monitored.
|
|
/// </summary>
|
|
public NodeState Node { get; set; }
|
|
|
|
/// <summary>
|
|
/// Gets the current list of data change MonitoredItems.
|
|
/// </summary>
|
|
public ConcurrentDictionary<uint, IDataChangeMonitoredItem2> DataChangeMonitoredItems { get; } = new();
|
|
|
|
/// <summary>
|
|
/// Gets the current list of event MonitoredItems.
|
|
/// </summary>
|
|
public ConcurrentDictionary<uint, IEventMonitoredItem> EventMonitoredItems { get; } = new();
|
|
|
|
/// <summary>
|
|
/// Gets a value indicating whether this instance has monitored items.
|
|
/// </summary>
|
|
/// <value>
|
|
/// <c>true</c> if this instance has monitored items; otherwise, <c>false</c>.
|
|
/// </value>
|
|
public bool HasMonitoredItems
|
|
{
|
|
get
|
|
{
|
|
if (DataChangeMonitoredItems != null && !DataChangeMonitoredItems.IsEmpty)
|
|
{
|
|
return true;
|
|
}
|
|
|
|
if (EventMonitoredItems != null && !EventMonitoredItems.IsEmpty)
|
|
{
|
|
return true;
|
|
}
|
|
|
|
return false;
|
|
}
|
|
}
|
|
|
|
/// <summary>
|
|
/// Adds the specified data change monitored item.
|
|
/// </summary>
|
|
/// <param name="datachangeItem">The monitored item.</param>
|
|
public void Add(IDataChangeMonitoredItem2 datachangeItem)
|
|
{
|
|
bool wasEmpty = DataChangeMonitoredItems.IsEmpty;
|
|
DataChangeMonitoredItems.TryAdd(datachangeItem.Id, datachangeItem);
|
|
|
|
Node.OnStateChanged = OnMonitoredNodeChanged;
|
|
|
|
// Subscribe to namespace default permission changes when the first item is added.
|
|
if (wasEmpty && m_server.ConfigurationNodeManager != null)
|
|
{
|
|
m_server.ConfigurationNodeManager.DefaultPermissionsChanged += OnDefaultPermissionsChanged;
|
|
}
|
|
}
|
|
|
|
/// <summary>
|
|
/// Removes the specified data change monitored item.
|
|
/// </summary>
|
|
/// <param name="datachangeItem">The monitored item.</param>
|
|
public void Remove(IDataChangeMonitoredItem2 datachangeItem)
|
|
{
|
|
if (DataChangeMonitoredItems.TryRemove(datachangeItem.Id, out _))
|
|
{
|
|
// Remove the cached context for the monitored item
|
|
m_contextCache.TryRemove(datachangeItem.Id, out _);
|
|
m_permissionCache.TryRemove(datachangeItem.Id, out _);
|
|
}
|
|
|
|
if (DataChangeMonitoredItems.IsEmpty)
|
|
{
|
|
Node.OnStateChanged = null;
|
|
|
|
// Unsubscribe from namespace default permission changes when the last item is removed.
|
|
m_server.ConfigurationNodeManager?.DefaultPermissionsChanged -= OnDefaultPermissionsChanged;
|
|
}
|
|
}
|
|
|
|
/// <summary>
|
|
/// Adds the specified event monitored item.
|
|
/// </summary>
|
|
/// <param name="eventItem">The monitored item.</param>
|
|
public void Add(IEventMonitoredItem eventItem)
|
|
{
|
|
EventMonitoredItems.TryAdd(eventItem.Id, eventItem);
|
|
|
|
Node.OnReportEvent = OnReportEvent;
|
|
|
|
// Scale up: add a consumer task for each new event MI beyond the first.
|
|
if (m_useMultipleConsumers && m_additionalConsumers != null)
|
|
{
|
|
lock (m_additionalConsumersLock)
|
|
{
|
|
// The primary consumer task always runs; add additional ones
|
|
// so total consumers = EventMonitoredItems.Count.
|
|
int totalDesired = EventMonitoredItems.Count;
|
|
if (totalDesired > m_additionalConsumers.Count + 1)
|
|
{
|
|
AddConsumer();
|
|
}
|
|
}
|
|
}
|
|
}
|
|
|
|
/// <summary>
|
|
/// Removes the specified event monitored item.
|
|
/// </summary>
|
|
/// <param name="eventItem">The monitored item.</param>
|
|
public void Remove(IEventMonitoredItem eventItem)
|
|
{
|
|
EventMonitoredItems.TryRemove(eventItem.Id, out _);
|
|
DropEventPermissionCacheEntries(eventItem.Id);
|
|
|
|
if (EventMonitoredItems.IsEmpty)
|
|
{
|
|
Node.OnReportEvent = null;
|
|
}
|
|
|
|
// Scale down: remove a consumer task when MIs decrease (keep at least 1 total = primary).
|
|
if (m_useMultipleConsumers && m_additionalConsumers != null)
|
|
{
|
|
lock (m_additionalConsumersLock)
|
|
{
|
|
// Total consumers = 1 (primary) + m_additionalConsumers.Count
|
|
int totalDesired = Math.Max(1, EventMonitoredItems.Count);
|
|
while (m_additionalConsumers.Count + 1 > totalDesired)
|
|
{
|
|
RemoveLastConsumer();
|
|
}
|
|
}
|
|
}
|
|
}
|
|
|
|
/// <summary>
|
|
/// Called when a Node produces an event.
|
|
/// </summary>
|
|
/// <param name="context">The system context.</param>
|
|
/// <param name="node">The affected node.</param>
|
|
/// <param name="e">The event.</param>
|
|
public void OnReportEvent(ISystemContext context, NodeState node, IFilterTarget e)
|
|
{
|
|
if (m_disposed)
|
|
{
|
|
return;
|
|
}
|
|
|
|
IFilterTarget eventTarget = e;
|
|
// Build snapshot so the original event state is preserved when the consumer processes it.
|
|
if (e is NodeState eventState)
|
|
{
|
|
eventTarget = (IFilterTarget)eventState.Clone();
|
|
}
|
|
|
|
var notification = new EventSnapshot
|
|
{
|
|
Context = context,
|
|
EventTargetSnapshot = eventTarget
|
|
};
|
|
|
|
try
|
|
{
|
|
m_channel.Writer.WriteAsync(notification).AsTask().GetAwaiter().GetResult();
|
|
}
|
|
catch (ChannelClosedException)
|
|
{
|
|
// The channel was completed during shutdown/disposal.
|
|
}
|
|
}
|
|
|
|
/// <summary>
|
|
/// Called when the state of a Node changes.
|
|
/// </summary>
|
|
/// <param name="context">The system context.</param>
|
|
/// <param name="node">The affected node.</param>
|
|
/// <param name="changes">The mask indicating what changes have occurred.</param>
|
|
public void OnMonitoredNodeChanged(
|
|
ISystemContext context,
|
|
NodeState node,
|
|
NodeStateChangeMasks changes)
|
|
{
|
|
if (m_disposed)
|
|
{
|
|
return;
|
|
}
|
|
|
|
if (DataChangeMonitoredItems == null || DataChangeMonitoredItems.IsEmpty)
|
|
{
|
|
return;
|
|
}
|
|
|
|
// Collect the distinct attribute IDs being monitored so we can pre-read them.
|
|
// The raw value is read once without any index range or data encoding; each
|
|
// monitored item applies its own range/encoding in ProcessDataChangeSnapshotAsync.
|
|
var attributeIds = new HashSet<uint>();
|
|
foreach (KeyValuePair<uint, IDataChangeMonitoredItem2> kvp in DataChangeMonitoredItems)
|
|
{
|
|
IDataChangeMonitoredItem2 item = kvp.Value;
|
|
bool isValueAttribute = item.AttributeId == Attributes.Value;
|
|
if (isValueAttribute && (changes & NodeStateChangeMasks.Value) == 0)
|
|
{
|
|
continue;
|
|
}
|
|
if (!isValueAttribute && (changes & NodeStateChangeMasks.NonValue) == 0)
|
|
{
|
|
continue;
|
|
}
|
|
attributeIds.Add(item.AttributeId);
|
|
}
|
|
|
|
var attributeSnapshots = new Dictionary<uint, DataValue>(attributeIds.Count);
|
|
|
|
foreach (uint attributeId in attributeIds)
|
|
{
|
|
var dataValue = new DataValue(
|
|
default,
|
|
StatusCodes.Good,
|
|
m_timeProvider.GetUtcNow().UtcDateTime,
|
|
DateTime.MinValue);
|
|
|
|
|
|
(_, attributeSnapshots[attributeId]) = node.ReadAttributeAsync(
|
|
context,
|
|
attributeId,
|
|
default,
|
|
QualifiedName.Null,
|
|
dataValue).AsTask().GetAwaiter().GetResult();
|
|
}
|
|
|
|
var notification = new DataChangeSnapshot
|
|
{
|
|
Context = context,
|
|
NodeId = node.NodeId,
|
|
Changes = changes,
|
|
AttributeSnapshots = attributeSnapshots
|
|
};
|
|
|
|
try
|
|
{
|
|
m_channel.Writer.WriteAsync(notification).AsTask().GetAwaiter().GetResult();
|
|
}
|
|
catch (ChannelClosedException)
|
|
{
|
|
// The channel was completed during shutdown/disposal.
|
|
}
|
|
}
|
|
|
|
/// <summary>
|
|
/// Consumer loop that processes notifications from the channel.
|
|
/// </summary>
|
|
private async Task ProcessChannelAsync(CancellationToken cancellationToken)
|
|
{
|
|
try
|
|
{
|
|
await foreach (INodeNotification notification in m_channel.Reader.ReadAllAsync(cancellationToken).ConfigureAwait(false))
|
|
{
|
|
try
|
|
{
|
|
if (notification is EventSnapshot eventSnapshot)
|
|
{
|
|
await ProcessEventSnapshotAsync(eventSnapshot, cancellationToken).ConfigureAwait(false);
|
|
}
|
|
else if (notification is DataChangeSnapshot dataChangeSnapshot)
|
|
{
|
|
await ProcessDataChangeSnapshotAsync(dataChangeSnapshot, cancellationToken).ConfigureAwait(false);
|
|
}
|
|
}
|
|
catch (OperationCanceledException) when (cancellationToken.IsCancellationRequested)
|
|
{
|
|
break;
|
|
}
|
|
catch (Exception ex)
|
|
{
|
|
m_logger?.LogWarning(ex, "MonitoredNode2 consumer encountered an error processing a notification.");
|
|
}
|
|
}
|
|
}
|
|
catch (OperationCanceledException)
|
|
{
|
|
// shutdown
|
|
}
|
|
catch (Exception ex)
|
|
{
|
|
m_logger?.LogError(ex, "MonitoredNode2 consumer terminated unexpectedly.");
|
|
}
|
|
}
|
|
|
|
/// <summary>
|
|
/// Processes an <see cref="EventSnapshot"/> from the channel.
|
|
/// </summary>
|
|
private async Task ProcessEventSnapshotAsync(EventSnapshot snapshot, CancellationToken cancellationToken)
|
|
{
|
|
// Extract EventTypeId + SourceNodeId once so the per-item
|
|
// permission cache can be keyed by them. The asynchronous
|
|
// permission lookup is the dominant cost; caching the verdict
|
|
// per (item, eventType, sourceNode) avoids two role-validation
|
|
// calls per delivered event.
|
|
(NodeId eventTypeId, NodeId sourceNodeId) = ExtractEventIdentity(snapshot.EventTargetSnapshot);
|
|
|
|
foreach (KeyValuePair<uint, IEventMonitoredItem> kvp in EventMonitoredItems)
|
|
{
|
|
IEventMonitoredItem monitoredItem = kvp.Value;
|
|
IFilterTarget e = snapshot.EventTargetSnapshot;
|
|
|
|
if (e is AuditEventState || (e is InstanceStateSnapshot sn && sn.Handle is AuditEventState))
|
|
{
|
|
if (!m_server.Auditing)
|
|
{
|
|
continue;
|
|
}
|
|
if (monitoredItem?.Session?.EndpointDescription?.SecurityMode !=
|
|
MessageSecurityMode.SignAndEncrypt &&
|
|
monitoredItem?.Session?.EndpointDescription?.TransportProfileUri !=
|
|
Profiles.HttpsBinaryTransport)
|
|
{
|
|
continue;
|
|
}
|
|
}
|
|
|
|
ServiceResult validationResult = await GetOrAddEventPermissionAsync(
|
|
monitoredItem!,
|
|
e,
|
|
eventTypeId,
|
|
sourceNodeId,
|
|
cancellationToken).ConfigureAwait(false);
|
|
|
|
if (ServiceResult.IsBad(validationResult))
|
|
{
|
|
continue;
|
|
}
|
|
|
|
if (snapshot.Context is ISessionSystemContext sessionContext &&
|
|
sessionContext.SessionId is { IsNull: false } contextSessionId &&
|
|
monitoredItem?.Session?.Id is { IsNull: false } monitoredItemSessionId &&
|
|
!monitoredItemSessionId.Equals(contextSessionId))
|
|
{
|
|
continue;
|
|
}
|
|
|
|
monitoredItem?.QueueEvent(e);
|
|
}
|
|
}
|
|
|
|
/// <summary>
|
|
/// Extracts the <c>EventTypeId</c> and <c>SourceNode</c> from an
|
|
/// event payload. Returns default <see cref="NodeId"/> values when
|
|
/// the payload is not a <see cref="BaseEventState"/> (or wrapped
|
|
/// in an <see cref="InstanceStateSnapshot"/> over one), in which
|
|
/// case the result is not cached.
|
|
/// </summary>
|
|
private static (NodeId EventTypeId, NodeId SourceNodeId) ExtractEventIdentity(IFilterTarget filterTarget)
|
|
{
|
|
var baseEventState = filterTarget as BaseEventState;
|
|
if (baseEventState == null && filterTarget is InstanceStateSnapshot snapshot)
|
|
{
|
|
baseEventState = snapshot.Handle as BaseEventState;
|
|
}
|
|
|
|
if (baseEventState == null)
|
|
{
|
|
return (default, default);
|
|
}
|
|
|
|
NodeId eventTypeId = baseEventState.EventType?.Value ?? default;
|
|
NodeId sourceNodeId = baseEventState.SourceNode?.Value ?? default;
|
|
return (eventTypeId, sourceNodeId);
|
|
}
|
|
|
|
/// <summary>
|
|
/// Returns the cached permission verdict for the
|
|
/// (item, eventType, sourceNode) tuple, computing and caching it
|
|
/// on miss. Falls back to an uncached lookup when either NodeId
|
|
/// is null so events that don't carry the standard identity
|
|
/// fields still go through the per-event check.
|
|
/// </summary>
|
|
private async ValueTask<ServiceResult> GetOrAddEventPermissionAsync(
|
|
IEventMonitoredItem monitoredItem,
|
|
IFilterTarget filterTarget,
|
|
NodeId eventTypeId,
|
|
NodeId sourceNodeId,
|
|
CancellationToken cancellationToken)
|
|
{
|
|
if (eventTypeId.IsNull || sourceNodeId.IsNull)
|
|
{
|
|
// Not a cacheable identity — defer to the existing path.
|
|
return await NodeManager.ValidateEventRolePermissionsAsync(
|
|
monitoredItem,
|
|
filterTarget,
|
|
cancellationToken).ConfigureAwait(false);
|
|
}
|
|
|
|
var key = new EventPermissionCacheKey(monitoredItem.Id, eventTypeId, sourceNodeId);
|
|
if (m_eventPermissionCache.TryGetValue(key, out ServiceResult? cached) && cached != null)
|
|
{
|
|
return cached;
|
|
}
|
|
|
|
ServiceResult result = await NodeManager.ValidateEventRolePermissionsAsync(
|
|
monitoredItem,
|
|
filterTarget,
|
|
cancellationToken).ConfigureAwait(false);
|
|
|
|
m_eventPermissionCache[key] = result;
|
|
return result;
|
|
}
|
|
|
|
/// <summary>
|
|
/// Processes a <see cref="DataChangeSnapshot"/> from the channel.
|
|
/// </summary>
|
|
private async Task ProcessDataChangeSnapshotAsync(DataChangeSnapshot snapshot, CancellationToken cancellationToken)
|
|
{
|
|
if (DataChangeMonitoredItems == null)
|
|
{
|
|
return;
|
|
}
|
|
|
|
// If RolePermissions or UserRolePermissions have changed, invalidate the permission cache.
|
|
if ((snapshot.Changes & NodeStateChangeMasks.RolePermissions) != 0)
|
|
{
|
|
m_permissionCache.Clear();
|
|
m_eventPermissionCache.Clear();
|
|
}
|
|
|
|
foreach (KeyValuePair<uint, IDataChangeMonitoredItem2> kvp in DataChangeMonitoredItems)
|
|
{
|
|
IDataChangeMonitoredItem2 monitoredItem = kvp.Value;
|
|
OperationContext operationContext;
|
|
ISystemContext contextToUse;
|
|
|
|
if (snapshot.Context is ServerSystemContext serverContext)
|
|
{
|
|
ServerSystemContext serverSystemContextToUse = GetOrCreateContext(serverContext, monitoredItem);
|
|
operationContext = serverSystemContextToUse.OperationContext!;
|
|
contextToUse = serverSystemContextToUse;
|
|
}
|
|
else
|
|
{
|
|
operationContext = new OperationContext(monitoredItem);
|
|
contextToUse = snapshot.Context;
|
|
}
|
|
|
|
if (monitoredItem.AttributeId == Attributes.Value &&
|
|
(snapshot.Changes & NodeStateChangeMasks.Value) != 0)
|
|
{
|
|
if (!m_permissionCache.TryGetValue(monitoredItem.Id, out ServiceResult? validationResult))
|
|
{
|
|
validationResult = await NodeManager.ValidateRolePermissionsAsync(
|
|
operationContext,
|
|
snapshot.NodeId,
|
|
PermissionType.Read,
|
|
cancellationToken).ConfigureAwait(false);
|
|
m_permissionCache[monitoredItem.Id] = validationResult;
|
|
}
|
|
|
|
if (ServiceResult.IsBad(validationResult))
|
|
{
|
|
continue;
|
|
}
|
|
|
|
if (snapshot.AttributeSnapshots.TryGetValue(monitoredItem.AttributeId, out DataValue snapshotValue))
|
|
{
|
|
DataValue valueToQueue = ApplyRangeAndEncoding(contextToUse, monitoredItem, snapshotValue);
|
|
monitoredItem.QueueValue(valueToQueue, valueToQueue.StatusCode);
|
|
}
|
|
|
|
continue;
|
|
}
|
|
|
|
if (monitoredItem.AttributeId != Attributes.Value &&
|
|
(snapshot.Changes & NodeStateChangeMasks.NonValue) != 0)
|
|
{
|
|
if (snapshot.AttributeSnapshots.TryGetValue(monitoredItem.AttributeId, out DataValue snapshotValue))
|
|
{
|
|
monitoredItem.QueueValue(snapshotValue, ServiceResult.Good);
|
|
}
|
|
}
|
|
}
|
|
}
|
|
|
|
/// <summary>
|
|
/// Reads the value of an attribute and reports it to the MonitoredItem.
|
|
/// </summary>
|
|
public async ValueTask QueueValueAsync(
|
|
ISystemContext context,
|
|
NodeState node,
|
|
IDataChangeMonitoredItem2 monitoredItem,
|
|
CancellationToken cancellationToken = default)
|
|
{
|
|
var value = new DataValue(
|
|
Variant.Null,
|
|
StatusCodes.Good,
|
|
DateTime.MinValue,
|
|
m_timeProvider.GetUtcNow().UtcDateTime);
|
|
(ServiceResult error, value) = await node.ReadAttributeAsync(
|
|
context,
|
|
monitoredItem.AttributeId,
|
|
monitoredItem.IndexRange,
|
|
monitoredItem.DataEncoding,
|
|
value,
|
|
cancellationToken).ConfigureAwait(false);
|
|
|
|
if (ServiceResult.IsBad(error))
|
|
{
|
|
value = default;
|
|
}
|
|
|
|
monitoredItem.QueueValue(value, error);
|
|
}
|
|
|
|
/// <summary>
|
|
/// Applies the monitored item's <see cref="IDataChangeMonitoredItem2.IndexRange"/> and
|
|
/// <see cref="IDataChangeMonitoredItem2.DataEncoding"/> to the raw snapshot value.
|
|
/// The snapshot <see cref="DataValue"/> is shared across items, so the <see cref="Variant"/>
|
|
/// is cloned before transformation to avoid mutating the shared copy.
|
|
/// </summary>
|
|
private static DataValue ApplyRangeAndEncoding(
|
|
ISystemContext context,
|
|
IDataChangeMonitoredItem2 monitoredItem,
|
|
in DataValue snapshotValue)
|
|
{
|
|
// Clone the Variant so we do not mutate the shared snapshot value.
|
|
Variant value = snapshotValue.WrappedValue.Copy();
|
|
|
|
ServiceResult applyResult = BaseVariableState.ApplyIndexRangeAndDataEncoding(
|
|
context,
|
|
monitoredItem.IndexRange,
|
|
monitoredItem.DataEncoding,
|
|
ref value);
|
|
|
|
if (ServiceResult.IsBad(applyResult))
|
|
{
|
|
return new DataValue(applyResult.StatusCode);
|
|
}
|
|
|
|
return new DataValue(value, snapshotValue.StatusCode, snapshotValue.SourceTimestamp, snapshotValue.ServerTimestamp);
|
|
}
|
|
|
|
/// <summary>
|
|
/// Gets or creates a cached context for the monitored item.
|
|
/// </summary>
|
|
/// <param name="context">The system context.</param>
|
|
/// <param name="monitoredItem">The monitored item.</param>
|
|
/// <returns>The cached or newly created context.</returns>
|
|
private ServerSystemContext GetOrCreateContext(
|
|
ServerSystemContext context,
|
|
IDataChangeMonitoredItem2 monitoredItem)
|
|
{
|
|
uint monitoredItemId = monitoredItem.Id;
|
|
long currentTimestamp = m_timeProvider.GetTimestamp();
|
|
OperationContext operationContext;
|
|
|
|
// Check if the context already exists in the cache
|
|
if (m_contextCache.TryGetValue(
|
|
monitoredItemId,
|
|
out (ServerSystemContext Context, long CreatedAtTimestamp) cachedEntry))
|
|
{
|
|
// Refresh context if the owning session changed (e.g. after subscription transfer)
|
|
// or if the cache entry has expired.
|
|
// Note: identity-based invalidation is handled proactively by
|
|
// InvalidatePermissionCacheForSession when ActivateSession changes the identity.
|
|
if (cachedEntry.Context.OperationContext!.Session != monitoredItem.Session ||
|
|
m_timeProvider.GetElapsedTime(cachedEntry.CreatedAtTimestamp) > m_cacheLifetime)
|
|
{
|
|
operationContext = new OperationContext(monitoredItem);
|
|
|
|
ServerSystemContext updatedContext = context.Copy(
|
|
operationContext);
|
|
m_contextCache[monitoredItemId] = (updatedContext, currentTimestamp);
|
|
|
|
// Invalidate the permission cache since the session context has changed.
|
|
m_permissionCache.TryRemove(monitoredItemId, out _);
|
|
|
|
return updatedContext;
|
|
}
|
|
|
|
return cachedEntry.Context;
|
|
}
|
|
|
|
// Create a new context and add it to the cache
|
|
operationContext = new OperationContext(monitoredItem);
|
|
ServerSystemContext newContext = context.Copy(operationContext);
|
|
m_contextCache.TryAdd(monitoredItemId, (newContext, currentTimestamp));
|
|
|
|
return newContext;
|
|
}
|
|
|
|
/// <summary>
|
|
/// Invalidates the permission and context caches for all monitored items belonging
|
|
/// to the specified session. Call this when the session's user identity changes so
|
|
/// that role permissions are re-evaluated on the next data change notification.
|
|
/// </summary>
|
|
/// <param name="sessionId">The NodeId of the session whose identity has changed.</param>
|
|
public void InvalidatePermissionCacheForSession(NodeId sessionId)
|
|
{
|
|
foreach (KeyValuePair<uint, IDataChangeMonitoredItem2> kvp in DataChangeMonitoredItems)
|
|
{
|
|
IDataChangeMonitoredItem2 monitoredItem = kvp.Value;
|
|
|
|
if (monitoredItem?.Session?.Id.Equals(sessionId) == true)
|
|
{
|
|
uint id = monitoredItem.Id;
|
|
m_permissionCache.TryRemove(id, out _);
|
|
m_contextCache.TryRemove(id, out _);
|
|
}
|
|
}
|
|
|
|
foreach (KeyValuePair<uint, IEventMonitoredItem> kvp in EventMonitoredItems)
|
|
{
|
|
IEventMonitoredItem monitoredItem = kvp.Value;
|
|
if (monitoredItem?.Session?.Id.Equals(sessionId) == true)
|
|
{
|
|
DropEventPermissionCacheEntries(monitoredItem.Id);
|
|
}
|
|
}
|
|
}
|
|
|
|
/// <summary>
|
|
/// Called when the namespace default permissions (<c>DefaultRolePermissions</c> or
|
|
/// <c>DefaultUserRolePermissions</c>) change. Invalidates the entire permission cache
|
|
/// so that all entries are re-validated on the next value change.
|
|
/// </summary>
|
|
private void OnDefaultPermissionsChanged(object? sender, EventArgs e)
|
|
{
|
|
m_permissionCache.Clear();
|
|
m_eventPermissionCache.Clear();
|
|
}
|
|
|
|
/// <summary>
|
|
/// Drops every <see cref="EventPermissionCacheKey"/> entry that
|
|
/// belongs to the specified monitored item id.
|
|
/// </summary>
|
|
private void DropEventPermissionCacheEntries(uint monitoredItemId)
|
|
{
|
|
foreach (KeyValuePair<EventPermissionCacheKey, ServiceResult> entry in m_eventPermissionCache)
|
|
{
|
|
if (entry.Key.MonitoredItemId == monitoredItemId)
|
|
{
|
|
m_eventPermissionCache.TryRemove(entry.Key, out _);
|
|
}
|
|
}
|
|
}
|
|
|
|
/// <summary>
|
|
/// Adds a new consumer task to the pool for the regular channel.
|
|
/// Must be called while holding the <see cref="m_additionalConsumersLock"/> lock.
|
|
/// </summary>
|
|
private void AddConsumer()
|
|
{
|
|
var cts = CancellationTokenSource.CreateLinkedTokenSource(m_consumerCts.Token);
|
|
var task = Task.Run(() => ProcessChannelAsync(cts.Token));
|
|
m_additionalConsumers!.Add(new ConsumerEntry(task, cts));
|
|
}
|
|
|
|
/// <summary>
|
|
/// Removes the last additional consumer task from the pool.
|
|
/// Must be called while holding the <see cref="m_additionalConsumersLock"/> lock.
|
|
/// </summary>
|
|
private void RemoveLastConsumer()
|
|
{
|
|
if (m_additionalConsumers!.Count == 0)
|
|
{
|
|
return;
|
|
}
|
|
|
|
int lastIndex = m_additionalConsumers.Count - 1;
|
|
ConsumerEntry entry = m_additionalConsumers[lastIndex];
|
|
m_additionalConsumers.RemoveAt(lastIndex);
|
|
entry.Cts.Cancel();
|
|
entry.Cts.Dispose();
|
|
}
|
|
|
|
/// <summary>
|
|
/// Represents a single consumer task and its associated cancellation token.
|
|
/// </summary>
|
|
private readonly struct ConsumerEntry
|
|
{
|
|
public ConsumerEntry(Task task, CancellationTokenSource cts)
|
|
{
|
|
Task = task;
|
|
Cts = cts;
|
|
}
|
|
|
|
public Task Task { get; }
|
|
public CancellationTokenSource Cts { get; }
|
|
}
|
|
|
|
private readonly ConcurrentDictionary<uint, (ServerSystemContext Context, long CreatedAtTimestamp)> m_contextCache =
|
|
new();
|
|
|
|
private readonly ConcurrentDictionary<uint, ServiceResult> m_permissionCache =
|
|
new();
|
|
|
|
private readonly ConcurrentDictionary<EventPermissionCacheKey, ServiceResult> m_eventPermissionCache =
|
|
new();
|
|
|
|
private readonly TimeSpan m_cacheLifetime = TimeSpan.FromMinutes(5);
|
|
|
|
private readonly IServerInternal m_server;
|
|
private readonly TimeProvider m_timeProvider;
|
|
private readonly ILogger? m_logger;
|
|
private readonly Channel<INodeNotification> m_channel;
|
|
private readonly CancellationTokenSource m_consumerCts;
|
|
private readonly Task m_consumerTask;
|
|
private readonly bool m_useMultipleConsumers;
|
|
private readonly List<ConsumerEntry>? m_additionalConsumers;
|
|
private readonly Lock m_additionalConsumersLock = new();
|
|
private bool m_disposed;
|
|
|
|
/// <inheritdoc/>
|
|
public void Dispose()
|
|
{
|
|
Dispose(true);
|
|
GC.SuppressFinalize(this);
|
|
}
|
|
|
|
/// <summary>
|
|
/// An overrideable version of the Dispose.
|
|
/// </summary>
|
|
protected virtual void Dispose(bool disposing)
|
|
{
|
|
if (m_disposed)
|
|
{
|
|
return;
|
|
}
|
|
m_disposed = true;
|
|
|
|
if (disposing)
|
|
{
|
|
// Complete the writer; consumers drain remaining items and exit normally.
|
|
m_channel.Writer.TryComplete();
|
|
|
|
// Wait for additional consumer tasks to finish.
|
|
if (m_additionalConsumers != null)
|
|
{
|
|
ConsumerEntry[] entries;
|
|
lock (m_additionalConsumersLock)
|
|
{
|
|
entries = [.. m_additionalConsumers];
|
|
m_additionalConsumers.Clear();
|
|
}
|
|
|
|
Task[] tasks = Array.ConvertAll(entries, e => e.Task);
|
|
|
|
try
|
|
{
|
|
// Bound the wait — do not block indefinitely if consumers are stuck.
|
|
bool completed = Task.WaitAll(tasks, TimeSpan.FromSeconds(5));
|
|
|
|
if (!completed)
|
|
{
|
|
m_logger?.LogWarning(
|
|
"MonitoredNode2 additional consumers did not drain within 5 s; cancelling forcibly.");
|
|
|
|
foreach (ConsumerEntry entry in entries)
|
|
{
|
|
entry.Cts.Cancel();
|
|
}
|
|
|
|
try
|
|
{
|
|
Task.WaitAll(tasks);
|
|
}
|
|
catch { }
|
|
}
|
|
}
|
|
catch (Exception ex)
|
|
{
|
|
m_logger?.LogWarning(ex, "MonitoredNode2 additional consumers faulted during shutdown.");
|
|
}
|
|
finally
|
|
{
|
|
foreach (ConsumerEntry entry in entries)
|
|
{
|
|
entry.Cts.Dispose();
|
|
}
|
|
}
|
|
}
|
|
|
|
if (m_consumerTask != null)
|
|
{
|
|
try
|
|
{
|
|
// Bound the wait — do not block indefinitely if the consumer is stuck.
|
|
bool completed = m_consumerTask
|
|
.Wait(TimeSpan.FromSeconds(5));
|
|
|
|
if (!completed)
|
|
{
|
|
m_logger?.LogWarning(
|
|
"MonitoredNode2 consumer did not drain within 5 s; cancelling forcibly.");
|
|
m_consumerCts.Cancel();
|
|
|
|
try
|
|
{
|
|
m_consumerTask.GetAwaiter().GetResult();
|
|
}
|
|
catch
|
|
{
|
|
}
|
|
}
|
|
}
|
|
catch (Exception ex)
|
|
{
|
|
m_logger?.LogWarning(ex, "MonitoredNode2 consumer faulted during shutdown.");
|
|
}
|
|
}
|
|
|
|
// Cancel and dispose only after the consumer has finished.
|
|
m_consumerCts?.Cancel();
|
|
m_consumerCts?.Dispose();
|
|
}
|
|
}
|
|
}
|
|
}
|