security(chevron): use yaml.safe_load instead of yaml.load with configurable loader#3216
Conversation
…gurable loader - Replace yaml.load(file, Loader=loader) with yaml.safe_load(file) - Remove --yaml-loader CLI flag and yaml_loader kwargs - Prevents arbitrary code execution from malicious YAML data files
Greptile SummaryThis PR hardens YAML parsing in the chevron template renderer by replacing
Confidence Score: 5/5Safe to merge — the change is a well-scoped security hardening with no functional regressions in this codebase. The switch to yaml.safe_load is correct, the removed yaml_loader parameter has no callers anywhere in the repository, and the original lines were already marked # not tested. The simplification is clean with no observed side effects. No files require special attention. Important Files Changed
Reviews (1): Last reviewed commit: "security(chevron): use yaml.safe_load in..." | Re-trigger Greptile |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #3216 +/- ##
=========================================
Coverage ? 28.32%
=========================================
Files ? 381
Lines ? 52350
Branches ? 9242
=========================================
Hits ? 14826
Misses ? 36584
Partials ? 940 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
No description provided.