Fix: add response type with arraybuffer readBinary call (#15)

microsoft · Sep 5, 2023 · 8608773 · 8608773
1 parent 33cb175
commit 8608773
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/SampleCode/LabelingUX/Client/src/providers/storageProvider.test.ts b/SampleCode/LabelingUX/Client/src/providers/storageProvider.test.ts
@@ -45,7 +45,9 @@ describe("StorageProvider", () => {
 
     it("should handle readBinary", async () => {
         await provider.readBinary(mockFilename);
-        expect(RequestHelper.getWithAutoRetry).toBeCalledWith(`${serverUrl}/files/${mockFilename}`);
+        expect(RequestHelper.getWithAutoRetry).toBeCalledWith(`${serverUrl}/files/${mockFilename}`, {
+            responseType: "arraybuffer",
+        });
     });
 
     it("should handle writeText", async () => {

diff --git a/SampleCode/LabelingUX/Client/src/providers/storageProvider.ts b/SampleCode/LabelingUX/Client/src/providers/storageProvider.ts
@@ -64,7 +64,7 @@ export class StorageProvider implements IStorageProvider {
     public async readBinary(filename: string, ignoreNotFound?: boolean): Promise<Buffer | undefined> {
         try {
             const api = `${serverUrl}/files/${filename}`;
-            const result = await getWithAutoRetry(api);
+            const result = await getWithAutoRetry(api, { responseType: "arraybuffer" });
 
             return result.data;
         } catch (exception) {