crash if trying to set path to / (#5003)

* crash if trying to set path to / fixes #4271 * Update packages/@uppy/companion/src/config/companion.js
transloadit · Mar 19, 2024 · 4f2abc5 · 4f2abc5
1 parent 48f4f34
commit 4f2abc5
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/packages/@uppy/companion/src/config/companion.js b/packages/@uppy/companion/src/config/companion.js
@@ -86,7 +86,13 @@ const validateConfig = (companionOptions) => {
     )
   }
 
-  const { providerOptions, periodicPingUrls } = companionOptions
+  const { providerOptions, periodicPingUrls, server } = companionOptions
+
+  if (server && server.path) {
+    // see https://github.com/transloadit/uppy/issues/4271
+    // todo fix the code so we can allow `/`
+    if (server.path === '/') throw new Error('server.path cannot be set to /')
+  }
 
   if (providerOptions) {
     const deprecatedOptions = { microsoft: 'providerOptions.onedrive', google: 'providerOptions.drive', s3: 's3' }