Skip to content

Commit e86e65f

Browse files
committed
fix: update download logic to use bucket from DriveFileItem
1 parent f532080 commit e86e65f

3 files changed

Lines changed: 6 additions & 6 deletions

File tree

src/commands/download-file.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -77,13 +77,13 @@ export default class DownloadFile extends Command {
7777
const { user } = await AuthService.instance.getAuthDetails();
7878

7979
CLIUtils.doing('Preparing Network', flags['json']);
80-
const { networkFacade, bucket, mnemonic } = await CLIUtils.prepareNetwork(user);
80+
const { networkFacade, mnemonic } = await CLIUtils.prepareNetwork(user);
8181
CLIUtils.done(flags['json']);
8282
// Download the file
8383
const fileWriteStream = createWriteStream(downloadPath);
8484

8585
const [executeDownload, abortable] = await networkFacade.downloadToStream(
86-
bucket,
86+
driveFile.bucket,
8787
mnemonic,
8888
driveFile.fileId,
8989
driveFile.size,

src/webdav/handlers/GET.handler.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -57,10 +57,10 @@ export class GETRequestHandler implements WebDavMethodHandler {
5757
throw new NotValidFileIdError();
5858
}
5959

60-
const { networkFacade, bucket, mnemonic } = await CLIUtils.prepareNetwork(user);
60+
const { networkFacade, mnemonic } = await CLIUtils.prepareNetwork(user);
6161

6262
const [executeDownload] = await networkFacade.downloadToStream(
63-
bucket,
63+
driveFile.bucket,
6464
mnemonic,
6565
driveFile.fileId,
6666
contentLength,

test/webdav/handlers/GET.handler.test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,7 @@ describe('GET request handler', () => {
9696
expect(getFileMetadataStub).toHaveBeenCalledOnce();
9797
expect(authDetailsStub).toHaveBeenCalledOnce();
9898
expect(downloadStreamStub).toHaveBeenCalledWith(
99-
networkOptions.bucket,
99+
mockFile.bucket,
100100
mockAuthDetails.user.mnemonic,
101101
mockFile.fileId,
102102
mockFile.size,
@@ -151,7 +151,7 @@ describe('GET request handler', () => {
151151
expect(getFileMetadataStub).toHaveBeenCalledOnce();
152152
expect(authDetailsStub).toHaveBeenCalledOnce();
153153
expect(downloadStreamStub).toHaveBeenCalledWith(
154-
networkOptions.bucket,
154+
mockFile.bucket,
155155
mockAuthDetails.user.mnemonic,
156156
mockFile.fileId,
157157
mockSize - rangeStart,

0 commit comments

Comments
 (0)