@zero-server/fetch 1.0.2 → 1.1.0

package/lib/fetch/index.js

@@ -10,13 +10,24 @@ const { URL } = require('url');
 
 const STATUS_CODES = http.STATUS_CODES;
 
+/**
+ * Default `User-Agent` sent when a request does not specify one.
+ * Many APIs (e.g. GitHub) reject requests without a User-Agent, so a sensible
+ * default avoids surprising 403s. Override per request via `opts.headers`.
+ * @private
+ */
+let DEFAULT_USER_AGENT;
+try { DEFAULT_USER_AGENT = `zero-server/${require('../../package.json').version}`; }
+catch (e) { DEFAULT_USER_AGENT = 'zero-server'; }
+
 /**
  * Perform an HTTP(S) request.
  *
  * @param {string} url  - Absolute URL to fetch.
  * @param {object} [opts] - Configuration options.
  * @param {string}   [opts.method='GET']           - HTTP method.
- * @param {object}   [opts.headers]                - Request headers.
+ * @param {object}   [opts.headers]                - Request headers. A default
+ *        `User-Agent` of `zero-server/<version>` is sent unless one is provided here.
  * @param {string|Buffer|object|ReadableStream} [opts.body] - Request body.
  * @param {number}   [opts.timeout]                - Request timeout in ms.
  * @param {AbortSignal} [opts.signal]              - Abort signal for cancellation.
@@ -59,6 +70,12 @@ function miniFetch(url, opts = {})
             const method = (opts.method || 'GET').toUpperCase();
             const headers = Object.assign({}, opts.headers || {});
 
+            // Send a default User-Agent unless the caller supplied one (any case).
+            if (!Object.keys(headers).some(h => h.toLowerCase() === 'user-agent'))
+            {
+                headers['User-Agent'] = DEFAULT_USER_AGENT;
+            }
+
             // Normalize body
             let body = opts.body;
             if (body && typeof body === 'object' && typeof body.toString === 'function' && body.constructor && body.constructor.name === 'URLSearchParams')

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zero-server/fetch",
-  "version": "1.0.2",
+  "version": "1.1.0",
   "description": "Server-side fetch with mTLS, timeouts, AbortSignal.",
   "keywords": [
     "zero-server",
@@ -45,7 +45,7 @@
   },
   "sideEffects": false,
   "peerDependencies": {
-    "@zero-server/sdk": ">=1.0.2"
+    "@zero-server/sdk": ">=1.1.0"
   },
   "peerDependenciesMeta": {
     "@zero-server/sdk": {