videoshow icon indicating copy to clipboard operation
videoshow copied to clipboard

UUID issue

Open heavell7870 opened this issue 2 years ago • 0 comments

Hi! 👋

Firstly, thanks for your work on this project! 🙂

Today I used patch-package to patch [email protected] for the project I'm working on.

Here is the diff that solved my problem:

diff --git a/node_modules/videoshow/lib/render.js b/node_modules/videoshow/lib/render.js
index 6fc8054..ab42696 100644
--- a/node_modules/videoshow/lib/render.js
+++ b/node_modules/videoshow/lib/render.js
@@ -2,7 +2,7 @@ var fs = require('fs')
 var fw = require('fw')
 var os = require('os')
 var path = require('path')
-var uuid = require('lil-uuid')
+const { v4: uuidv4 } = require('uuid');
 var union = require('lodash.merge')
 var ffmpeg = require('fluent-ffmpeg')
 var EventEmitter = require('events').EventEmitter
@@ -320,5 +320,5 @@ function remove(file, cb) {
 }
 
 function randomName() {
-  return path.join(os.tmpdir(), 'videoshow-' + uuid())
+  return path.join(os.tmpdir(), 'videoshow-' + uuidv4())
 }

This issue body was partially generated by patch-package.

heavell7870 avatar Jul 21 '23 11:07 heavell7870