New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
FIX make sure pre_dispatch cannot do arbitrary code execution #1321
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1321 +/- ##
==========================================
+ Coverage 93.90% 93.92% +0.01%
==========================================
Files 50 50
Lines 7270 7270
==========================================
+ Hits 6827 6828 +1
+ Misses 443 442 -1
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
|
Thanks for the fix! I assume that this is enough but alternatively we could try to use the Let's wait for the CI to complete before merging. |
|
I'm not sure if it's worth having this feature if we're going to parse it and add that complexity to the code |
|
The CI was green, merged. |
|
Cool! |
See joblib/joblib#1321 for details.
Fixes #1128
Make sure nothing's available to
evalforpre_dispatch.cc @ogrisel