Skip to content

Clear develop Sonar code smells without HTTP/2 send-path changes - #1074

Merged
justcoding121 merged 2 commits into
developfrom
fix/sonar-zero-smells
Oct 4, 2026
Merged

justcoding121 merged 2 commits into
developfrom
fix/sonar-zero-smells

Conversation

@justcoding121

Copy link
Copy Markdown
Owner

Summary

  • Clears the 52 open SonarCloud code smells on develop.
  • Hot-path HTTP/2 DATA queue, deferred drain, and header-block signatures stay as they are (explicit parameters, no per-frame context object, no LINQ on the wire path). Header delimiter checks now use allocation-free IndexOfAny.
  • Fixes the two unit tests that looked up instance terminate-lite helpers with static binding flags, which was failing the .NET build before SonarCloud could upload.

Test plan

  • dotnet build src/Titanium.Web.Proxy.sln -c Release --warnaserror
  • Full Titanium.Web.Proxy.UnitTests (1838 passed)
  • Inspector session/archive/UI filter (186 passed)
  • .NET workflow, including SonarCloud upload, reports 0 code smells on this branch
  • After merge, the develop code-smells badge is 0 and the quality gate stays green

@sonarqubecloud

sonarqubecloud Bot commented Oct 4, 2026

Copy link
Copy Markdown

@justcoding121
justcoding121 merged commit 7fd3892 into develop Oct 4, 2026
20 checks passed
@justcoding121
justcoding121 deleted the fix/sonar-zero-smells branch October 4, 2026 11:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant