Merge pull request #10 from blinkafrica/patch/x-request-id

patch: fix instances where x-request-id header does not exist in the request header
blinkafrica · Apr 9, 2024 · 0bd9479 · 0bd9479
2 parents 4c32ba3 + f98e377
commit 0bd9479
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 7 deletions.
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@blinkclaud/octobus",
-  "version": "0.3.1",
+  "version": "0.4.0",
   "description": "A toolkit for Blink HQ's microservices",
   "author": "Blink HQ",
   "private": false,

diff --git a/src/http/wrapper.ts b/src/http/wrapper.ts
@@ -136,13 +136,11 @@ export class RequestWrapper<T extends object> {
    * @param req source request if there's any
    */
   track(req?: Request) {
-    // make sure request ID exists for non-base requests
-    if (req && !req.headers['x-request-id']) {
-      throw new NoRequestIDError(this.request.url as string);
-    }
-
     Object.assign(this.request.headers as object, {
-      'X-Request-ID': !!req ? req.headers['x-request-id'] : v4(),
+      'X-Request-ID':
+        !!req && req.headers['x-request-id']
+          ? req.headers['x-request-id']
+          : v4(),
       'X-Origin-Service': this.service,
     });