## Proposed changes Fixes #4857. The stack's log messages wrote network addresses (endpoint, discovery and reverse connect URLs, host names, IP addresses, ports), application and user identities, file paths and exception texts as plain parameters. Applications had no reliable way to keep them out of general-purpose logs. This PR replaces the stack's own redaction mechanism with the standard .NET [data classification and redaction](https://learn.microsoft.com/dotnet/core/extensions/data-redaction) packages. Each sensitive `[LoggerMessage]` parameter is classified. The application decides per classification what reaches its logs: erase it, replace it with an HMAC token so entries stay correlatable, or pass it through. ### Taxonomy `Opc.Ua.Redaction.OpcUaDataClassifications` defines the `OpcUa` taxonomy and one attribute per classification: | Classification | Used for | Parameters | |---|---|---| | `[NetworkAddress]` | URLs, host and domain names, IP/MAC addresses, end points, ports | 120 | | `[ApplicationIdentity]` | certificates, subject, issuer, thumbprint, ApplicationUri | 103 | | `[FilePath]` | file, directory and certificate store paths | 67 | | `[SensitiveText]` | exception or remote-supplied text passed as a parameter | 55 | | `[UserIdentity]` | user names, token subjects | 7 | Correlation ids (`ChannelId`, `SessionId`, ...), status codes and NodeIds stay unclassified. ### Pipeline and defaults - Opc.Ua.Core now references `Microsoft.Extensions.Telemetry(.Abstractions)` 10.11.0, so every project above Core uses the `Microsoft.Gen.Logging` generator. Opc.Ua.Client already did, through Http.Resilience. The packages ship net462 and netstandard2.0 builds, so net48 is supported. - **A plain `ILoggerFactory` logs `(null)` in place of classified values.** Classified values are only written by the redaction-aware logger. - The new `ILoggingBuilder.EnableOpcUaRedaction()` turns that logger on. It is `EnableRedaction()` with the discriminator off; otherwise every unredacted value gets `:ParamName` appended. `AddOpcUa().AddLogging(...)` and `DefaultTelemetry.Create(...)` call it, as do the samples, so they log the same information as before until redactors are registered. - `services.AddOpcUa()` alone does not change the host's logging. Generic-host apps call `builder.Logging.EnableOpcUaRedaction()`. This is documented as a behavior-breaking change in `docs/migrate/2.0.x/telemetry.md`. ### Obsolete API `Redact`, `RedactionWrapper<T>`, `IRedactionStrategy`, `RedactionStrategies` and `SimpleRedactionStrategy` are `[Obsolete]`. They still work, but the stack no longer uses them. ### Not classified - The `Exception` passed to a log call. Applications can drop exception messages with `LoggerEnrichmentOptions.IncludeExceptionMessage`. - Exception and `ServiceResult` message texts. - `ServiceResult`/`StatusCode` parameters and logged data values. - The few log calls whose level is decided at run time. - `Opc.Ua.Vision.OpenUsd`, which references only Opc.Ua.Types and cannot see the attributes. ### Generator behavior differences These are visible to log consumers and to tests: - `IFormattable` values such as `NodeId` are stored in the structured state as invariant strings, not as objects. - The generated log state is a thread-local object that is cleared after the call. - Classified values are pooled redactor objects, valid only during the `Log` call. Tests that inspected log state after the fact now capture it during the call. The shared `RecordingLoggerProvider` snapshots classified values. ## Related Issues - Fixes #4857 ## Types of changes - [ ] Bugfix (non-breaking change which fixes an issue) - [x] Enhancement (non-breaking change which adds functionality) - [ ] Test enhancement (non-breaking change to increase test coverage) - [x] Breaking change (fix or feature that would cause existing functionality to not work as expected; plain logger factories drop classified values) - [x] Documentation Update (if none of the other choices apply) ## Checklist - [x] I have read the CONTRIBUTING doc. - [x] I have signed the CLA. - [x] I ran tests locally with my changes, all passed. - [x] I fixed all failing tests in the CI pipelines. - [x] I fixed all introduced issues with CodeQL and LGTM. - [x] I have added tests that prove my fix is effective or that my feature works and increased code coverage. - [x] I have added necessary documentation (if appropriate). - [ ] Any dependent changes have been merged and published in downstream modules. ## Further comments New tests: - `LogClassificationCoverageTests` (Tools.Tests) scans all 104 stack assemblies and fails if an address, identity or path parameter of a `[LoggerMessage]` method is unclassified. - `DataClassificationLoggingTests` (Core.Tests) pins erasing and HMAC redaction, pass-through, the plain-factory drop, and both stack defaults. Local results on net10.0: | Suite | Passed | Failed | |---|---|---| | Core.Tests | 6,489 | 0 | | Client.Tests | 3,231 | 0 | | Tools.Tests | 7,210 | 0 | | PubSub.Tests | 1,536 | 0 | | Core.Diagnostics.Tests | 571 | 0 | | Configuration.Tests | 241 | 0 | - **Server.Tests:** the full run predates the final test fixes and had 12 failures. All 12 were in the fixtures fixed afterwards, which now pass: RoleStateBinding (45/45) and the FileDirectory and compatibility-logging fixtures (74/74 in total). The full Core.Tests and Server.Tests runs were not repeated after those fixes. - **Builds:** `UA.slnx` builds with 0 errors for net10.0 and net48, and the whitespace check passes. - **NativeAOT:** not confirmed locally, because the machine has no C++ linker. A trimmed publish of `Opc.Ua.Aot.Tests` reported no trim or AOT warnings from the stack, the new packages or the generated logging code; its only errors come from the TUnit test framework (`FSharp.Core` not found). The `aot-test` CI job has to confirm the native build. 🤖 Generated with [Claude Code](https://claude.com/claude-code) --------- Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com> Co-authored-by: Marc Schier <marcschier@users.noreply.github.com>
AI Model Management sample
An OPC UA Server that exposes AI models — hosted somewhere else, or running on the machine — through the draft companion specification OPC UA — AI Model Management and Inference. Plus a client that exercises it, and a Helm chart that runs it on Kubernetes.
The point of the sample is to demonstrate: where inference runs should not
change how it is called. A client discovers a deployment, calls Invoke, and
gets an answer together with the identity of the model that produced it —
whether that model was a hosted service in another jurisdiction or a quantized
copy on the same box.
The specification is a draft. Namespace URIs and NodeIds are provisional.
What is in here
The reusable parts live under src/ as the Opc.Ua.AI package family, in the
same shape as the Robotics and Vision families: a model assembly, a server-side
assembly, a client-side assembly, and the inference backends. What remains here
is the two sample applications that compose them.
../../src/Opc.Ua.AI/ |
The companion specification's NodeSet, source-generated at build time |
../../src/Opc.Ua.AI.Inference/ |
Reaching a model: the IInferenceBackend contract, a Microsoft.Extensions.AI IChatClient backend, an OpenAI-compatible REST backend, credential resolution |
../../src/Opc.Ua.AI.Server/ |
The node manager: publishes the address space and serves the specification's Methods |
../../src/Opc.Ua.AI.Client/ |
Discovery, typed reads, Method calls and artefact transfer |
ModelManagementServer/ |
The Server sample. Hosts the node manager and picks a backend |
ModelManagementClient/ |
A console client that browses from the entry point and exercises what it finds |
deploy/helm/ |
The chart, its tests, and a cluster smoke test |
../../tests/Opc.Ua.AI.Tests/ |
Unit and integration tests against a fake backend |
No cloud-vendor SDK is referenced anywhere. The default backend goes
through Microsoft.Extensions.AI, which a hosted service and an on-device
runtime both implement. The built-in workload-identity resolver reads a projected
token file when the deployment provides one. Hosts that need token exchange,
metadata-service acquisition or another identity mechanism supply an acquisition
delegate. The endpoint must accept the token the resolver returns.
A sample that only ran against one vendor would not be demonstrating a
platform-independent Server. RestChatCompletionsBackend remains available
when the endpoint only exposes the OpenAI-compatible REST contract and the host
cannot supply an IChatClient; set InferenceBackend:Kind (or
FallbackInferenceBackend:Kind) to RestChatCompletions for that wire
contract.
Running it
The Server needs something to infer with. The quickest is the throwaway endpoint
in verify_backend.py, which speaks just enough of the OpenAI-compatible
contract to answer:
python samples/AI/verify_backend.py 5273
Then, in two more terminals:
dotnet run --project samples/AI/ModelManagementServer
dotnet run --project samples/AI/ModelManagementClient
The client browses to the AI root under the Server Object, walks every
deployment, follows UsesModel to the model and its digest, and then calls
GetCapabilities, Invoke, BeginTransfer, InvokeAsync and the source's
TestConnection and ListModels.
Point it at a real endpoint by configuration:
$env:InferenceBackend__Kind = "ChatClient"
$env:InferenceBackend__EndpointUri = "https://<resource>.services.ai.azure.com/openai/"
$env:InferenceBackend__Authentication = "ApiKey"
$env:InferenceBackend__CredentialReference = "inference-api-key"
$env:InferenceBackend__Site = "Cloud"
$env:InferenceBackend__EgressPermitted = "true"
ChatClient is the default. This sample registers a small IChatClient over
the OpenAI-compatible endpoint so no vendor package is needed. A production
host can replace IChatClientFactory with one that creates the chat client
from its own SDK or local runtime. Use RestChatCompletions for deployments
whose contract is the REST shape itself rather than the
Microsoft.Extensions.AI abstraction.
The control plane is OPC UA
There is no second management API, and that is a decision rather than an omission. The specification defines Methods for the things an operator needs to do — test a source, list what it offers, invoke a model, start a job, promote a candidate — so adding an HTTP surface beside them would create two ways to do the same thing that could disagree.
Everything else is startup configuration: which endpoint to reach, which deployments to publish, and which credential to present.
What the address space says, and why
A few members answer questions that are asked before a call rather than after one.
ModelUsed names the model that actually produced a result. It exists for
the fallback case: a caller that cannot see which model answered cannot tell a
degraded answer from a good one, and a fallback that answers silently looks
exactly like a healthy primary. FallbackTests checks that ModelUsed identifies
the model that answered; ProvenanceTests follows the result's model reference
to its metadata.
EgressPermitted, DataJurisdiction, RetainsInput say where the data
goes. Egress is not made false by encryption — that answers who can read data in
flight, not where the data went.
MaxInlinePayloadSize is published before a client calls, rather than
discovered from a rejection, because the real bound is the smallest of several
limits a client can see none of.
PayloadUri is not implemented by this sample. A URI-only Invoke or
InvokeAsync call returns BadNotSupported; the sample never accepts it and
silently invokes a backend without request bytes. Inline calls can set
temperature (0 through 2), max_tokens (positive integer), and top_p
(0 through 1). These parameters apply to both shipped backends.
CredentialReference names the credential; it never carries one. A client
is entitled to know which credential is configured so it can tell whether the
right one is. A client that could read the value could use it.
Digest is empty when the backend declares none. A hosted endpoint that
will not say which weights answered cannot be made to say so by hashing its
name, and a digest that looks like an artefact digest but is not one is worse
than none, because something will eventually compare it.
Implementation status
Implemented. The address space, the Methods, the provenance references, the
chunked transfer over Part 5 FileType, the asynchronous job on the Part 10
program lifecycle, the fallback and its reporting, the credential handling, and
the HTTP client that reaches an OpenAI-compatible endpoint. The learning job
node is also real: the Server publishes a LearningJobType instance, and
SamplesCollected is incremented only when host-level code records a distinct
ground-truth sample. Empty or retracted observations count the same way as
samples carrying geometry. The sample includes endpoint verification tooling
and a Kubernetes smoke-test script; see the verification instructions below.
Not implemented. There is no retraining loop. A sample cannot retrain a
model, and PromoteModel is not wired to a simulated MLOps integration — faking
candidate generation or timed promotion would mislead a reader about the one
part of the specification a sample cannot honestly demonstrate.
Note:
FakeInferenceBackendlives in the test project and is not configurable from the Server. It exists so CI needs no inference service; if it shipped as a supported option the sample could look healthy while never having reached a model.
Kubernetes
docker build -f samples/AI/ModelManagementServer/Dockerfile `
-t modelmanagementserver:local .
helm install ai samples/AI/deploy/helm/ai-model-management `
--set image.repository=modelmanagementserver `
--set image.tag=local --set image.pullPolicy=Never
The defaults describe an on-device runtime on loopback with no credential,
because that is the shape that installs and runs without anything else existing.
deploy/helm/values-cloud.yaml is the hosted shape: a remote endpoint with a
mounted credential, and a local fallback.
Supply the credential as a Secret you manage, not through Helm:
kubectl create secret generic inference-credentials `
--from-literal=inference-api-key=<key>
helm install ai ./ai-model-management -f values-cloud.yaml `
--set credentials.existingSecret=inference-credentials
credentials.create exists for local clusters and puts the value in the release
history, where helm get values will show it. The chart says so when you use it.
The chart refuses some configurations
Each refusal corresponds to a deployment that would come up green and describe itself wrongly, which is worse than one that fails to start because nobody investigates a healthy pod:
ApiKeyauthentication with no credential mounted.- A fallback deployment with no fallback endpoint — it would always fail.
- A fallback pointing at the primary's endpoint — that is a retry, and it fails for every reason the primary just failed for.
EgressPermitted: falsewith a backend that is not on the machine — the Server would publish a promise it does not keep.
Probes
The probes are TCP against the OPC UA port. That proves the listener accepts connections; it does not prove the Server is serving the address space. An exec probe running a real OPC UA client would prove it and costs a process launch every period. The trade is deliberate, and it is the reason the smoke test drives a real session rather than trusting readiness.
Testing the chart
helm lint samples/AI/deploy/helm/ai-model-management
python samples/AI/deploy/helm/chart_tests.py
# slow, needs Docker and kind; creates and deletes a cluster
pwsh samples/AI/deploy/helm/smoke-test.ps1
chart_tests.py renders the chart and asserts what the manifests must and must
not say — including that a credential passed inline appears exactly once, inside
the Secret it belongs in. Every refusal is asserted to fire, because a guardrail
that never triggers is indistinguishable from one that does not work.
smoke-test.ps1 builds the image, creates a cluster, deploys a stub endpoint
alongside, installs the chart, and opens a real OPC UA session from outside. It
is the slowest and most fragile check here — image build, image load, chart
install and a live session — so run it deliberately rather than letting it gate
everything else.
Why AOT is disabled
PublishAot is off for this sample. The workload-identity credential path
resolves tokens through a library that serialises by reflection, which the AOT
and trimming analyzers warn on, and this repository builds warnings as errors.
Nothing in the OPC UA surface requires AOT, and disabling it for one sample was
preferable to weakening the analyzer settings for the whole repository.