Skip to content

Improve sanitizer warning details#107

Open
Suhaibinator wants to merge 1 commit intomainfrom
codex/improve-route-logging-for-sanitizers
Open

Improve sanitizer warning details#107
Suhaibinator wants to merge 1 commit intomainfrom
codex/improve-route-logging-for-sanitizers

Conversation

@Suhaibinator
Copy link
Owner

Motivation

  • The sanitizer registration warning lacked route-specific context, making it hard to identify which routes were missing a sanitizer.

Description

  • Updated pkg/router/route.go to compute a methods slice and include the HTTP methods and path directly in the warning message using fmt.Sprintf and strings.Join, while still emitting structured zap fields (path and methods).

Testing

  • No automated tests were run for this change.

Codex Task

@codecov
Copy link

codecov bot commented Feb 5, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.49%. Comparing base (41d9e05) to head (5190283).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #107   +/-   ##
=======================================
  Coverage   97.48%   97.49%           
=======================================
  Files          18       18           
  Lines        1872     1873    +1     
=======================================
+ Hits         1825     1826    +1     
  Misses         35       35           
  Partials       12       12           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant