@adobe/helix-onedrive-support 7.0.0 → 7.1.1

package/CHANGELOG.md

@@ -1,3 +1,25 @@
+## [7.1.1](https://github.com/adobe/helix-onedrive-support/compare/v7.1.0...v7.1.1) (2022-04-02)
+
+
+### Bug Fixes
+
+* **deps:** update dependency @adobe/helix-fetch to v3.0.8 ([0035641](https://github.com/adobe/helix-onedrive-support/commit/0035641545b9c4a0deef5de7f3e1bab0f961b0ff))
+
+# [7.1.0](https://github.com/adobe/helix-onedrive-support/compare/v7.0.1...v7.1.0) (2022-04-01)
+
+
+### Features
+
+* speed up fuzzyItem lookup ([#259](https://github.com/adobe/helix-onedrive-support/issues/259)) ([5963fa6](https://github.com/adobe/helix-onedrive-support/commit/5963fa6389329d0cf737f64724688593b9291923)), closes [#258](https://github.com/adobe/helix-onedrive-support/issues/258)
+
+## [7.0.1](https://github.com/adobe/helix-onedrive-support/compare/v7.0.0...v7.0.1) (2022-03-23)
+
+
+### Bug Fixes
+
+* detect *-my.sharepoint.com links ([#256](https://github.com/adobe/helix-onedrive-support/issues/256)) ([001f57d](https://github.com/adobe/helix-onedrive-support/commit/001f57d7d73fd760df86c5a18ac5f27bcf9f0cf5))
+* trigger release ([c8826a3](https://github.com/adobe/helix-onedrive-support/commit/c8826a38c69279593338b34918b2c305912225b1))
+
 # [7.0.0](https://github.com/adobe/helix-onedrive-support/compare/v6.2.2...v7.0.0) (2022-03-23)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adobe/helix-onedrive-support",
-  "version": "7.0.0",
+  "version": "7.1.1",
   "description": "Helix OneDrive Support",
   "main": "src/index.js",
   "types": "src/index.d.ts",
@@ -23,7 +23,7 @@
   },
   "homepage": "https://github.com/adobe/helix-onedrive-support#readme",
   "dependencies": {
-    "@adobe/helix-fetch": "3.0.7",
+    "@adobe/helix-fetch": "3.0.8",
     "adal-node": "https://github.com/adobe-rnd/azure-activedirectory-library-for-nodejs.git#adobe",
     "jose": "4.6.0"
   },
@@ -31,12 +31,12 @@
     "@adobe/eslint-config-helix": "1.3.2",
     "@semantic-release/changelog": "6.0.1",
     "@semantic-release/git": "10.0.1",
-    "ajv": "8.10.0",
+    "ajv": "8.11.0",
     "codecov": "3.8.3",
     "commitizen": "4.2.4",
     "cz-conventional-changelog": "3.3.0",
     "dotenv": "16.0.0",
-    "eslint": "8.11.0",
+    "eslint": "8.12.0",
     "eslint-plugin-header": "3.1.1",
     "eslint-plugin-import": "2.25.4",
     "jsdoc-to-markdown": "7.1.1",

package/src/OneDrive.d.ts

@@ -194,6 +194,8 @@ export declare class OneDrive extends EventEmitter {
 
   me(): Promise<GraphResult>;
 
+  initTenantFromShareLink(sharingUrl: string|URL): Promise<void>;
+
   resolveShareLink(sharingUrl: string|URL): Promise<GraphResult>;
 
   /**
@@ -225,16 +227,17 @@ export declare class OneDrive extends EventEmitter {
    * - replace all non-alphanumeric characters with a dash
    * - remove all consecutive dashes
    * - remove all leading and trailing dashes
-   * - extensions are ignored, if the given path doesn't have one
+   * - extensions are ignored, if the given path doesn't have one or if ignoreExtension is true
    *
    * The result is an array of drive items that match the given path. They are ordered by the edit
    * distance to the original name and then alphanumerically.
    *
    * @param folderItem
    * @param relPath
+   * @param ignoreExtension
    * @returns {Promise<DriveItem[]>}
    */
-  fuzzyGetDriveItem(folderItem: DriveItem, relPath?: string): Promise<DriveItem[]>;
+  fuzzyGetDriveItem(folderItem: DriveItem, relPath?: string, ignoreExtension?: boolean): Promise<DriveItem[]>;
 
   downloadDriveItem(driveItem: DriveItem): Promise<GraphResult>;
 

package/src/OneDrive.js

@@ -155,7 +155,14 @@ class OneDrive extends EventEmitter {
       return;
     }
     const { log } = this;
-    const [tenantHost] = new URL(sharingUrl).hostname.split('.');
+    const url = sharingUrl instanceof URL
+      ? sharingUrl
+      : new URL(sharingUrl);
+    let [tenantHost] = url.hostname.split('.');
+    // special case: `xxxx-my.sharepoint.com`
+    if (url.hostname.endsWith('-my.sharepoint.com')) {
+      tenantHost = tenantHost.substring(0, tenantHost.length - 3);
+    }
 
     if (this.tenantCache) {
       this.tenant = this.tenantCache.get(tenantHost);
@@ -495,22 +502,39 @@ class OneDrive extends EventEmitter {
    * - convert to lower case
    * - replace all non-alphanumeric characters with a dash
    * - remove all consecutive dashes
-   * - extensions are ignored, if the given path doesn't have one
+   * - extensions are ignored, if the given path doesn't have one or if ignoreExtension is true
    *
    * The result is an array of drive items that match the given path. They are ordered by the edit
    * distance to the original name and then alphanumerically.
    *
    * @param {DriveItem} folderItem
-   * @param {string} relPath
+   * @param {string} [relPath = '']
+   * @param {boolean} [ignoreExtension = false]
    * @returns {Promise<DriveItem[]>}
    */
-  async fuzzyGetDriveItem(folderItem, relPath = '') {
-    const idx = relPath.lastIndexOf('/');
-    if (idx < 0) {
-      const ret = await this.getDriveItem(folderItem, '', false);
-      // todo: add extra extension
-      return [ret.value];
+  async fuzzyGetDriveItem(folderItem, relPath = '', ignoreExtension = false) {
+    if (relPath && !relPath.startsWith('/')) {
+      throw new Error('relPath must be empty or start with /');
     }
+
+    // first try to get item directly
+    try {
+      const ret = await this.getDriveItem(folderItem, relPath, false);
+      if (relPath) {
+        // eslint-disable-next-line prefer-destructuring
+        ret.extension = splitByExtension(relPath)[1];
+      }
+      this.log.info(`fetched drive item directly: /drives/${folderItem.parentReference.driveId}/items/${folderItem.id}:${relPath}`);
+      return [ret];
+    } catch (e) {
+      this.log.info(`fetched drive item directly failed: /drives/${folderItem.parentReference.driveId}/items/${folderItem.id}:${relPath} (${e.statusCode})`);
+      // if no 404 or no relPath, propagate error
+      if (e.statusCode !== 404 || !relPath) {
+        throw e;
+      }
+    }
+
+    const idx = relPath.lastIndexOf('/');
     const folderRelPath = relPath.substring(0, idx);
     const name = relPath.substring(idx + 1);
     const [baseName, ext] = splitByExtension(name);
@@ -534,7 +558,7 @@ class OneDrive extends EventEmitter {
       }
     } while (query.$skiptoken);
 
-    this.log.debug(`loaded ${fileList.length} children from ${relPath}`);
+    this.log.info(`loaded ${fileList.length} children from /drives/${folderItem.parentReference.driveId}/items/${folderItem.id}:${relPath}`);
     const items = fileList.filter((item) => {
       if (!item.file) {
         return false;
@@ -543,7 +567,7 @@ class OneDrive extends EventEmitter {
       // remember extension
       // eslint-disable-next-line no-param-reassign
       item.extension = itemExt;
-      if (ext && ext !== itemExt) {
+      if (ext && ext !== itemExt && !ignoreExtension) {
         // only match extension if given via relPath
         return false;
       }

package/src/OneDriveMock.js

@@ -239,6 +239,9 @@ class OneDriveMock extends OneDrive {
     const url = new URL(`https://dummy.org${uri}`);
     if (url.pathname in this.driveItems) {
       const result = this.driveItems[url.pathname];
+      if (result instanceof Error) {
+        throw result;
+      }
       if (!Array.isArray(result.value)) {
         return result;
       }