fix(FTP Node): Use filename instead of remote filepath for downloaded…

… binary data (#6170)
n8n-io · May 15, 2023 · 4993212 · 4993212
1 parent 4f16d74
commit 4993212
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/packages/nodes-base/nodes/Ftp/Ftp.node.ts b/packages/nodes-base/nodes/Ftp/Ftp.node.ts
@@ -601,11 +601,11 @@ export class Ftp implements INodeType {
 							await sftp!.get(path, createWriteStream(binaryFile.path));
 
 							const dataPropertyNameDownload = this.getNodeParameter('binaryPropertyName', i);
-							const filePathDownload = this.getNodeParameter('path', i) as string;
+							const remoteFilePath = this.getNodeParameter('path', i) as string;
 
 							items[i].binary![dataPropertyNameDownload] = await this.nodeHelpers.copyBinaryFile(
 								binaryFile.path,
-								filePathDownload,
+								basename(remoteFilePath),
 							);
 
 							const executionData = this.helpers.constructExecutionMetaData(
@@ -697,11 +697,11 @@ export class Ftp implements INodeType {
 							await streamPipeline(stream, createWriteStream(binaryFile.path));
 
 							const dataPropertyNameDownload = this.getNodeParameter('binaryPropertyName', i);
-							const filePathDownload = this.getNodeParameter('path', i) as string;
+							const remoteFilePath = this.getNodeParameter('path', i) as string;
 
 							items[i].binary![dataPropertyNameDownload] = await this.nodeHelpers.copyBinaryFile(
 								binaryFile.path,
-								filePathDownload,
+								basename(remoteFilePath),
 							);
 
 							const executionData = this.helpers.constructExecutionMetaData(