Fix #10

jacopo-j · Jun 30, 2021 · 22e9d41 · 22e9d41
1 parent e9a6acc
commit 22e9d41
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/src/content.js b/src/content.js
@@ -91,6 +91,11 @@ function composeDownloadURL(params, filename) {
     return url;
 }
 
+function sanitizeFilename(filename) {
+    const allowedChars = /[^\w\s\d\-_~,;\[\]\(\).]/g;
+    return filename.replaceAll(allowedChars, "_");
+}
+
 /**
  * Callback used by a MutationObserver object in a
  * WebEx page containing a registration to download.
@@ -140,7 +145,7 @@ function addDownloadButtonToViewer(text, params) {
     const filename = data.getElementsByTagName("Sequence")[0].textContent;
 
     // Set the recording name as the save name
-    const savename = `${params.recordName}.mp4`;
+    const savename = `${sanitizeFilename(params.recordName)}.mp4`;
 
     // Compose the download link of the video
     const downloadURL = composeDownloadURL(params, filename);

diff --git a/src/manifest.json b/src/manifest.json
@@ -1,6 +1,6 @@
 {
   "description": "Enables downloading of Webex meeting recordings",
-  "version": "1.3",
+  "version": "1.3.1",
   "permissions": [
     "downloads",
     "storage",