@vpalmisano/webrtcperf 4.7.9 → 4.8.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vpalmisano/webrtcperf",
-  "version": "4.7.9",
+  "version": "4.8.0",
   "repository": {
     "type": "git",
     "url": "https://github.com/vpalmisano/webrtcperf.git"
@@ -53,14 +53,14 @@
   "dependencies": {
     "@google/genai": "^1.50.1",
     "@modelcontextprotocol/sdk": "^1.29.0",
-    "@puppeteer/browsers": "^2.13.0",
+    "@puppeteer/browsers": "^3.0.4",
     "@vpalmisano/throttler": "0.0.14",
-    "@vpalmisano/webrtcperf-js": "^1.2.11",
+    "@vpalmisano/webrtcperf-js": "^1.3.1",
     "axios": "^1.15.2",
     "chalk-template": "^1.1.2",
     "change-case": "^4.1.2",
     "compression": "^1.8.1",
-    "convict": "^6.2.4",
+    "convict": "^6.2.5",
     "convict-format-with-validator": "^6.2.0",
     "debug-level": "^4.1.1",
     "dockerode": "^4.0.10",
@@ -69,7 +69,7 @@
     "fast-stats": "^0.0.7",
     "form-data": "^4.0.4",
     "googleapis": "^164.1.0",
-    "ipaddr.js": "^2.2.0",
+    "ipaddr.js": "^2.4.0",
     "json5": "^2.2.3",
     "lorem-ipsum": "^2.0.8",
     "marked": "^16.3.0",
@@ -79,16 +79,16 @@
     "pidtree": "^0.6.0",
     "pidusage": "^4.0.1",
     "prom-client": "^15.1.3",
-    "puppeteer": "^24.42.0",
-    "puppeteer-core": "^24.42.0",
+    "puppeteer": "^25.1.0",
+    "puppeteer-core": "^25.1.0",
     "puppeteer-extra": "^3.3.6",
     "puppeteer-extra-plugin-stealth": "^2.11.2",
     "puppeteer-intercept-and-modify-requests": "^1.3.1",
     "sprintf-js": "^1.1.3",
     "tar-fs": "^3.1.1",
     "toml": "^3.0.0",
-    "ws": "^8.20.0",
-    "yaml": "^2.8.1",
+    "ws": "^8.21.0",
+    "yaml": "^2.9.0",
     "zod": "3"
   },
   "devDependencies": {

package/src/config.ts

@@ -767,6 +767,7 @@ alert will be successful only when at least 95% of the checks pass.`,
     doc: `The HTTP server basic auth secret. The auth user name is set to \`admin\` by default.`,
     format: String,
     default: 'secret',
+    nullable: true,
     env: 'SERVER_SECRET',
     arg: 'server-secret',
   },

package/src/server.ts

@@ -85,7 +85,7 @@ export class Server {
     )
 
     this.app.use((req: express.Request, res: express.Response, next: express.NextFunction) => {
-      if (req.query.auth === this.serverSecret) {
+      if (!this.serverSecret || req.query.auth === this.serverSecret) {
         return next()
       }
       const credentials = auth(req)
@@ -121,7 +121,7 @@ export class Server {
         log.error(`mkdir ${this.serverData} error: ${err.message}`)
       })
       this.app.get('/data', this.getDataArchive.bind(this))
-      this.app.get('/data/:path', this.getData.bind(this))
+      this.app.get('/data/*filepath', this.getData.bind(this))
     }
     if (this.videoCachePath) {
       log.debug(`using videoCachePath: ${this.videoCachePath}`)
@@ -432,7 +432,9 @@ export class Server {
    * content in tar.gz format.
    */
   private getData(req: express.Request, res: express.Response, next: express.NextFunction): void {
-    const paramPath = path.normalize(req.params.path).replace(/^(\.\.(\/|\\|$))+/, '')
+    const requestPath = req.params.filepath as string | string[]
+    const filepath = Array.isArray(requestPath) ? requestPath.join('/') : requestPath
+    const paramPath = path.normalize(filepath).replace(/^(\.\.(\/|\\|$))+/, '')
     log.debug(`GET /data/${paramPath}`, req.query)
     const fpath = path.resolve(this.serverData, paramPath)
     if (!fs.existsSync(fpath)) {
@@ -441,6 +443,9 @@ export class Server {
     if (req.query.range && !req.headers.range) {
       req.headers.range = `bytes=${req.query.range}`
     }
+    if (['.js', '.mjs', '.cjs'].includes(path.extname(fpath).toLowerCase())) {
+      res.type('application/javascript')
+    }
     res.sendFile(fpath, { dotfiles: 'allow' })
   }
 

package/src/session.ts

@@ -727,7 +727,8 @@ export class Session extends EventEmitter {
       ]
 
       log.debug(`[session ${this.id}] Using args:\n  ${args.join('\n  ')}`)
-      log.debug(`[session ${this.id}] Default args:\n  ${puppeteer.defaultArgs().join('\n  ')}`)
+      const defaultArgs = await puppeteer.defaultArgs()
+      log.debug(`[session ${this.id}] Default args:\n  ${defaultArgs.join('\n  ')}`)
 
       try {
         this.browser = await puppeteer.launch({
@@ -1670,10 +1671,10 @@ mv ${logFilePath}.tmp ${logFilePath};
         collectedStats.usedCpu = systemStats.usedCpu
         collectedStats.usedMemory = systemStats.usedMemory
         collectedStats.usedGpu = systemStats.usedGpu
-        if (collectedStats.usedCpu > 80) {
+        if (collectedStats.usedCpu > 90) {
           log.warn(`High system CPU usage: ${collectedStats.usedCpu.toFixed(2)}%`)
         }
-        if (collectedStats.usedMemory > 80) {
+        if (collectedStats.usedMemory > 90) {
           log.warn(`High system memory usage: ${collectedStats.usedMemory.toFixed(2)}%`)
         }
       }