fix: better sanitize fetched recording name

beryxz · Oct 2, 2021 · 2149a7d · 2149a7d
1 parent dbd79f7
commit 2149a7d
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/app.js b/app.js
@@ -155,7 +155,7 @@ async function processCourseRecordings(course, recordings, downloadConfigs) {
 
         for (const recording of chunk) {
             // filename
-            let filename = `${recording.name}.${recording.format}`.replace(/[\\/:"*?<>| ]/g, '_');
+            let filename = `${recording.name}.${recording.format}`.replace(/[\\/:"*?<>| \r\n]/g, '_');
             if (course.prepend_date)
                 filename = `${getUTCDateTimestamp(recording.created_at, '')}-${filename}`;
 
@@ -164,6 +164,7 @@ async function processCourseRecordings(course, recordings, downloadConfigs) {
                 downloadConfigs.base_path,
                 course.name ? `${course.name}_${course.id}` : `${course.id}`
             );
+
             let downloadPath = join(folderPath, filename);
             let tmpDownloadPath = join('./tmp/', filename);
             try {