fix: Fix 8 security issues in jsonpath, mailparser, mysql2 and 5 more#26314
fix: Fix 8 security issues in jsonpath, mailparser, mysql2 and 5 more#26314aikido-autofix[bot] wants to merge 1 commit intomasterfrom
Conversation
There was a problem hiding this comment.
No issues found across 6 files
Architecture diagram
sequenceDiagram
participant User as User / Attacker
participant FE as Frontend (Editor UI)
participant BE as Backend (n8n Core / Nodes)
participant MP as mailparser (nodes-base)
participant DB as MySQL DB (mysql2)
participant CLI as CLI Utility (zx)
Note over User, CLI: Security Hardening: Runtime Protection against CVEs (SQLi, XSS, ReDoS)
rect rgb(30, 41, 59)
Note right of User: UI Data Mapping & Expression Flow
User->>FE: Input JSONPath expression / Data
FE->>FE: CHANGED: jsonpath.value() validation
Note over FE: Prevents Prototype Pollution<br/>& Regex Injection
FE-->>FE: CHANGED: Escape URL/HTML in links
Note over FE: Prevents XSS in Editor UI
FE-->>User: Rendered JSON / Preview
end
rect rgb(23, 37, 84)
Note right of User: Workflow Execution (Email/DB)
User->>BE: Execute Workflow
BE->>MP: CHANGED: Parse incoming email stream
MP-->>BE: Sanitized Mail Object
BE->>DB: CHANGED: Execute Parameterized Query
alt Parameter Validation
DB->>DB: CHANGED: Strict type-handling
Note over DB: Fixes SQLi via non-string types
end
DB-->>BE: Query Result
BE-->>User: Workflow Success
end
rect rgb(5, 46, 22)
Note right of CLI: Development & Scripting Flow
CLI->>CLI: Execute script with --prefer-local
CLI->>CLI: Create node_modules symlink
opt Cleanup Phase
CLI->>CLI: CHANGED: Resolve correct symlink path
Note over CLI: Prevents accidental symlink<br/>deletion logic error
end
end
Note over FE,BE: Transitive Security: Glob & Word-Wrap
BE->>BE: CHANGED: Process Glob patterns / String wrapping
Note over BE: ReDoS mitigation via minimatch/word-wrap upgrades
❌ 5 Tests Failed:
View the top 3 failed test(s) by shortest run time
To view more test analytics, go to the Test Analytics Dashboard |
Bundle ReportChanges will increase total bundle size by 3.24kB (0.01%) ⬆️. This is within the configured threshold ✅ Detailed changes
Affected Assets, Files, and Routes:view changes for bundle: editor-ui-esmAssets Changed:
|
|
Hey @aikido-autofix[bot], Thank you for your contribution. We appreciate the time and effort you’ve taken to submit this pull request. Before we can proceed, please ensure the following: Regarding new nodes: If your node integrates with an AI service that you own or represent, please email nodes@n8n.io and we will be happy to discuss the best approach. About review timelines: Thank you again for contributing to n8n. |
|
Found 5 test failures on Blacksmith runners: Failures
|
Upgrades multiple packages to address critical security vulnerabilities including XSS, SQL injection, prototype pollution, symlink deletion, and regex injection risks.
✅ 8 CVEs resolved by this upgrade
This PR will resolve the following CVEs:
🔗 Related Tasks