Files
UA-.NETStandard/docs/SessionBindings.md
5899354441 Clarify documentation, migration guidance, and SDK selection (#4823)
# Description

Remove stock phrasing and correct misleading or inconsistent technical
documentation. The initial two passes covered 367 Markdown files,
followed by an interactive review of 169 editorial flags. The U5
interpretation originally deferred in that review has since received the
upstream Mantis answer recorded by #4843; remaining server/spec
follow-ups stay explicit in the CTT log.

- Correct package responsibilities, migration steps, API names,
ownership, protocol descriptions and links. Keep bundled migration
guides synchronized. Preserve historical conclusions as historical
records with separate re-check notes.
- Provide at least two rendered TOC levels throughout `docs/`. Convert
appropriate architecture figures to Mermaid without changing code
examples, transcripts or historical quotations.
- Address the October 19 feedback with concrete setup/migration
instructions, corrected certificate paths and PubSub names, shorter
repeated caveats, consistent XML comments, and a full review of the
Concepts guide. Retain #4839's API corrections.
- Integrate #4841's runtime/performance guidance and review
incoming/recent prose. Preserve dated measurements rather than claiming
new benchmark runs; clarify allocation ratios, startup-only processor
settings, JIT warm-up and the benchmark commit's scope. Missing raw
measurement artifacts and repeat ranges remain tracked by #4842.
- Merge master through `e8c5d800a` in `32c6394b7`. Resolve the CTT log
conflict using the updated Mantis states and OPC answers, retaining
historical framing and re-checks for C28/C32/C43.
- Address the latest naming/index feedback in `adb5bf693`: rename the
guides to `docs/WotCon.md` and `docs/WotNodeSetConversion.md`; move the
provider and DataType-declaration contracts into the conversion guide
and remove `docs/WoT.md`. Rename `samples/DI` to `samples/Di` and
`samples/Isa95` to `samples/ISA95`. Add content-index READMEs to Di,
FluentApi, ISA95, Lds and PubSub; update solution entries, project
references, Docker/CI paths, test fixture paths and documentation links
with exact casing.

**WoT guide scope.** This branch retains the detailed conversion guide
that #4758 had replaced with an anchor stub. Conversion limits, verified
document sets, preservation rules and provider ownership now live
together in `WotNodeSetConversion.md`; binding/runtime guidance remains
in `WotBindings.md` and `WotCon.md`. The design-rationale/test-ledger
prose was shortened, the oversized argument-table cell moved into its
section, and existing anchors preserved.

**SDK policy is a separate, explicit user-directed change.**
`global.json` sets a stable minimum of `10.0.400`, `rollForward:
latestMajor`, and `allowPrerelease: false`. Newer major SDKs are
permitted; CI's explicit SDK pins are unchanged. README, GettingStarted,
DeveloperGuide and Recycling instructions reflect the possible analyzer
differences. The image-contract regression fix accepts compatible stable
SDK images instead of requiring exact equality with the floor.

The latest four rename/index threads are resolved after publication.
Three older threads remain open: universal obsolete-removal timing,
unconditional nonce-strength wording, and a blanket Vision NativeAOT
claim. Existing implementation/evidence constraints are explained in
their inline replies. The two-level TOCs remain as requested.

## Validation

- `git diff --check` and staged diff checks: passed. The untouched
upstream AutoID NodeSet retains its provenance bytes, including two
existing license-header trailing spaces.
- `.\.azurepipelines\validate-migration-plugin.ps1`: passed; 15 bundled
migration documents and manifests synchronized.
- Documentation/path validation: **101** documents retain two rendered
TOC levels (**2,228** entries), with idempotent formatting. **52** new
relative links and their anchors passed exact-case checks. No tracked
references to the old guide/sample paths remain. Both relocated
provider-contract sections are preserved in the destination.
- `dotnet sln UA.slnx list`: passed; shows the renamed Di/ISA95 projects
and the existing FluentApi/Lds/PubSub projects. Updated solution and
pump ProjectReference paths also passed exact-case existence checks.
- Latest focused commands, run from the repository root:

```powershell
# Passed: 61 channel send-order and reconnect tests.
dotnet test tests\Opc.Ua.Core.Tests\Opc.Ua.Core.Tests.csproj -p:CustomTestTarget=net10.0 --filter 'FullyQualifiedName~UaSCBinaryChannelSendOrderTests|FullyQualifiedName~TcpReconnectKeyAgreementTests' --verbosity minimal

# Passed: 5 pump datasheet tests; builds the renamed pump sample.
dotnet test tests\Opc.Ua.Di.Tests\Opc.Ua.Di.Tests.csproj -p:CustomTestTarget=net10.0 --filter 'FullyQualifiedName~PumpDatasheetConformanceTests' --verbosity minimal

# Passed: 4 repository NodeSet path/round-trip tests, including the moved pump models.
dotnet test tests\Opc.Ua.Types.Tests\Opc.Ua.Types.Tests.csproj -p:CustomTestTarget=net10.0 --filter 'FullyQualifiedName~RepositorySourceSurvivesSerializedLinkedPartitionsAsync' --verbosity minimal

# Passed: build, zero warnings/errors.
dotnet build samples\ISA95\MinimalIsa95Server\MinimalIsa95Server.csproj -p:CustomTestTarget=net10.0 -f net10.0 --verbosity minimal

# Passed: dependent sample-test project build, zero warnings/errors.
dotnet build tests\Opc.Ua.WotCon.Samples.Tests\Opc.Ua.WotCon.Samples.Tests.csproj -p:CustomTestTarget=net10.0 -f net10.0 --verbosity minimal
```

Earlier focused integration checks passed 2,620 tests across AutoID,
event queues/scheduling, source generation, MCP/package profiles and
WoT. Both reviewed Mermaid diagrams rendered locally without network
access. These checks are not a full solution run.

The full solution has **not** been rerun for this review. Earlier
`dotnet test UA.slnx -p:CustomTestTarget=net10.0 --verbosity quiet` did
not pass: three AOT projects rejected VSTest and 20 Vision tests failed
during graphics initialization with `OutOfMemoryException`. Remaining
long-running tests were stopped at the user's request. A focused Vision
rerun passed 558 tests and reproduced those 20 failures. The earlier
`dotnet test UA.slnx -p:CustomTestTarget=net48 --verbosity quiet` was
blocked by the former SDK pin and was not retried after the policy
change. The user approved proceeding with those limitations documented;
current remote CI is not claimed green.

## Related Issues

- Follow-up #4842 tracks the requested 2.0.2+ benchmark/tuning and
diagnostic-tool documentation refresh. #4841 supplies documentation and
earlier results, not all newly requested measurements.
- #4758 explains the conversion guide's earlier consolidation; the
current provider contracts are now consolidated into
`WotNodeSetConversion.md`.
- #4839 supplies the reviewed API corrections; #4843 supplies the
updated CTT issue record.
- The original prose cleanup and SDK policy were user-requested without
a source issue. No new runtime architecture is introduced by these
documentation edits.

## Checklist

_Put an `x` in the boxes that apply. You can complete these step by step
after opening the PR._

- [ ] 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.
- [ ] I have added tests that prove my fix is effective or that my
feature works and increased code coverage.
- [x] I have added all necessary documentation.
- [ ] I have verified that my changes do not introduce (new) build or
analyzer warnings.
- [ ] I ran **all** tests locally using the **UA.slnx** solution against
at least .net **framework** and .net **10**, and all passed.
- [ ] 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: Roman Ettlinger <romanett98@gmail.com>
Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
2026-10-21 10:10:55 +02:00

6.5 KiB

Generation-bound session clients

ISessionBindingProvider is an optional capability for operations whose identity must not change during a sequence of requests or after a generated client is returned. Stock Session and ManagedSession implement it. Existing ISession, ISessionClient and ITransportChannel implementations acquire no new mandatory members, and ordinary unbound clients retain their normal reconnect behavior.

This client capability is distinct from the server's IServerSessionBindingProvider, which provides read-only committed-session membership and classification snapshots through HasSession and TryGetSessionContext. Neither interface requires the other's members.

Contents

Capture and use

Establish and authenticate the owning session through the usual direct factory or client DI setup. Then capture a client:

if (session is not ISessionBindingProvider provider)
{
    throw new ServiceResultException(StatusCodes.BadNotSupported);
}

ISessionClient binding = await provider.CreateBindingAsync(ct);
try
{
    DataValue state = await binding.ReadValueAsync(
        VariableIds.Server_ServerStatus_State, ct);
    var file = new FileTypeClient(binding, fileNodeId, telemetry);
    uint handle = await file.OpenAsync(1, ct);
    try
    {
        ByteString bytes = await file.ReadAsync(handle, 4096, ct);
    }
    finally
    {
        await file.CloseAsync(handle, CancellationToken.None);
    }
}
finally
{
    await binding.CloseAsync(CancellationToken.None);
    binding.Dispose();
}

The existing generated ObjectType clients consume ISessionClient, so no parallel Read/Browse/Call implementation or new connection manager is needed. Capture does not create a new server session or open document content.

The binding is non-owning with respect to the original session and network connection. Closing or disposing it invalidates only the captured client and releases its local diagnostics. It does not hold an exclusive gate for its lifetime. The application continues to own the original session's lifetime.

What is captured

The binding retains the authenticated native transport generation, the session/authentication incarnation, and private namespace/server mapping snapshots. A source map mutation, including a change followed by restoration of the same values, invalidates the old binding. StringTable.GetSnapshot captures the mapping and its mutation Version together.

Capture also checks that the owning session's maps agree with the live context of the transport actually selected for dispatch. Contradictory holders reject capture. Both the source context and the copied context track table-holder generations: replacing either table invalidates the binding even if the original reference is restored before the next request. Assigning an unchanged reference does not itself invalidate a binding.

The bound client cannot replace its channel/session incarnation or override its authentication token. Its endpoint and channel-certificate observations are captured values, not references to a later selected peer. Its generated clients continue interpreting NodeIds against the captured maps.

For UA Secure Conversation binary transports, dispatch validation and submission to the captured inner channel occur at the native channel owner's replacement gate. Response completion is awaited outside that gate. Managed bindings compose the lease's entry, swap and reconnect generation checks with that native path; they do not enter the ordinary managed idempotent-retry loop.

Before-dispatch invalidation rejects the request. A request already dispatched can complete only against its captured transport; if the binding is invalidated while it is in flight, its result is rejected rather than treated as evidence for the replacement generation. A transport failure associated with proven invalidation retains the original exception as its cause.

Invalidation, cancellation and support

Invalidated bindings report BadSecurityChecksFailed; a locally closed client can also report its normal closed-client status. Invalidation is permanent for that binding, even if the owning capability later becomes current again. Cancellation remains OperationCanceledException and is not converted into a binding success. Capture on a closed session fails, and reconnect is not supported on the captured transport itself.

After a legitimate reconnect, namespace-table update or authorized endpoint relocation, capture a fresh binding and resolve portable identities again. An old binding fails rather than silently retargeting old NodeIds. This does not change the origin/identity policy applied by a higher-level client.

The stock implementation supports UA-SC binary native channels (including their TCP and byte-transport-derived channels) and managed leases over those channels. Other transport implementations and custom mapping-table subclasses without the required owner tracking reject capture with BadNotSupported, as do custom message-context implementations. Custom session adapters may expose ISessionBindingProvider only if they enforce the complete dispatch/result guarantee; comparing current endpoint labels is not sufficient. No support is inferred merely from an adapter implementing ISession.

This is an additive dispatch capability, not a new resolver, cache, reconnect engine, registry persistence mechanism or server-restoration phase.

Compatibility and deployment

The client capability adds no mandatory members to existing session or transport interfaces. The server lookup previously named Opc.Ua.ISessionBindingProvider is renamed to Opc.Ua.IServerSessionBindingProvider to distinguish the two capabilities. This is a source and binary breaking change for consumers of that preview server lookup: update implementations, casts and DI registrations, and rebuild against the renamed interface. Its committed-session semantics are unchanged.

Capture uses existing session factories and client DI configuration; it requires no separate registration. The implementation introduces no dynamic code generation or reflection-based member discovery. These facts do not replace validation of an application's generated types, transport and dependency closure under NativeAOT.