Skip to content

fix(process): reject a pid that would signal a process group - #593

Merged
retr0h merged 2 commits into
mainfrom
fix/process-pid-validation
Oct 7, 2026
Merged

retr0h merged 2 commits into
mainfrom
fix/process-pid-validation

test(process): cover the pid guard in Get

30aa04f
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Oct 7, 2026 in 0s

99.95% (target 99.90%)

View this Pull Request on Codecov

99.95% (target 99.90%)

Details

Codecov Report

✅ All modified and coverable lines are covered by tests.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #593   +/-   ##
=======================================
  Coverage   99.95%   99.95%           
=======================================
  Files         501      501           
  Lines       24099    24120   +21     
=======================================
+ Hits        24089    24110   +21     
  Misses         10       10           
Files with missing lines Coverage Δ
...nternal/controller/api/node/process/process_get.go 100.00% <100.00%> (ø)
...controller/api/node/process/process_signal_post.go 100.00% <100.00%> (ø)
internal/controller/api/node/process/validate.go 100.00% <100.00%> (ø)
internal/provider/node/process/debian.go 99.04% <100.00%> (+0.10%) ⬆️

Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2f036a6...30aa04f. Read the comment docs.

🚀 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.