feat(http): surface unsafe replay suppression - #417
Conversation
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 50b857ff09
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
Addressed structured warning state in 4c4adb3: warning event 1009 now retains SuppressionReason=unsafe_method. Validation: Release solution build; logging integration tests net8.0/net10.0 (49 each). |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 4c4adb30c8
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Select the actionable unsafe-method suppression format from telemetry identity before applying the configured log level. Refs #387
1 participant
Closes #387
Summary
attempts_suppressedtelemetry eventAllowUnsafeMethodReplayand the per-requestAllowReplayopt-in in the warningValidation
dotnet build Kevlar.slnx -c Release --no-restoreKevlar.Testsnet10/net8: 1283/1249 passedKevlar.Extensions.Logging.Testsnet10/net8: 49/49 passedKevlar.IntegrationTests: 171 passedscripts/Verify-Docs.ps1scripts/Build-ApiDocs.ps1 -SkipBuildandscripts/Verify-ApiDocs.ps1scripts/Verify-DocSnippets.ps1with uniquely versioned local packages: 209 snippets compiled and executed on net8/net10npm run build