[App Config] Migrate to core-v2

sdk/appconfiguration/app-configuration/CHANGELOG.md

@@ -1,9 +1,13 @@
 # Release History
 
-## 1.3.2 (Unreleased)
+## 1.4.0-beta.1 (Unreleased)
 
 ### Features Added
 
+- Migrated from using `@azure/core-http` to depend on newer version of Core libraries `@azure/core-client` and `@azure/core-rest-pipeline` which bring better maintainability and performance. [#20766](https://github.com/Azure/azure-sdk-for-js/pull/20766). See [Azure Core v1 vs v2](https://github.com/Azure/azure-sdk-for-js/blob/main/sdk/core/core-rest-pipeline/documentation/core2.md) for more on the difference and benefits of the move.
+  - As part of the migration to `@azure/core-client` and `@azure/core-rest-pipeline`,
+    through the operation options, the methods such as `AppConfgurationClient#addConfigurationSetting` can take a callback (`onResponse`) to access the HTTP response. See [Change to the \_response property](https://github.com/Azure/autorest.typescript/wiki/%60core-http%60-dependency-migration-to-%60core-client%60-%60core-rest-pipeline%60#change-to-the-_response-property).
+
 ### Breaking Changes
 
 ### Bugs Fixed
@@ -101,8 +105,8 @@ This release marks the general availability of the `@azure/app-configuration` pa
   ```typescript
   new AppConfigurationClient(connectionString, {
     userAgentOptions: {
-      userAgentPrefix: "MyUserAgent"
-    }
+      userAgentPrefix: "MyUserAgent",
+    },
   });
   ```
 
@@ -158,11 +162,11 @@ In previous previews:
 await client.getConfigurationSetting("MyKey", { label: "MyLabel" });
 await client.addConfigurationSetting("MyKey", {
   label: "MyLabel",
-  value: "MyValue"
+  value: "MyValue",
 });
 await client.setConfigurationSetting("MyKey", {
   label: "MyLabel",
-  value: "MyValue"
+  value: "MyValue",
 });
 await client.deleteConfigurationSetting("MyKey", { label: "MyLabel" });
 ```
@@ -175,12 +179,12 @@ await client.getConfigurationSetting({ key: "MyKey", label: "MyLabel" });
 await client.addConfigurationSetting({
   key: "MyKey",
   label: "MyLabel",
-  value: "MyValue"
+  value: "MyValue",
 });
 await client.setConfigurationSetting({
   key: "MyKey",
   label: "MyLabel",
-  value: "MyValue"
+  value: "MyValue",
 });
 await client.deleteConfigurationSetting({ key: "MyKey", label: "MyLabel" });
 ```

sdk/appconfiguration/app-configuration/package.json

@@ -2,7 +2,7 @@
   "name": "@azure/app-configuration",
   "author": "Microsoft Corporation",
   "description": "An isomorphic client library for the Azure App Configuration service.",
-  "version": "1.3.2",
+  "version": "1.4.0-beta.1",
   "sdk-type": "client",
   "keywords": [
     "node",
@@ -81,14 +81,20 @@
       {
         "path": "src/generated/src/appConfigurationContext.ts",
         "prefix": "packageVersion"
+      },
+      {
+        "path": "swagger/swagger.md",
+        "prefix": "package-version"
       }
     ]
   },
   "dependencies": {
     "@azure/abort-controller": "^1.0.0",
     "@azure/core-asynciterator-polyfill": "^1.0.0",
-    "@azure/core-http": "^2.0.0",
+    "@azure/core-client": "^1.5.0",
+    "@azure/core-http-compat": "^1.2.0",
     "@azure/core-paging": "^1.1.1",
+    "@azure/core-rest-pipeline": "^1.6.0",
     "@azure/core-tracing": "1.0.0-preview.13",
     "@azure/core-auth": "^1.3.0",
     "tslib": "^2.2.0"
@@ -110,6 +116,7 @@
     "@types/mocha": "^7.0.2",
     "@types/node": "^12.0.0",
     "@types/sinon": "^9.0.4",
+    "@types/uuid": "^8.3.4",
     "chai": "^4.2.0",
     "dotenv": "^8.2.0",
     "eslint": "^7.15.0",
@@ -139,6 +146,7 @@
     "ts-node": "^10.0.0",
     "typescript": "~4.2.0",
     "uglify-js": "^3.4.9",
+    "uuid": "^8.3.0",
     "cross-env": "^7.0.2"
   },
   "//sampleConfiguration": {

sdk/appconfiguration/app-configuration/review/app-configuration.api.md

@@ -6,11 +6,11 @@
 
 /// <reference lib="esnext.asynciterable" />
 
-import { HttpResponse } from '@azure/core-http';
-import { OperationOptions } from '@azure/core-http';
+import { CommonClientOptions } from '@azure/core-client';
+import { CompatResponse } from '@azure/core-http-compat';
+import { OperationOptions } from '@azure/core-client';
 import { PagedAsyncIterableIterator } from '@azure/core-paging';
 import { TokenCredential } from '@azure/core-auth';
-import { UserAgentOptions } from '@azure/core-http';
 
 // @public
 export interface AddConfigurationSettingOptions extends OperationOptions {
@@ -38,9 +38,7 @@ export class AppConfigurationClient {
 }
 
 // @public
-export interface AppConfigurationClientOptions {
-    retryOptions?: RetryOptions;
-    userAgentOptions?: UserAgentOptions;
+export interface AppConfigurationClientOptions extends CommonClientOptions {
 }
 
 // @public
@@ -124,7 +122,7 @@ export interface HttpOnlyIfUnchangedField {
 
 // @public
 export interface HttpResponseField<HeadersT> {
-    _response: HttpResponse & {
+    _response: CompatResponse & {
         parsedHeaders: HeadersT;
         bodyAsText: string;
     };

sdk/appconfiguration/app-configuration/src/appConfigCredential.ts

@@ -2,55 +2,44 @@
 // Licensed under the MIT license.
 
 import {
-  ServiceClientCredentials,
-  WebResource,
-  URLBuilder,
-  WebResourceLike,
-} from "@azure/core-http";
+  PipelinePolicy,
+  PipelineRequest,
+  PipelineResponse,
+  SendRequest,
+} from "@azure/core-rest-pipeline";
 import { sha256Digest, sha256Hmac } from "./internal/cryptoHelpers";
 
 /**
- * @internal
+ * Create an HTTP pipeline policy to authenticate a request
+ * using an `AzureKeyCredential` for AppConfig.
  */
-export class AppConfigCredential implements ServiceClientCredentials {
-  private credential: string;
-  private secret: string;
-
-  constructor(credential: string, secret: string) {
-    this.credential = credential;
-    this.secret = secret;
-  }
-
-  /**
-   * Signs a request with the values provided in the credential and secret parameter.
-   *
-   * @param webResource - The WebResource to be signed.
-   * @returns The signed request object.
-   */
-  async signRequest(webResource: WebResourceLike): Promise<WebResource> {
-    const verb = webResource.method.toUpperCase();
-    const utcNow = new Date().toUTCString();
-
-    const contentHash = await sha256Digest(webResource.body || "");
-
-    const signedHeaders = "x-ms-date;host;x-ms-content-sha256";
-
-    const url = URLBuilder.parse(webResource.url);
-    const query = url.getQuery();
-    const urlPathAndQuery = `${url.getPath()}${query ? "?" + query : ""}`;
-
-    const stringToSign = `${verb}\n${urlPathAndQuery}\n${utcNow};${url.getHost()};${contentHash}`;
-
-    const signature = await sha256Hmac(this.secret, stringToSign);
-
-    webResource.headers.set("x-ms-date", utcNow);
-    webResource.headers.set("x-ms-content-sha256", contentHash);
-    // Syntax for Authorization header
-    // Reference - https://docs.microsoft.com/en-us/azure/azure-app-configuration/rest-api-authentication-hmac#syntax
-    webResource.headers.set(
-      "Authorization",
-      `HMAC-SHA256 Credential=${this.credential}&SignedHeaders=${signedHeaders}&Signature=${signature}`
-    );
-    return webResource;
-  }
+export function createAppConfigKeyCredentialPolicy(
+  credential: string,
+  secret: string
+): PipelinePolicy {
+  return {
+    name: "AppConfigKeyCredentialPolicy",
+    async sendRequest(request: PipelineRequest, next: SendRequest): Promise<PipelineResponse> {
+      const verb = request.method.toUpperCase();
+      const utcNow = new Date().toUTCString();
+      const contentHash = await sha256Digest(request.body?.toString() || "");
+      const signedHeaders = "x-ms-date;host;x-ms-content-sha256";
+      const url = new URL(request.url);
+      const query = url.search;
+      const urlPathAndQuery = query ? `${url.pathname}${query}` : url.pathname;
+      const stringToSign = `${verb}\n${urlPathAndQuery}\n${utcNow};${url.host};${contentHash}`;
+      const signature = await sha256Hmac(secret, stringToSign);
+
+      request.headers.set("x-ms-date", utcNow);
+      request.headers.set("x-ms-content-sha256", contentHash);
+      // Syntax for Authorization header
+      // Reference - https://docs.microsoft.com/en-us/azure/azure-app-configuration/rest-api-authentication-hmac#syntax
+      request.headers.set(
+        "Authorization",
+        `HMAC-SHA256 Credential=${credential}&SignedHeaders=${signedHeaders}&Signature=${signature}`
+      );
+
+      return next(request);
+    },
+  };
 }