mudhorn/n8n
1
0
Fork 0
mirror of https://github.com/n8n-io/n8n.git synced 2025-03-05 20:50:17 -08:00
Code Issues Actions Packages Projects Releases Wiki Activity

remove subworkflow license check

Browse source
This commit is contained in:
Danny Martini 2024-09-19 23:15:11 +02:00
parent 48294e7ec1
commit 49c6942e6b
No known key found for this signature in database
2 changed files with 1 additions and 21 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

18
packages/cli/src/subworkflows/__tests__/subworkflow-policy-checker.test.ts
View file

@ -107,24 +107,6 @@ describe('SubworkflowPolicyChecker', () => {
}); });
describe('`any` caller policy', () => { describe('`any` caller policy', () => {
it('if no sharing, should be overriden to `workflows-from-same-owner`', async () => {
license.isSharingEnabled.mockReturnValueOnce(false);
const parentWorkflow = mock<WorkflowEntity>();
const subworkflowId = 'subworkflow-id';
const subworkflow = mock<Workflow>({ id: subworkflowId, settings: { callerPolicy: 'any' } }); // should be overridden
const parentWorkflowProject = mock<Project>({ id: uuid() });
const subworkflowProject = mock<Project>({ id: uuid(), type: 'team' });
ownershipService.getWorkflowProjectCached.mockResolvedValueOnce(parentWorkflowProject);
ownershipService.getWorkflowProjectCached.mockResolvedValueOnce(subworkflowProject);
const check = checker.check(subworkflow, parentWorkflow.id);
await expect(check).rejects.toThrowError(SubworkflowPolicyDenialError);
});
it('should not throw on a regular subworkflow call', async () => { it('should not throw on a regular subworkflow call', async () => {
const parentWorkflow = mock<WorkflowEntity>({ id: uuid() }); const parentWorkflow = mock<WorkflowEntity>({ id: uuid() });
const subworkflow = mock<Workflow>({ settings: { callerPolicy: 'any' } }); const subworkflow = mock<Workflow>({ settings: { callerPolicy: 'any' } });

4
packages/cli/src/subworkflows/subworkflow-policy-checker.service.ts
View file

@ -82,10 +82,8 @@ export class SubworkflowPolicyChecker {
* Find the subworkflow's caller policy. * Find the subworkflow's caller policy.
*/ */
private findPolicy(subworkflow: Workflow): WorkflowSettings.CallerPolicy { private findPolicy(subworkflow: Workflow): WorkflowSettings.CallerPolicy {
if (!this.license.isSharingEnabled()) return 'workflowsFromSameOwner';
return ( return (
subworkflow.settings?.callerPolicy ?? this.globalConfig.workflows.callerPolicyDefaultOption subworkflow.settings.callerPolicy ?? this.globalConfig.workflows.callerPolicyDefaultOption
); );
} }