fix: Check buffer is ready before promise in node

HazAT · HazAT · commit 7f793291cdf7 · 2019-03-22T10:54:13.000+01:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,9 @@
 # Changelog
 
+## 5.0.0-rc.1
+
+- [node] fix: Check if buffer isReady before sending/creating Promise for request.
+
 ## 5.0.0-rc.0
 
 - Fix: Tag npm release with `next` to not make it latest
diff --git a/packages/node/src/transports/base.ts b/packages/node/src/transports/base.ts
@@ -73,6 +73,9 @@ export abstract class BaseTransport implements Transport {
 
   /** JSDoc */
   protected async _sendWithModule(httpModule: HTTPRequest, event: Event): Promise<Response> {
+    if (!this._buffer.isReady()) {
+      return Promise.reject(new SentryError('Not adding Promise due to buffer limit reached.'));
+    }
     return this._buffer.add(
       new Promise<Response>((resolve, reject) => {
         const req = httpModule.request(this._getRequestOptions(), (res: http.IncomingMessage) => {
