@backstage/plugin-events-node - npm Package Compare versions

Comparing version 0.0.0-nightly-20241031023248 to 0.0.0-nightly-20241101023511

alpha/package.json

 {
   "name": "@backstage/plugin-events-node__alpha",
-  "version": "0.0.0-nightly-20241031023248",
+  "version": "0.0.0-nightly-20241101023511",
   "main": "../dist/alpha.cjs.js",
   "types": "../dist/alpha.d.ts"
 }

CHANGELOG.md

 # @backstage/plugin-events-node
 
-## 0.0.0-nightly-20241031023248
+## 0.0.0-nightly-20241101023511
 
@@ -12,4 +12,5 @@ ### Patch Changes
 - e02a02b: Fix `events.useEventBus` by propagating config to `DefaultEventsService`
+- 0b57aa1: Fixed an issue where the event bus polling would duplicate and increase exponentially over time.
 - Updated dependencies
-  - @backstage/backend-plugin-api@0.0.0-nightly-20241031023248
+  - @backstage/backend-plugin-api@0.0.0-nightly-20241101023511
   - @backstage/errors@1.2.4
@@ -16,0 +17,0 @@ - @backstage/types@1.1.1

dist/api/DefaultEventsService.cjs.js

@@ -138,17 +138,5 @@ 'use strict';
           );
-          if (!res.ok) {
-            if (res.status === 404) {
-              this.logger.info(
-                `Polling event subscription resulted in a 404, recreating subscription`
-              );
-              hasSubscription = false;
-            } else {
-              throw await errors.ResponseError.fromResponse(res);
-            }
-          }
-          backoffMs = POLL_BACKOFF_START_MS;
           if (res.status === 202) {
             lock.release();
-            await res.body?.getReader()?.closed;
-            process.nextTick(poll);
+            await res.text();
           } else if (res.status === 200) {
@@ -170,6 +158,11 @@ const data = await res.json();
               }
+            }
+          } else {
+            if (res.status === 404) {
+              this.logger.info(
+                `Polling event subscription resulted in a 404, recreating subscription`
+              );
+              hasSubscription = false;
             } else {
-              this.logger.warn(
-                `Unexpected response status ${res.status} from events backend for subscription "${subscriptionId}"`
-              );
+              throw await errors.ResponseError.fromResponse(res);
             }
@@ -198,2 +191,3 @@ }
         }
+        backoffMs = POLL_BACKOFF_START_MS;
         process.nextTick(poll);
@@ -200,0 +194,0 @@ } catch (error) {

package.json

 {
   "name": "@backstage/plugin-events-node",
-  "version": "0.0.0-nightly-20241031023248",
+  "version": "0.0.0-nightly-20241101023511",
   "description": "The plugin-events-node module for @backstage/plugin-events-backend",
@@ -54,3 +54,3 @@ "backstage": {
   "dependencies": {
-    "@backstage/backend-plugin-api": "0.0.0-nightly-20241031023248",
+    "@backstage/backend-plugin-api": "0.0.0-nightly-20241101023511",
     "@backstage/errors": "1.2.4",
@@ -63,4 +63,4 @@ "@backstage/types": "1.1.1",
     "@backstage/backend-common": "^0.25.0",
-    "@backstage/backend-test-utils": "0.0.0-nightly-20241031023248",
-    "@backstage/cli": "0.0.0-nightly-20241031023248",
+    "@backstage/backend-test-utils": "0.0.0-nightly-20241101023511",
+    "@backstage/cli": "0.0.0-nightly-20241101023511",
     "msw": "^1.0.0"
@@ -67,0 +67,0 @@ },

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Worsened metrics

Total package byte prevSize

96199

decreased by-0.47%

Lines of code

829

decreased by-0.72%

Dependency changes

• + Added@backstage/backend-plugin-api@0.0.0-nightly-20241101023511(transitive)

• + Added@backstage/catalog-client@0.0.0-nightly-20241101023511(transitive)

• + Added@backstage/cli-common@0.0.0-nightly-20241101023511(transitive)

• + Added@backstage/plugin-auth-node@0.0.0-nightly-20241101023511(transitive)

• - Removed@backstage/backend-plugin-api@0.0.0-nightly-20241031023248(transitive)

• - Removed@backstage/catalog-client@0.0.0-nightly-20241031023248(transitive)

• - Removed@backstage/cli-common@0.0.0-nightly-20241031023248(transitive)

• - Removed@backstage/plugin-auth-node@0.0.0-nightly-20241031023248(transitive)

• Updated@backstage/backend-plugin-api@0.0.0-nightly-20241101023511