Skip to content

Remove module access, and improve security.#171

Merged
danthedeckie merged 5 commits into
mainfrom
remove-module-access
Mar 13, 2026
Merged

Remove module access, and improve security.#171
danthedeckie merged 5 commits into
mainfrom
remove-module-access

Conversation

@danthedeckie

Copy link
Copy Markdown
Owner

Fixes 2 security issues.

  1. Accessing modules or disallowed objects as attributes of other items
  2. Passing modules or other disallowed objects as args to other callback functions.

@danthedeckie danthedeckie self-assigned this Mar 13, 2026
@codecov

codecov Bot commented Mar 13, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 96.48094% with 12 lines in your changes missing coverage. Please review.
✅ Project coverage is 98.68%. Comparing base (0425898) to head (7c9180c).

Files with missing lines Patch % Lines
test_simpleeval.py 95.90% 12 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #171      +/-   ##
==========================================
- Coverage   99.33%   98.68%   -0.65%     
==========================================
  Files           2        2              
  Lines        1209     1526     +317     
==========================================
+ Hits         1201     1506     +305     
- Misses          8       20      +12     

☔ View full report in Codecov by Sentry.
📢 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.

@danthedeckie danthedeckie merged commit a4659fa into main Mar 13, 2026
34 of 37 checks passed
@danthedeckie danthedeckie deleted the remove-module-access branch March 13, 2026 09:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant