Skip to content

fix(api): confine notification, log, and API-key file operations - #2076

Merged
Eli Bosley (elibosley) merged 3 commits into
mainfrom
fix/notification-path-validation
Sep 8, 2026
Merged

fix(api): confine notification, log, and API-key file operations#2076
Eli Bosley (elibosley) merged 3 commits into
mainfrom
fix/notification-path-validation

fix(api): remove unused unrestricted boot writer

ba02e3e
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Sep 8, 2026 in 0s

53.36% (+0.03%) compared to 3ec4764

View this Pull Request on Codecov

53.36% (+0.03%) compared to 3ec4764

Details

Codecov Report

❌ Patch coverage is 92.10526% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 53.36%. Comparing base (3ec4764) to head (ba02e3e).

Files with missing lines Patch % Lines
api/src/unraid-api/auth/api-key.service.ts 83.33% 2 Missing ⚠️
.../src/core/utils/files/resolve-file-in-directory.ts 92.30% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2076      +/-   ##
==========================================
+ Coverage   53.33%   53.36%   +0.03%     
==========================================
  Files        1043     1043              
  Lines       72664    72674      +10     
  Branches     8387     8395       +8     
==========================================
+ Hits        38758    38786      +28     
+ Misses      33779    33761      -18     
  Partials      127      127              

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