redact for all lines, add e2e test (#1912)

Author: lerouxb

packages/cli-repl/src/mongosh-repl.ts

@@ -451,7 +451,7 @@ class MongoshNodeRepl implements EvaluationListener {
       // repl.history is an array of previous commands. We need to hijack the
       // value we just typed, and shift it off the history array if the info is
       // sensitive.
-      repl.on('flushHistory', () => {
+      repl.on('line', () => {
         if (this.redactHistory !== 'keep') {
           const history: string[] = (repl as any).history;
           changeHistory(

packages/e2e-tests/test/e2e.spec.ts

@@ -1486,6 +1486,19 @@ describe('e2e', function () {
 
           expect((await fs.stat(historyPath)).mode & 0o077).to.equal(0);
         });
+
+        it('redacts secrets', async function () {
+          await shell.executeLine('db.auth("myusername", "mypassword")');
+          await shell.executeLine('a = 42');
+          await shell.executeLine('foo = "bar"');
+          shell.writeInput('.exit\n');
+          await shell.waitForExit();
+
+          const contents = await fs.readFile(historyPath, 'utf8');
+          expect(contents).to.not.match(/mypassword/);
+          expect(contents).to.match(/^a = 42$/m);
+          expect(contents).to.match(/^foo = "bar"$/m);
+        });
       });
 
       describe('mongoshrc', function () {