-
Notifications
You must be signed in to change notification settings - Fork 8.7k
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): Remove SAML config metadataUrl if XML metadata is set directly #6143
fix(core): Remove SAML config metadataUrl if XML metadata is set directly #6143
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. |
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #6143 +/- ##
=======================================
Coverage 19.33% 19.33%
=======================================
Files 2590 2590
Lines 117067 117068 +1
Branches 18371 18371
=======================================
+ Hits 22630 22631 +1
Misses 93791 93791
Partials 646 646
☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Would be great if we could add tests for those scenarios too.
✅ All Cypress E2E specs passed |
Got released with |
* master: feat(JotForm Trigger Node): Add support for hipaa-api.jotform.com (#6171) 🚀 Release 0.227.0 (#6167) fix(editor): Flag issues only on workflow activation (#6127) fix(editor): Faster reconnects for push (no-changelog) (#6164) fix: Log unhandled errors during license activation (no-changelog) (#6165) test: Address flaky setup e2e (no-changelog) (#6085) fix(core): Better errors for common status codes fix fix(core): Fix bug running addUserActivatedColumn migration on MariaDB (#6157) fix: Prevent invocations of 'GET /rest/license' from returning an error when ephemeral licenses are used (#6154) fix(editor): Restrict `[empty]` in parameter input hint to zero-length string (#6003) fix(core): Assign Unknown Error only if message or description not present in error fix(AWS S3 Node): Fix File upload, and add node tests (#6153) fix(core): Better error message in Webhook node when using the POST method feat(Microsoft Excel 365 Node): Overhaul fix(core): Remove SAML config metadataUrl if XML metadata is set directly (#6143) ci: Restore load options methods validation (no-changelog) (#6148) feat(core): Add notice to alert users a new version is available fix(core): Fix canceled execution status (#6142) ci: Expand ESLint to tests in BE packages (no-changelog) (#6147) fix(editor): Fix focus jumping when using chrome autofill (#6140)
…rce-mapper-ui-P2 * feature/resource-mapping-component: feat(JotForm Trigger Node): Add support for hipaa-api.jotform.com (#6171) 🚀 Release 0.227.0 (#6167) fix(editor): Flag issues only on workflow activation (#6127) fix(editor): Faster reconnects for push (no-changelog) (#6164) fix: Log unhandled errors during license activation (no-changelog) (#6165) test: Address flaky setup e2e (no-changelog) (#6085) fix(core): Better errors for common status codes fix fix(core): Fix bug running addUserActivatedColumn migration on MariaDB (#6157) fix: Prevent invocations of 'GET /rest/license' from returning an error when ephemeral licenses are used (#6154) fix(editor): Restrict `[empty]` in parameter input hint to zero-length string (#6003) fix(core): Assign Unknown Error only if message or description not present in error fix(AWS S3 Node): Fix File upload, and add node tests (#6153) fix(core): Better error message in Webhook node when using the POST method feat(Microsoft Excel 365 Node): Overhaul fix(core): Remove SAML config metadataUrl if XML metadata is set directly (#6143) ci: Restore load options methods validation (no-changelog) (#6148) feat(core): Add notice to alert users a new version is available fix(core): Fix canceled execution status (#6142) ci: Expand ESLint to tests in BE packages (no-changelog) (#6147) fix(editor): Fix focus jumping when using chrome autofill (#6140)
No description provided.