Default the glob options follow & strict to true, with new options to override

packages/workbox-build/src/_types.js

@@ -132,6 +132,19 @@ import './_version.mjs';
  *
  * E.g. `['**\/ignored.html']`
  *
+ * @property {Boolean} [globFollow=true] Follow symlinked directories when
+ * expanding ** patterns. Note that this can result in a lot of duplicate
+ * references in the presence of cyclic links.
+ *
+ * E.g. `true`
+ *
+ * @property {Boolean} [globStrict=true] When an unusual error is encountered
+ * when attempting to read a directory, the process will just continue on in
+ * search of other matches. Set the strict option to raise an error in these
+ * cases.
+ *
+ * E.g. `true`
+ *
  * @property {Object<String,Array|string>} [templatedUrls]
  * If a URL is rendered generated based on some server-side logic, its contents
  * may depend on multiple files or on some other unique string value.

packages/workbox-build/src/lib/get-file-details.js

@@ -21,12 +21,22 @@ const errors = require('./errors');
 const getFileSize = require('./get-file-size');
 const getFileHash = require('./get-file-hash');
 
-module.exports = ({globDirectory, globIgnores, globPattern}) => {
+module.exports = (globOptions) => {
+  const {
+    globDirectory,
+    globIgnores,
+    globPattern,
+    globFollow,
+    globStrict,
+  } = globOptions;
   let globbedFiles;
   try {
     globbedFiles = glob.sync(globPattern, {
       cwd: globDirectory,
       ignore: globIgnores,
+      follow: typeof globFollow !== 'undefined'? globFollow : true,
+      strict: typeof globStrict !== 'undefined'? globStrict : true,
+
     });
   } catch (err) {
     throw new Error(errors['unable-to-glob-files'] + ` '${err.message}'`);

test/workbox-build/node/lib/get-file-details.js

@@ -88,4 +88,46 @@ describe(`[workbox-build] lib/get-file-details.js`, function() {
       size: SIZE,
     }]);
   });
+
+  it(`should call sync with follow and strict options by default`, function() {
+    const getFileDetails = proxyquire(MODULE_PATH, {
+      'glob': {
+        sync: (pattern, options) => {
+          expect(options.follow).to.be.true;
+          expect(options.strict).to.be.true;
+
+          return [FILE1];
+        },
+      },
+      './get-file-size': (value) => SIZE,
+      './get-file-hash': (value) => HASH,
+    });
+
+    getFileDetails({
+      globDirectory: GLOB_DIRECTORY,
+      globPattern: GLOB_PATTERN,
+    });
+  });
+
+  it(`should call sync with follow and strict options with value`, function() {
+    const getFileDetails = proxyquire(MODULE_PATH, {
+      'glob': {
+        sync: (pattern, options) => {
+          expect(options.follow).to.be.false;
+          expect(options.strict).to.be.false;
+
+          return [FILE1];
+        },
+      },
+      './get-file-size': (value) => SIZE,
+      './get-file-hash': (value) => HASH,
+    });
+
+    getFileDetails({
+      globDirectory: GLOB_DIRECTORY,
+      globPattern: GLOB_PATTERN,
+      globFollow: false,
+      globStrict: false,
+    });
+  });
 });