deps: Update dependency NLog to 5.5.1 - #9
Closed
renovate[bot] wants to merge 1 commit into
Closed
Conversation
ANcpLua
added a commit
that referenced
this pull request
Jul 1, 2026
…trument (CODE RED #1/#2/#10) (#22) An adversarial OTel-compliance + honesty audit rated this code 3/10. This lands the verified subset of the confirmed HIGH/MEDIUM defects. #1 (HIGH, ~0-duration lie): the HttpClient/AspNetCore/gRPC DiagnosticListeners created their span on the framework's *.Stop event via the 2-arg StartActivity (StartTimeUtc=now) and immediately disposed it, so every emitted span had ~0 duration instead of the real operation latency. New internal QylActivitySource.StartAtAmbientStart(name, kind) stamps the span to the ambient framework Activity's real StartTimeUtc (parented to it for trace correlation), with a now-stamped fallback when there is no ambient activity. All three listeners switched to it. #2 (HIGH, dishonest doc): DiagnosticListenerSubscriber claimed it "publishes the same span shapes" — false while durations were fabricated. Doc now states the actual mechanism (reacts on *.Stop, stamps to ambient start for real duration). #10 (MEDIUM, metrics-semconv): dotnet.thread_pool.thread.count was an unitless ObservableGauge<int>; per OTel semconv it is an (Observable)UpDownCounter with UCUM unit {thread}. Switched instrument type + added unit. Verified: core.slnf Release 0/0; verify-webapi-aot-demo passes (fixture unchanged — the attribute shape is identical; note the fixture is duration-insensitive, which is itself tracked as a separate finding). No PublicAPI change (all internal/private). NOT in this commit (separate verified increments): #3 double-count between the listener and interceptor lanes (+ the fixture that masks it), #4/#11 OTLP Events/Links/Status.Message dropped on ingest, #6 url.scheme, #7 http.request.method_original on the interceptor path, #9 Azure span name ignores methodName. Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
ANcpLua
added a commit
that referenced
this pull request
Jul 1, 2026
…ginal, Azure operation (CODE RED #6/#7/#9) (#25) #6 (url.scheme, Required on HTTP server spans): StartServerActivity now sets url.scheme from context.Request.Scheme (threaded through QylInterceptedAspNetCore). Added to the AOT demo keep-list so the fixture asserts it. #7 (http.request.method_original): the interceptor client + server paths dropped the original method when Normalize collapsed a non-standard verb to _OTHER. New QylHttpMethod.Normalize(method, out original) returns the raw value for non-standard methods; SetRequestMethod emits http.request.method_original. Threaded through StartClientActivity / StartServerActivity and all three callers (HttpClient, HttpWebRequest, AspNetCore). #9 (Azure operation): QylInterceptedAzure.StartActivity received methodName and dropped it (every Azure span named the fixed "Azure SDK"). Now records it on code.function.name so spans are distinguishable (span name stays low-cardinality per Azure semconv). New public consts QylSemanticAttributes.UrlScheme + CodeFunctionName (PublicAPI.Unshipped updated). Verified: core 0/0; verify-webapi-aot-demo (fixture now asserts url.scheme=http on the server span), source-interceptor-consumer, public-api-baseline all pass. Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
ANcpLua
added a commit
that referenced
this pull request
Jul 3, 2026
… fix-markers (#33) - Delete the templated <example><code>var apiType = typeof(X);</code></example> filler line from 27 runtime files (pure padding, documents nothing) - QylInterceptedAzure: drop the CODE RED #9 past-bug marker; keep the live low-cardinality span-name constraint - QylSemConvRegistry: drop the 'previous substrate' history narration; keep the AOT-safety constraint that motivates the generator-emitted registry Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR contains the following updates:
5.5.0→5.5.1Release Notes
NLog/NLog (NLog)
v5.5.1Improvements
Configuration
📅 Schedule: (in timezone UTC)
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.