fix(collector): resolved PayloadTooLargeError (#1985)

Author: kirrg001

packages/collector/src/agentConnection.js

@@ -22,7 +22,10 @@ let pidStore;
 // file descriptor fields in the collector announce cycle.
 const paddingForInodeAndFileDescriptor = 200;
 
-const maxContentLength = 1024 * 1024 * 5;
+// NOTE: The agent/sensor has a limit of 49mb.
+//       We had 5mb for a few years, but we were running into PayloadTooLargeError for
+//       profiles. We increase it to 20mb and see if that helps.
+const maxContentLength = 1024 * 1024 * 20;
 let maxContentErrorHasBeenLogged = false;
 
 const http = uninstrumentedHttp.http;

packages/collector/test/agentConnection_test.js

@@ -119,4 +119,34 @@ mochaSuiteFn('agent connection', function () {
         .catch(e => done(e));
     });
   });
+
+  it('should throw error if size of payload exceeds maxContentLength', done => {
+    const bigData = 'a'.repeat(1024 * 1024 * 21);
+    const bigSpan = {
+      n: 'span.exit',
+      t: 'trace-id',
+      p: 'entry-span-id',
+      s: 'exit-span-id',
+      k: constants.ENTRY,
+      data: {
+        bigData
+      }
+    };
+    const spans = [bigSpan];
+
+    agentConnection.sendSpans(spans, err => {
+      expect(err.message).to.match(/Request payload is too large/);
+      done();
+    });
+  });
+
+  it('should throw error if size of payload exceeds maxContentLength for profiles', done => {
+    const bigData = 'a'.repeat(1024 * 1024 * 21);
+    const profiles = { profile: bigData };
+
+    agentConnection.sendProfiles(profiles, err => {
+      expect(err.message).to.match(/Request payload is too large/);
+      done();
+    });
+  });
 });