[KEP-9] Trusted Extensibility Features in Security Model #5597
Replies: 2 comments 3 replies
-
|
You might also want to put it behind a flag the user has to opt in to |
Beta Was this translation helpful? Give feedback.
-
|
Honestly, I don't think this requires a KEP/vote. I don't even see this as a change to Kedro. Kedro has ALWAYS been a framework that is extensible by design, that users own a lot of custom logic for. These aren't security vulnerabilities. Are the reporters even security researchers? 😂 My personal preference would be to put the responsibility on reporters to explain how the "vulnerability" would be exploited based on how Kedro is used, and to close anything without sufficient work explaining that, following a policy similar to https://tiangolo.com/open-source/contributing/#closing-automated-and-ai-prs (I like the policy to be applied for code contributions, too, honestly). +1 if it helps move forward with this :) |
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
Summary
Kedro allows custom runners, datasets, hooks, config loaders, and plugins by design. If these contain malicious code, that's the developer's responsibility, not a Kedro vulnerability.
Problem
Security researchers report "vulnerabilities" when Kedro runs user-provided code.
Example:
kedro run --runner my.CustomRunner→ reported as "arbitrary code execution vulnerability"Kedro is designed to do this. We need to clarify what's user responsibility vs. framework bug.
Proposal
Add a new section to the security model explaining that Kedro's extensibility features are designed to run user code.
The section will list the extensibility points (runners, datasets, hooks, config loaders, plugins, custom resolvers) and state clearly: if you write or install malicious code through these features, that's your responsibility, not a Kedro vulnerability.
We'll also clarify what still counts as a framework bug: configuration files executing code unintentionally, bypassing documented security restrictions, or the framework exposing credentials.
Benefits
Implementation
Update
docs/about/security_model.mdonce approved.Beta Was this translation helpful? Give feedback.
All reactions