feat: add validation of the prefix

Author: gagik

packages/mongodb-log-writer/src/mongo-log-manager.spec.ts

@@ -29,6 +29,48 @@ describe('MongoLogManager', function () {
     await fs.rmdir(directory, { recursive: true });
   });
 
+  it('constructor throws with invalid prefixes', function () {
+    expect(() => {
+      new MongoLogManager({
+        directory,
+        retentionDays,
+        prefix: '%asdabs/',
+        onwarn,
+        onerror,
+      });
+    }).to.throw();
+
+    expect(() => {
+      new MongoLogManager({
+        directory,
+        retentionDays,
+        prefix: '$$$$',
+        onwarn,
+        onerror,
+      });
+    }).to.throw();
+
+    expect(() => {
+      new MongoLogManager({
+        directory,
+        retentionDays,
+        prefix: 'abc_',
+        onwarn,
+        onerror,
+      });
+    }).not.to.throw();
+
+    expect(() => {
+      new MongoLogManager({
+        directory,
+        retentionDays,
+        prefix: 'something',
+        onwarn,
+        onerror,
+      });
+    }).not.to.throw();
+  });
+
   it('allows creating and writing to log files', async function () {
     const manager = new MongoLogManager({
       directory,

packages/mongodb-log-writer/src/mongo-log-manager.ts

@@ -36,6 +36,13 @@ export class MongoLogManager {
   _options: MongoLogOptions;
 
   constructor(options: MongoLogOptions) {
+    if (options.prefix) {
+      if (!/^[a-z0-9_]+$/i.test(options.prefix)) {
+        throw new Error(
+          'Prefix must only contain letters, numbers, and underscores'
+        );
+      }
+    }
     this._options = options;
   }
 
@@ -69,7 +76,6 @@ export class MongoLogManager {
       ? this._options.logRetentionGB * 1024 * 1024 * 1024
       : Infinity;
     let usedStorageSize = this._options.logRetentionGB ? 0 : -Infinity;
-    // eslint-disable-next-line no-console
 
     for await (const dirent of dirHandle) {
       // Cap the overall time spent inside this function. Consider situations like