-
Notifications
You must be signed in to change notification settings - Fork 8.8k
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(core): Fix broken API permissions in public API #5978
Conversation
Great PR! Please pay attention to the following items before merging: Files matching
Files matching
Make sure to check off this list before asking for review. |
packages/cli/src/PublicApi/v1/handlers/workflows/workflows.service.ts
Outdated
Show resolved
Hide resolved
✅ All Cypress E2E specs passed |
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #5978 +/- ##
==========================================
+ Coverage 17.92% 18.07% +0.15%
==========================================
Files 2539 2540 +1
Lines 115279 115391 +112
Branches 17966 17990 +24
==========================================
+ Hits 20660 20860 +200
+ Misses 93992 93903 -89
- Partials 627 628 +1
... and 7 files with indirect coverage changes Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
✅ All Cypress E2E specs passed |
Got released with |
Github issue / Community forum post (link here to close automatically):
https://linear.app/n8n/issue/N8N-6326/api-n8n-node-normal-user-can-access-execution-data-for-other-workflows