fix(deps): add dynaconf >= 3.2.13 (CVE-2026-33154)#1526
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
✅ Files skipped from review due to trivial changes (1)
📝 WalkthroughWalkthroughAdded a new dependency entry: Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
/run-e2e |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@pyproject.toml`:
- Line 46: The project adds dynaconf = ">=3.2.13" in pyproject.toml which
conflicts with the pinned dynaconf ==3.2.12 in the django-ansible-base devel
branch (requirements/requirements_all.txt), preventing resolution; to fix,
coordinate with or update django-ansible-base to relax or bump its dynaconf pin
to >=3.2.13 (or remove the strict ==3.2.12), then re-run dependency resolution
so pyproject.toml's dynaconf >=3.2.13 can be applied; reference the dynaconf
entry in pyproject.toml and the dynaconf pin in
requirements/requirements_all.txt (django-ansible-base) when making the change.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 191d5ed6-83fa-4f62-96e4-b5320baa0dcb
⛔ Files ignored due to path filters (1)
poetry.lockis excluded by!**/*.lock
📒 Files selected for processing (1)
pyproject.toml
|
/run-e2e |
Codecov Report✅ All modified and coverable lines are covered by tests. @@ Coverage Diff @@
## main #1526 +/- ##
=======================================
Coverage 91.93% 91.93%
=======================================
Files 239 239
Lines 10810 10810
=======================================
Hits 9938 9938
Misses 872 872
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
ptoscano
left a comment
There was a problem hiding this comment.
Successful e2e run: https://github.com/ansible/eda-server/actions/runs/24343812401
Also please check the feedback of coderabbit, in particular what DAB uses/pins, and in case check with the DAB folks too.
SSTI via unsafe Jinja2 template evaluation in the @Jinja resolver. Adds dynaconf as a direct dependency to force upgrade from 3.2.10. Ref: AAP-69465 Assisted by: Claude Code
c6a7d37 to
53d3ecf
Compare
|



Summary
Ref: AAP-69465
Summary by CodeRabbit