fix(utils): limit characters of env overrides to context (#585)

Author: patrickhulce

packages/utils/src/build-context.js

@@ -200,7 +200,7 @@ function getCommitMessage(hash = 'HEAD') {
     // GitLab CI
     'CI_COMMIT_MESSAGE',
   ]);
-  if (envHash) return envHash;
+  if (envHash) return envHash.trim().slice(0, 80);
 
   const result = childProcess.spawnSync('git', ['log', '--format=%s', '-n', '1', hash], {
     encoding: 'utf8',
@@ -226,7 +226,7 @@ function getAuthor(hash = 'HEAD') {
     // GitLab CI: https://gitlab.com/gitlab-org/gitlab/-/issues/284079
     'CI_COMMIT_AUTHOR',
   ]);
-  if (envHash) return envHash;
+  if (envHash) return envHash.trim().slice(0, 256);
 
   const result = childProcess.spawnSync('git', ['log', '--format=%aN <%aE>', '-n', '1', hash], {
     encoding: 'utf8',

packages/utils/test/build-context.test.js

@@ -86,6 +86,11 @@ describe('build-context.js', () => {
       process.env.LHCI_BUILD_CONTEXT__COMMIT_MESSAGE = 'Daily run';
       expect(buildContext.getCommitMessage(hash)).toEqual('Daily run');
     });
+
+    it('should limit the message to 80 characters', () => {
+      process.env.LHCI_BUILD_CONTEXT__COMMIT_MESSAGE = 'a'.repeat(1000);
+      expect(buildContext.getCommitMessage(hash)).toEqual('a'.repeat(80));
+    });
   });
 
   describe('#getAuthor()', () => {
@@ -97,6 +102,11 @@ describe('build-context.js', () => {
       process.env.LHCI_BUILD_CONTEXT__AUTHOR = 'Paul Irish <[email protected]>';
       expect(buildContext.getAuthor(hash)).toEqual('Paul Irish <[email protected]>');
     });
+
+    it('should limit the message to 256 characters', () => {
+      process.env.LHCI_BUILD_CONTEXT__AUTHOR = 'a'.repeat(1000);
+      expect(buildContext.getAuthor(hash)).toEqual('a'.repeat(256));
+    });
   });
 
   describe('#getEmailFromAuthor', () => {