Skip to content

Bound and sanitize TestTelemetry TRX processing - #3618

Merged
Widthdom merged 4 commits into
mainfrom
fix-issue3442-3443
Jun 13, 2026
Merged

Bound and sanitize TestTelemetry TRX processing#3618
Widthdom merged 4 commits into
mainfrom
fix-issue3442-3443

Conversation

@Widthdom

Copy link
Copy Markdown
Owner

Summary

  • Bound TestTelemetry TRX traversal, file discovery, file size, XML parser work, and retained result aggregation.
  • Sanitize TRX diagnostics to use relative paths and stable reason codes instead of raw XML/parser messages.
  • Skip non-regular .trx filesystem entries and discard malformed TRX files atomically.

Validation

  • dotnet build
  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --filter "FullyQualifiedName~TestTelemetryTests" --no-restore -p:UseSharedCompilation=false
  • dotnet test CodeIndex.sln --no-restore -p:UseSharedCompilation=false
  • dotnet run --project tools/CodeIndex.Changelog -- check
  • dotnet format CodeIndex.sln --verify-no-changes
  • dotnet ./src/CodeIndex/bin/Debug/net8.0/cdidx.dll status --check --json

Documentation and changelog

  • Added changelog.d/unreleased/3442.security.md.
  • Added changelog.d/unreleased/3443.security.md.
  • No README/developer guide update was needed because the changed behavior is limited to the internal test telemetry helper.

Review

Follow-up candidates

  • None.

Fixes #3442
Fixes #3443

@Widthdom
Widthdom marked this pull request as ready for review June 13, 2026 16:19
@Widthdom
Widthdom merged commit 2a12564 into main Jun 13, 2026
10 checks passed
@Widthdom
Widthdom deleted the fix-issue3442-3443 branch June 13, 2026 16:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

1 participant