Impact
Flowise allows developers to inject configuration into the Chainflow during execution through the overrideConfig option. This is supported in both the frontend web integration and the backend Prediction API.
This has a range of fundamental issues that I believe are a major security vulnerability.
While this feature is intentional I think it should have strong protections added and be disabled by default.
These issues include:
- Remote code execution. While inside a sandbox this allows for
- Sandbox escape
- DoS by crashing the server
- SSRF
- Prompt Injection, both System and User
- Full control over LLM prompts
- Server variable and data exfiltration
And many many more such as altering the flow of a conversation, prompt exfiltration via LLM proxying etc.
These issues are self-targeted and do not persist to other users but do leave the server and business exposed.
All issues are shown with the API but also work with the web embed.
Workarounds
overrideConfig should be disabled by defaultoverrideConfig should have an explicit allow list of variables that are allowed to be modified. This way the user opts-in to where modifications can be made.vm2 and any forks of it should be removed as in the authors own words, "fixing the vulnerability seems impossible". The recommended replacement is https://www.npmjs.com/package/isolated-vm
ryanhalliday Reporter
Impact
Flowise allows developers to inject configuration into the Chainflow during execution through the
overrideConfigoption. This is supported in both the frontend web integration and the backend Prediction API.This has a range of fundamental issues that I believe are a major security vulnerability.
While this feature is intentional I think it should have strong protections added and be disabled by default.
These issues include:
And many many more such as altering the flow of a conversation, prompt exfiltration via LLM proxying etc.
These issues are self-targeted and do not persist to other users but do leave the server and business exposed.
All issues are shown with the API but also work with the web embed.
Workarounds
overrideConfigshould be disabled by defaultoverrideConfigshould have an explicit allow list of variables that are allowed to be modified. This way the user opts-in to where modifications can be made.vm2and any forks of it should be removed as in the authors own words, "fixing the vulnerability seems impossible". The recommended replacement is https://www.npmjs.com/package/isolated-vm