@azure/data-tables 12.2.0-alpha.20211027.1 → 13.0.0-alpha.20211109.5

package/CHANGELOG.md

@@ -1,17 +1,24 @@
 # Release History
 
-## 12.2.0 (Unreleased)
+## 13.0.0 (2021-11-09)
 
 ### Features Added
-- Take `continuationToken` as a `PageSetting` and expose it in the page when listing entities `byPage`. [#18179](https://github.com/Azure/azure-sdk-for-js/pull/18179)
+
+- TableClient `listEntities` expose and can take as PageSetting `continuationToken` as a `PageSetting` when using `byPage`. [#18179](https://github.com/Azure/azure-sdk-for-js/pull/18179)
+- TableServiceClient `listTables` expose and can take PageSetting `continuationToken` as a `PageSetting` when using `byPage`. [#18277](https://github.com/Azure/azure-sdk-for-js/pull/18277)
 
 ### Breaking Changes
 
+- Encode single quote where the partition/row key is used to format the URL - i.e. upsert, update and delete. For more details see Issue [#3356](https://github.com/Azure/azure-sdk/issues/3356). Fixed in [#18520](https://github.com/Azure/azure-sdk-for-js/pull/18520)
+- Setting a binary property on an entity without automatic type conversion takes raw string instead of Uint8Array [#18251](https://github.com/Azure/azure-sdk-for-js/pull/18251)
+
 ### Bugs Fixed
+
 - Document usage of SDK with Azurite. [#18211](https://github.com/Azure/azure-sdk-for-js/pull/18211)
 - Issue #17407 - Correctly handle etag in select filter. [#18211](https://github.com/Azure/azure-sdk-for-js/pull/18211)
-- Issue #18079  - Correctly handle creating entities with properties containing empty strings "". Fixes Insert throws "Unknown EDM type object" error with property value { value: "", type: "String" }. [#18211](https://github.com/Azure/azure-sdk-for-js/pull/18211)
+- Issue #18079 - Correctly handle creating entities with properties containing empty strings "". Fixes Insert throws "Unknown EDM type object" error with property value { value: "", type: "String" }. [#18211](https://github.com/Azure/azure-sdk-for-js/pull/18211)
 - Issue #18148 - Correctly deserialize Decimal numbers checking for isSafeInteger. Fixes listEntities always returns an Int32 type for a value of "1.23456789012346e+24". [#18211](https://github.com/Azure/azure-sdk-for-js/pull/18211)
+- Issue #18521 - `upsertEntity` doesn't work with "" for partition or row keys. [#18586](https://github.com/Azure/azure-sdk-for-js/pull/18586)
 
 ### Other Changes
 

package/dist/index.js

@@ -3068,7 +3068,7 @@ class GeneratedClientContext extends coreClient.ServiceClient {
         const defaults = {
             requestContentType: "application/json; charset=utf-8"
         };
-        const packageDetails = `azsdk-js-data-tables/12.2.0`;
+        const packageDetails = `azsdk-js-data-tables/13.0.0`;
         const userAgentPrefix = options.userAgentOptions && options.userAgentOptions.userAgentPrefix
             ? `${options.userAgentOptions.userAgentPrefix} ${packageDetails}`
             : `${packageDetails}`;
@@ -3534,6 +3534,9 @@ class TableServiceClient {
             },
             byPage: (settings) => {
                 const pageOptions = Object.assign(Object.assign({}, options), { queryOptions: { top: settings === null || settings === void 0 ? void 0 : settings.maxPageSize } });
+                if (settings === null || settings === void 0 ? void 0 : settings.continuationToken) {
+                    pageOptions.continuationToken = settings.continuationToken;
+                }
                 return this.listTablesPage(pageOptions);
             }
         };
@@ -3542,10 +3545,10 @@ class TableServiceClient {
         return tslib.__asyncGenerator(this, arguments, function* listTablesAll_1() {
             var e_1, _a;
             const firstPage = yield tslib.__await(this._listTables(options));
-            const { nextTableName } = firstPage;
+            const { continuationToken } = firstPage;
             yield tslib.__await(yield* tslib.__asyncDelegator(tslib.__asyncValues(firstPage)));
-            if (nextTableName) {
-                const optionsWithContinuation = Object.assign(Object.assign({}, options), { nextTableName });
+            if (continuationToken) {
+                const optionsWithContinuation = Object.assign(Object.assign({}, options), { continuationToken });
                 try {
                     for (var _b = tslib.__asyncValues(this.listTablesPage(optionsWithContinuation)), _c; _c = yield tslib.__await(_b.next()), !_c.done;) {
                         const page = _c.value;
@@ -3568,8 +3571,8 @@ class TableServiceClient {
             try {
                 let result = yield tslib.__await(this._listTables(updatedOptions));
                 yield yield tslib.__await(result);
-                while (result.nextTableName) {
-                    const optionsWithContinuation = Object.assign(Object.assign({}, updatedOptions), { nextTableName: result.nextTableName });
+                while (result.continuationToken) {
+                    const optionsWithContinuation = Object.assign(Object.assign({}, updatedOptions), { continuationToken: result.continuationToken });
                     result = yield tslib.__await(this._listTables(optionsWithContinuation));
                     yield yield tslib.__await(result);
                 }
@@ -3583,9 +3586,10 @@ class TableServiceClient {
             }
         });
     }
-    async _listTables(options) {
-        const { xMsContinuationNextTableName: nextTableName, value = [] } = await this.table.query(options);
-        return Object.assign([...value], { nextTableName });
+    async _listTables(options = {}) {
+        const { continuationToken: nextTableName } = options, listOptions = tslib.__rest(options, ["continuationToken"]);
+        const { xMsContinuationNextTableName: continuationToken, value = [] } = await this.table.query(Object.assign(Object.assign({}, listOptions), { nextTableName }));
+        return Object.assign([...value], { continuationToken });
     }
     /**
      *
@@ -4120,14 +4124,11 @@ function serializeObject(obj) {
         obj.type === "Guid" ||
         obj.type === "Int32" ||
         obj.type === "Int64" ||
-        obj.type === "String") {
+        obj.type === "String" ||
+        obj.type === "Binary") {
         serializedValue.value = obj.value;
         serializedValue.type = Edm[obj.type];
     }
-    else if (obj.type === "Binary") {
-        serializedValue.value = base64Encode(obj.value);
-        serializedValue.type = Edm.Binary;
-    }
     else {
         throw new Error(`Unknown EDM type ${typeof obj.value}`);
     }
@@ -4319,6 +4320,36 @@ function decodeContinuationToken(encodedToken) {
     return continuationToken;
 }
 
+// Copyright (c) Microsoft Corporation.
+// Licensed under the MIT license.
+function escapeQuotesIfString(input, previous) {
+    let result = input;
+    if (typeof input === "string") {
+        result = escapeQuotes(input);
+        // check if we need to escape this literal
+        if (!previous.trim().endsWith("'")) {
+            result = `'${result}'`;
+        }
+    }
+    return result;
+}
+function escapeQuotes(input) {
+    return input.replace(/'/g, "''");
+}
+/**
+ * Escapes an odata filter expression to avoid errors with quoting string literals.
+ */
+function odata(strings, ...values) {
+    const results = [];
+    for (let i = 0; i < strings.length; i++) {
+        results.push(strings[i]);
+        if (i < values.length) {
+            results.push(escapeQuotesIfString(values[i], strings[i]));
+        }
+    }
+    return results.join("");
+}
+
 // Copyright (c) Microsoft Corporation.
 /**
  * A TableClient represents a Client to the Azure Tables service allowing you
@@ -4485,7 +4516,7 @@ class TableClient {
         }
         try {
             const _a = updatedOptions || {}, { disableTypeConversion, queryOptions } = _a, getEntityOptions = tslib.__rest(_a, ["disableTypeConversion", "queryOptions"]);
-            await this.table.queryEntitiesWithPartitionAndRowKey(this.tableName, partitionKey, rowKey, Object.assign(Object.assign({}, getEntityOptions), { queryOptions: serializeQueryOptions(queryOptions || {}), onResponse }));
+            await this.table.queryEntitiesWithPartitionAndRowKey(this.tableName, escapeQuotes(partitionKey), escapeQuotes(rowKey), Object.assign(Object.assign({}, getEntityOptions), { queryOptions: serializeQueryOptions(queryOptions || {}), onResponse }));
             const tableEntity = deserialize(parsedBody, disableTypeConversion !== null && disableTypeConversion !== void 0 ? disableTypeConversion : false);
             return tableEntity;
         }
@@ -4687,7 +4718,7 @@ class TableClient {
         try {
             const _a = updatedOptions || {}, { etag = "*" } = _a, rest = tslib.__rest(_a, ["etag"]);
             const deleteOptions = Object.assign({}, rest);
-            return await this.table.deleteEntity(this.tableName, partitionKey, rowKey, etag, deleteOptions);
+            return await this.table.deleteEntity(this.tableName, escapeQuotes(partitionKey), escapeQuotes(rowKey), etag, deleteOptions);
         }
         catch (e) {
             span.setStatus({ code: coreTracing.SpanStatusCode.ERROR, message: e.message });
@@ -4741,15 +4772,14 @@ class TableClient {
     options = {}) {
         const { span, updatedOptions } = createSpan(`TableClient-updateEntity-${mode}`, options);
         try {
-            if (!entity.partitionKey || !entity.rowKey) {
-                throw new Error("partitionKey and rowKey must be defined");
-            }
+            const partitionKey = escapeQuotes(entity.partitionKey);
+            const rowKey = escapeQuotes(entity.rowKey);
             const _a = updatedOptions || {}, { etag = "*" } = _a, updateEntityOptions = tslib.__rest(_a, ["etag"]);
             if (mode === "Merge") {
-                return await this.table.mergeEntity(this.tableName, entity.partitionKey, entity.rowKey, Object.assign({ tableEntityProperties: serialize(entity), ifMatch: etag }, updateEntityOptions));
+                return await this.table.mergeEntity(this.tableName, partitionKey, rowKey, Object.assign({ tableEntityProperties: serialize(entity), ifMatch: etag }, updateEntityOptions));
             }
             if (mode === "Replace") {
-                return await this.table.updateEntity(this.tableName, entity.partitionKey, entity.rowKey, Object.assign({ tableEntityProperties: serialize(entity), ifMatch: etag }, updateEntityOptions));
+                return await this.table.updateEntity(this.tableName, partitionKey, rowKey, Object.assign({ tableEntityProperties: serialize(entity), ifMatch: etag }, updateEntityOptions));
             }
             throw new Error(`Unexpected value for update mode: ${mode}`);
         }
@@ -4801,14 +4831,13 @@ class TableClient {
     options = {}) {
         const { span, updatedOptions } = createSpan(`TableClient-upsertEntity-${mode}`, options);
         try {
-            if (!entity.partitionKey || !entity.rowKey) {
-                throw new Error("partitionKey and rowKey must be defined");
-            }
+            const partitionKey = escapeQuotes(entity.partitionKey);
+            const rowKey = escapeQuotes(entity.rowKey);
             if (mode === "Merge") {
-                return await this.table.mergeEntity(this.tableName, entity.partitionKey, entity.rowKey, Object.assign({ tableEntityProperties: serialize(entity) }, updatedOptions));
+                return await this.table.mergeEntity(this.tableName, partitionKey, rowKey, Object.assign({ tableEntityProperties: serialize(entity) }, updatedOptions));
             }
             if (mode === "Replace") {
-                return await this.table.updateEntity(this.tableName, entity.partitionKey, entity.rowKey, Object.assign({ tableEntityProperties: serialize(entity) }, updatedOptions));
+                return await this.table.updateEntity(this.tableName, partitionKey, rowKey, Object.assign({ tableEntityProperties: serialize(entity) }, updatedOptions));
             }
             throw new Error(`Unexpected value for update mode: ${mode}`);
         }
@@ -4948,33 +4977,6 @@ class TableClient {
     }
 }
 
-// Copyright (c) Microsoft Corporation.
-// Licensed under the MIT license.
-function escapeQuotesIfString(input, previous) {
-    let result = input;
-    if (typeof input === "string") {
-        result = input.replace(/'/g, "''");
-        // check if we need to escape this literal
-        if (!previous.trim().endsWith("'")) {
-            result = `'${result}'`;
-        }
-    }
-    return result;
-}
-/**
- * Escapes an odata filter expression to avoid errors with quoting string literals.
- */
-function odata(strings, ...values) {
-    const results = [];
-    for (let i = 0; i < strings.length; i++) {
-        results.push(strings[i]);
-        if (i < values.length) {
-            results.push(escapeQuotesIfString(values[i], strings[i]));
-        }
-    }
-    return results.join("");
-}
-
 Object.defineProperty(exports, 'AzureNamedKeyCredential', {
     enumerable: true,
     get: function () {