Skip to content

Feature: Sanitize credential-like URLs in telemetry events to avoid False CredScan detection#340

Open
yashnap wants to merge 25 commits into
masterfrom
37515045lpe-uri-credential-redaction
Open

Feature: Sanitize credential-like URLs in telemetry events to avoid False CredScan detection#340
yashnap wants to merge 25 commits into
masterfrom
37515045lpe-uri-credential-redaction

Remove newly added lines

4974262
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project failed Jul 9, 2026 in 1s

87.53% (-6.52%) compared to f14a3a9

View this Pull Request on Codecov

87.53% (-6.52%) compared to f14a3a9

Details

Codecov Report

❌ Patch coverage is 80.50847% with 23 lines in your changes missing coverage. Please review.
✅ Project coverage is 87.53%. Comparing base (f14a3a9) to head (4974262).
⚠️ Report is 2 commits behind head on master.

Files with missing lines Patch % Lines
src/extension/tests/Test_TelemetryWriter.py 26.31% 14 Missing ⚠️
src/extension/src/CredentialSanitizer.py 41.66% 7 Missing ⚠️
src/extension/src/TelemetryWriter.py 50.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #340      +/-   ##
==========================================
- Coverage   94.04%   87.53%   -6.52%     
==========================================
  Files         109      111       +2     
  Lines       19967    20078     +111     
==========================================
- Hits        18778    17575    -1203     
- Misses       1189     2503    +1314     

☔ View full report in Codecov by Harness.
📢 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.