Fix/prevent retry for non retryable 404 status code (#488)

mohitpubnub · Mohit Tejani · parfeon · web-flow · commit 32cc0bdfac26 · 2026-01-13T15:23:29.000+05:30
* exclude 404 from retryable

* update branch

* refactor(error): add error description for HTTP 404 status code

* test: appcontext response with 404

* fix lint issue

* text enhancement to check retry would not take place, add dist/lib updated

* refactor: test, to verify retry not happening on 404

* PubNub SDK v10.2.6 release.

---------

Co-authored-by: Mohit Tejani &lt;mohit.tejani@Mohits-MacBook-Pro.local&gt;
Co-authored-by: Serhii Mamontov &lt;parfeon@me.com&gt;
Co-authored-by: PubNub Release Bot &lt;120067856+pubnub-release-bot@users.noreply.github.com&gt;
diff --git a/.pubnub.yml b/.pubnub.yml
@@ -1,5 +1,10 @@
 ---
 changelog:
+  - date: 2026-01-13
+    version: v10.2.6
+    changes:
+      - type: improvement
+        text: "Prevent retry when response is having http status code 404."
   - date: 2025-12-16
     version: v10.2.5
     changes:
@@ -1379,7 +1384,7 @@ supported-platforms:
       - 'Ubuntu 14.04 and up'
       - 'Windows 7 and up'
     version: 'Pubnub Javascript for Node'
-version: '10.2.5'
+version: '10.2.6'
 sdks:
   - full-name: PubNub Javascript SDK
     short-name: Javascript
@@ -1395,7 +1400,7 @@ sdks:
           - distribution-type: source
             distribution-repository: GitHub release
             package-name: pubnub.js
-            location: https://github.com/pubnub/javascript/archive/refs/tags/v10.2.5.zip
+            location: https://github.com/pubnub/javascript/archive/refs/tags/v10.2.6.zip
             requires:
               - name: 'agentkeepalive'
                 min-version: '3.5.2'
@@ -2066,7 +2071,7 @@ sdks:
           - distribution-type: library
             distribution-repository: GitHub release
             package-name: pubnub.js
-            location: https://github.com/pubnub/javascript/releases/download/v10.2.5/pubnub.10.2.5.js
+            location: https://github.com/pubnub/javascript/releases/download/v10.2.6/pubnub.10.2.6.js
             requires:
               - name: 'agentkeepalive'
                 min-version: '3.5.2'
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,9 @@
+## v10.2.6
+January 13 2026
+
+#### Modified
+- Prevent retry when response is having http status code 404.
+
 ## v10.2.5
 December 16 2025
 
diff --git a/README.md b/README.md
@@ -27,8 +27,8 @@ Watch [Getting Started with PubNub JS SDK](https://app.dashcam.io/replay/64ee0d2
      npm install pubnub
      ```
    * or download one of our builds from our CDN: 
-     * https://cdn.pubnub.com/sdk/javascript/pubnub.10.2.5.js
-     * https://cdn.pubnub.com/sdk/javascript/pubnub.10.2.5.min.js
+     * https://cdn.pubnub.com/sdk/javascript/pubnub.10.2.6.js
+     * https://cdn.pubnub.com/sdk/javascript/pubnub.10.2.6.min.js
 
 2. Configure your keys:
 
diff --git a/dist/web/pubnub.js b/dist/web/pubnub.js
@@ -3118,6 +3118,8 @@
 	        data !== null && data !== void 0 ? data : (data = response.body);
 	        if (status === 402)
 	            message = 'Not available for used key set. Contact support@pubnub.com';
+	        else if (status === 404)
+	            message = 'Resource not found';
 	        else if (status === 400) {
 	            category = StatusCategory$1.PNBadRequestCategory;
 	            message = 'Bad request';
@@ -5436,7 +5438,7 @@
 	            return base.PubNubFile;
 	        },
 	        get version() {
-	            return '10.2.5';
+	            return '10.2.6';
 	        },
 	        getVersion() {
 	            return this.version;
@@ -5790,7 +5792,7 @@
 	                    activeCancellation = attemptCancellation;
 	                    const responseHandler = (res, error) => {
 	                        const retriableError = error ? error.category !== StatusCategory$1.PNCancelledCategory : true;
-	                        const retriableStatusCode = !res || res.status >= 400;
+	                        const retriableStatusCode = (!res || res.status >= 400) && (error === null || error === void 0 ? void 0 : error.statusCode) !== 404;
 	                        let delay = -1;
 	                        if (retriableError &&
 	                            retriableStatusCode &&
diff --git a/dist/web/pubnub.min.js b/dist/web/pubnub.min.js
diff --git a/lib/core/components/configuration.js b/lib/core/components/configuration.js
@@ -168,7 +168,7 @@ const makeConfiguration = (base, setupCryptoModule) => {
             return base.PubNubFile;
         },
         get version() {
-            return '10.2.5';
+            return '10.2.6';
         },
         getVersion() {
             return this.version;
diff --git a/lib/errors/pubnub-api-error.js b/lib/errors/pubnub-api-error.js
@@ -125,6 +125,8 @@ class PubNubAPIError extends Error {
         data !== null && data !== void 0 ? data : (data = response.body);
         if (status === 402)
             message = 'Not available for used key set. Contact support@pubnub.com';
+        else if (status === 404)
+            message = 'Resource not found';
         else if (status === 400) {
             category = categories_1.default.PNBadRequestCategory;
             message = 'Bad request';
diff --git a/lib/transport/middleware.js b/lib/transport/middleware.js
@@ -125,7 +125,7 @@ class PubNubMiddleware {
                     activeCancellation = attemptCancellation;
                     const responseHandler = (res, error) => {
                         const retriableError = error ? error.category !== categories_1.default.PNCancelledCategory : true;
-                        const retriableStatusCode = !res || res.status >= 400;
+                        const retriableStatusCode = (!res || res.status >= 400) && (error === null || error === void 0 ? void 0 : error.statusCode) !== 404;
                         let delay = -1;
                         if (retriableError &&
                             retriableStatusCode &&
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "pubnub",
-  "version": "10.2.5",
+  "version": "10.2.6",
   "author": "PubNub <support@pubnub.com>",
   "description": "Publish & Subscribe Real-time Messaging with PubNub",
   "scripts": {
diff --git a/src/core/components/configuration.ts b/src/core/components/configuration.ts
@@ -236,7 +236,7 @@ export const makeConfiguration = (
       return base.PubNubFile;
     },
     get version(): string {
-      return '10.2.5';
+      return '10.2.6';
     },
     getVersion(): string {
       return this.version;
diff --git a/src/errors/pubnub-api-error.ts b/src/errors/pubnub-api-error.ts
@@ -102,6 +102,7 @@ export class PubNubAPIError extends Error {
     data ??= response.body;
 
     if (status === 402) message = 'Not available for used key set. Contact support@pubnub.com';
+    else if (status === 404) message = 'Resource not found';
     else if (status === 400) {
       category = StatusCategory.PNBadRequestCategory;
       message = 'Bad request';
diff --git a/src/transport/middleware.ts b/src/transport/middleware.ts
@@ -173,7 +173,7 @@ export class PubNubMiddleware implements Transport {
 
           const responseHandler = (res?: TransportResponse, error?: PubNubAPIError) => {
             const retriableError = error ? error.category !== StatusCategory.PNCancelledCategory : true;
-            const retriableStatusCode = !res || res.status >= 400;
+            const retriableStatusCode = (!res || res.status >= 400) && error?.statusCode !== 404;
             let delay = -1;
 
             if (
diff --git a/test/integration/endpoints/objects/channel.test.ts b/test/integration/endpoints/objects/channel.test.ts
@@ -4,6 +4,7 @@ import nock from 'nock';
 import { asResponse, allChannels, channel1 } from './fixtures.js';
 import PubNub from '../../../../src/node/index';
 import utils from '../../../utils';
+import { RetryPolicy } from '../../../../src/core/components/retry-policy.js';
 
 describe('objects channel', () => {
   const PUBLISH_KEY = 'myPublishKey';
@@ -727,4 +728,62 @@ describe('objects channel', () => {
       expect(lastResult.data).to.have.length.lessThan(10);
     });
   });
+  describe('retry policy', () => {
+    it('should not retry on 404 Not Found error with linear retry policy', async () => {
+      const channelName = 'non-existent-channel';
+      
+      // Set up nock mock before creating PubNub instance
+      const scope = utils
+        .createNock()
+        .get(`/v2/objects/${SUBSCRIBE_KEY}/channels/${channelName}`)
+        .times(1)
+        .query(true)
+        .reply(404, {
+          status: 404,
+          error: {
+            message: 'Requested object was not found.',
+            source: 'objects',
+          },
+        });
+
+      // Create a new PubNub instance with linear retry policy
+      const pubnubWithRetry = new PubNub({
+        subscribeKey: SUBSCRIBE_KEY,
+        publishKey: PUBLISH_KEY,
+        uuid: UUID,
+        // @ts-expect-error Force override default value.
+        useRequestId: false,
+        authKey: AUTH_KEY,
+        retryConfiguration: RetryPolicy.LinearRetryPolicy({
+          delay: 2,
+          maximumRetry: 2,
+        }),
+      });
+
+      let caughtError: any;
+      try {
+        await pubnubWithRetry.objects.getChannelMetadata({ channel: channelName });
+      } catch (error) {
+        caughtError = error;
+      }
+      
+      // Verify that an error was thrown
+      expect(caughtError).to.exist;
+      
+      // Verify the error status code is 404
+      expect(caughtError).to.have.property('status');
+      expect(caughtError.status.statusCode).to.equal(404);
+      
+      // Verify the error message
+      expect(caughtError.status.errorData).to.exist;
+      expect(caughtError.status.errorData.error).to.exist;
+      expect(caughtError.status.errorData.error.message).to.equal('Requested object was not found.');
+      expect(caughtError.status.errorData.error.source).to.equal('objects');
+      
+      // Verify the scope was called exactly once (no retries on 404
+      expect(scope.isDone()).to.be.true;
+      
+      pubnubWithRetry.destroy(true);
+    });
+  });
 });
diff --git a/test/integration/endpoints/objects/uuid.test.ts b/test/integration/endpoints/objects/uuid.test.ts
@@ -3,6 +3,7 @@ import nock from 'nock';
 
 import { asResponse, allUsers, user1 } from './fixtures';
 import PubNub from '../../../../src/node/index';
+import { RetryPolicy } from '../../../../src/core/components/retry-policy.js';
 import utils from '../../../utils';
 
 describe('objects UUID', () => {
@@ -322,4 +323,63 @@ describe('objects UUID', () => {
       });
     });
   });
+
+  describe('retry policy', () => {
+    it('should not retry on 404 Not Found error with linear retry policy', async () => {
+      const nonExistentUUID = 'non-existent-uuid';
+
+      // Set up nock mock before creating PubNub instance
+      const scope = utils
+        .createNock()
+        .get(`/v2/objects/${SUBSCRIBE_KEY}/uuids/${nonExistentUUID}`)
+        .times(1)
+        .query(true)
+        .reply(404, {
+          status: 404,
+          error: {
+            message: 'Requested object was not found.',
+            source: 'objects',
+          },
+        });
+
+      // Create a new PubNub instance with linear retry policy
+      const pubnubWithRetry = new PubNub({
+        subscribeKey: SUBSCRIBE_KEY,
+        publishKey: PUBLISH_KEY,
+        uuid: UUID,
+        // @ts-expect-error Force override default value.
+        useRequestId: false,
+        authKey: AUTH_KEY,
+        retryConfiguration: RetryPolicy.LinearRetryPolicy({
+          delay: 2,
+          maximumRetry: 2,
+        }),
+      });
+
+      let caughtError: any;
+      try {
+        await pubnubWithRetry.objects.getUUIDMetadata({ uuid: nonExistentUUID });
+      } catch (error) {
+        caughtError = error;
+      }
+
+      // Verify that an error was thrown
+      expect(caughtError).to.exist;
+
+      // Verify the error status code is 404
+      expect(caughtError).to.have.property('status');
+      expect(caughtError.status.statusCode).to.equal(404);
+
+      // Verify the error message
+      expect(caughtError.status.errorData).to.exist;
+      expect(caughtError.status.errorData.error).to.exist;
+      expect(caughtError.status.errorData.error.message).to.equal('Requested object was not found.');
+      expect(caughtError.status.errorData.error.source).to.equal('objects');
+
+      // Verify the scope was called exactly once (no retries on 404)
+      expect(scope.isDone()).to.be.true;
+
+      pubnubWithRetry.destroy(true);
+    });
+  });
 });

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "pubnub",`
`3`		`- "version": "10.2.5",`
	`3`	`+ "version": "10.2.6",`
`4`	`4`	`"author": "PubNub <support@pubnub.com>",`
`5`	`5`	`"description": "Publish & Subscribe Real-time Messaging with PubNub",`
`6`	`6`	`"scripts": {`