Skip to content

security(dci): restrict Data API pull to non-sensitive provider variables#257

Open
gonzalesedwin1123 wants to merge 2 commits into
19.0from
security-dci-crvs-cache
Open

security(dci): restrict Data API pull to non-sensitive provider variables#257
gonzalesedwin1123 wants to merge 2 commits into
19.0from
security-dci-crvs-cache

security(dci): address review on Data API pull/list hardening

5c31c51
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Jun 29, 2026 in 0s

74.80% (+0.05%) compared to f04d936

View this Pull Request on Codecov

74.80% (+0.05%) compared to f04d936

Details

Codecov Report

❌ Patch coverage is 92.00000% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 74.80%. Comparing base (f04d936) to head (5c31c51).

Files with missing lines Patch % Lines
spp_api_v2_data/routers/data.py 71.42% 2 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             19.0     #257      +/-   ##
==========================================
+ Coverage   74.75%   74.80%   +0.05%     
==========================================
  Files        1090     1092       +2     
  Lines       64813    64836      +23     
==========================================
+ Hits        48453    48503      +50     
+ Misses      16360    16333      -27     
Files with missing lines Coverage Δ
spp_cel_domain/models/cel_variable.py 82.58% <100.00%> (+0.60%) ⬆️
spp_dci_indicators/models/__init__.py 100.00% <100.00%> (ø)
spp_dci_indicators/models/cel_variable_dci.py 100.00% <100.00%> (ø)
spp_api_v2_data/routers/data.py 69.17% <71.42%> (+20.58%) ⬆️

... and 1 file with indirect coverage changes

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