Skip to content

[worker] Mask secrets before writing build logs to stdout - #4634

Closed
sjchmiela wants to merge 2 commits into
mainfrom
stanley/mask-worker-stdout
Closed

sjchmiela wants to merge 2 commits into
mainfrom
stanley/mask-worker-stdout

[worker] Mask secrets in all build log fields without stalling on stdout

fe9ade0
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Oct 15, 2026 in 0s

74.46% (+0.03%) compared to 6d8c29c

View this Pull Request on Codecov

74.46% (+0.03%) compared to 6d8c29c

Details

Codecov Report

❌ Patch coverage is 97.67442% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 74.46%. Comparing base (6d8c29c) to head (fe9ade0).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
packages/worker/src/logger.ts 97.68% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4634      +/-   ##
==========================================
+ Coverage   74.44%   74.46%   +0.03%     
==========================================
  Files        1029     1029              
  Lines       50357    50395      +38     
  Branches    10875    10891      +16     
==========================================
+ Hits        37481    37524      +43     
+ Misses      11886    11881       -5     
  Partials      990      990              

☔ 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.