fix AzureClient to support local tinylicious (#7571)

microsoft · Sep 23, 2021 · 185d4a8 · 185d4a8
1 parent c3a2071
commit 185d4a8
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 2 deletions.
diff --git a/api-report/azure-client.api.md b/api-report/azure-client.api.md
@@ -88,6 +88,9 @@ export { ITokenResponse }
 
 export { IUser }
 
+// @public
+export const LOCAL_MODE_TENANT_ID = "local";
+
 export { ScopeType }
 
 

diff --git a/examples/hosts/app-integration/external-controller/src/app.ts b/examples/hosts/app-integration/external-controller/src/app.ts
@@ -7,6 +7,7 @@ import {
     AzureClient,
     AzureConnectionConfig,
     AzureContainerServices,
+    LOCAL_MODE_TENANT_ID,
 } from "@fluidframework/azure-client";
 import { InsecureTokenProvider } from "@fluidframework/test-client-utils";
 import {
@@ -47,7 +48,7 @@ const connectionConfig: AzureConnectionConfig = useAzure ? {
     orderer: "",
     storage: "",
 } : {
-    tenantId: "local",
+    tenantId: LOCAL_MODE_TENANT_ID,
     tokenProvider: new InsecureTokenProvider("fooBar", user),
     orderer: "http://localhost:7070",
     storage: "http://localhost:7070",

diff --git a/packages/framework/azure-client/src/AzureClient.ts b/packages/framework/azure-client/src/AzureClient.ts
@@ -25,6 +25,11 @@ import {
 import { AzureAudience } from "./AzureAudience";
 import { AzureUrlResolver, createAzureCreateNewRequest } from "./AzureUrlResolver";
 
+/**
+ * Strongly typed id for connecting to a local Azure Fluid Relay service
+ */
+export const LOCAL_MODE_TENANT_ID = "local";
+
 /**
  * AzureClient provides the ability to have a Fluid object backed by the Azure Relay Service or,
  * when running with local tenantId, have it be backed by a Tinylicious local service instance
@@ -43,9 +48,12 @@ export class AzureClient {
             this.props.connection.orderer,
             this.props.connection.storage,
         );
+        // The local service implementation differs from the Azure Fluid Relay service in blob
+        // storage format. Azure Fluid Relay supports whole summary upload. Local currently does not.
+        const enableWholeSummaryUpload = this.props.connection.tenantId !== LOCAL_MODE_TENANT_ID;
         this.documentServiceFactory = new RouterliciousDocumentServiceFactory(
             this.props.connection.tokenProvider,
-            { enableWholeSummaryUpload: true },
+            { enableWholeSummaryUpload },
         );
     }