@cap-js-community/event-queue 1.11.0-beta.5 → 1.11.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cap-js-community/event-queue",
-  "version": "1.11.0-beta.5",
+  "version": "1.11.0",
   "description": "An event queue that enables secure transactional processing of asynchronous and periodic events, featuring instant event processing with Redis Pub/Sub and load distribution across all application instances.",
   "main": "src/index.js",
   "types": "src/index.d.ts",
@@ -47,24 +47,24 @@
     "node": ">=18"
   },
   "dependencies": {
-    "@sap/xssec": "^4.6.0",
-    "cron-parser": "^5.3.1",
+    "@sap/xssec": "^4.10.0",
+    "cron-parser": "^5.4.0",
     "redis": "^4.7.0",
     "verror": "^1.10.1",
     "yaml": "^2.7.1"
   },
   "devDependencies": {
     "@cap-js/cds-test": "^0.4.0",
-    "@cap-js/hana": "^2.2.0",
-    "@cap-js/sqlite": "^2.0.1",
-    "@sap/cds": "^9.3.1",
-    "@sap/cds-dk": "^9.3.1",
+    "@cap-js/hana": "^2.3.3",
+    "@cap-js/sqlite": "^2.0.3",
+    "@sap/cds": "^9.4.1",
+    "@sap/cds-dk": "^9.4.1",
     "eslint": "^8.57.0",
     "eslint-config-prettier": "^9.1.0",
     "eslint-plugin-jest": "^28.6.0",
     "eslint-plugin-node": "^11.1.0",
     "express": "^4.21.2",
-    "hdb": "^2.25.1",
+    "hdb": "^2.26.1",
     "jest": "^29.7.0",
     "prettier": "^2.8.8",
     "sqlite3": "^5.1.7",

package/src/redis/redisPub.js

@@ -77,7 +77,7 @@ const broadcastEvent = async (tenantId, events, forceBroadcast = false) => {
           for (let i = 0; i < TRIES_FOR_PUBLISH_PERIODIC_EVENT; i++) {
             const result = eventConfig.multiInstanceProcessing
               ? false
-              : await distributedLock.checkLockExistsAndReturnValue(context, [type, subType].join("##"));
+              : await distributedLock.checkLockExists(context, [type, subType].join("##"));
             if (result) {
               logger.debug("skip publish redis event as no lock is available", {
                 type,

package/src/runner/runner.js

@@ -405,7 +405,7 @@ const _acquireRunId = async (context) => {
       overrideValue: true,
     });
   } else {
-    runId = await distributedLock.checkLockExistsAndReturnValue(context, EVENT_QUEUE_RUN_ID, {
+    runId = await distributedLock.getValue(context, EVENT_QUEUE_RUN_ID, {
       tenantScoped: false,
     });
   }
@@ -422,7 +422,7 @@ const _calculateOffsetForFirstRun = async () => {
   try {
     await trace(dummyContext, "calculateOffsetForFirstRun", async () => {
       if (eventQueueConfig.redisEnabled) {
-        let lastRunTs = await distributedLock.checkLockExistsAndReturnValue(dummyContext, EVENT_QUEUE_RUN_TS, {
+        let lastRunTs = await distributedLock.getValue(dummyContext, EVENT_QUEUE_RUN_TS, {
           tenantScoped: false,
         });
         if (!lastRunTs) {
@@ -434,7 +434,7 @@ const _calculateOffsetForFirstRun = async () => {
           if (couldSetValue) {
             lastRunTs = ts;
           } else {
-            lastRunTs = await distributedLock.checkLockExistsAndReturnValue(dummyContext, EVENT_QUEUE_RUN_TS, {
+            lastRunTs = await distributedLock.getValue(dummyContext, EVENT_QUEUE_RUN_TS, {
               tenantScoped: false,
             });
           }

package/src/shared/cdsHelper.js

@@ -165,38 +165,7 @@ const getAllTenantIds = async () => {
   }, []);
 };
 
-const TENANT_COLUMNS = ["subscribedSubdomain", "createdAt", "modifiedAt"];
-
-const getAllTenantWithMetadata = async () => {
-  const response = await _getAllTenantBase();
-  if (!response) {
-    return null;
-  }
-
-  return response.reduce(async (result, row) => {
-    const tenantId = row.subscribedTenantId ?? row.tenant;
-    result = await result;
-    if (await common.isTenantIdValidCb(TenantIdCheckTypes.eventProcessing, tenantId)) {
-      const data = Object.entries(row).reduce(
-        (result, [key, value]) => {
-          if (TENANT_COLUMNS.includes(key)) {
-            result[key] = value;
-          } else {
-            result.metadata[key] = value;
-          }
-          return result;
-        },
-        { metadata: {} }
-      );
-      data.metadata = JSON.stringify(data.metadata);
-      result.push(data);
-    }
-    return result;
-  }, []);
-};
-
 module.exports = {
   executeInNewTransaction,
   getAllTenantIds,
-  getAllTenantWithMetadata,
 };

package/src/shared/common.js

@@ -92,7 +92,7 @@ const hashStringTo32Bit = (value) => crypto.createHash("sha256").update(String(v
 const _getNewAuthContext = async (tenantId) => {
   try {
     if (!_getNewAuthContext._xsuaaService) {
-      _getNewAuthContext._xsuaaService = new xssec.XsuaaService(cds.requires.auth.credentials);
+      _getNewAuthContext._xsuaaService = new xssec.XsuaaService(cds.requires["xsuaa-eventQueue"]?.credentials);
     }
     const authService = _getNewAuthContext._xsuaaService;
     const token = await authService.fetchClientCredentialsToken({ zid: tenantId });

package/src/shared/distributedLock.js

@@ -60,12 +60,21 @@ const releaseLock = async (context, key, { tenantScoped = true } = {}) => {
   }
 };
 
-const checkLockExistsAndReturnValue = async (context, key, { tenantScoped = true } = {}) => {
+const checkLockExists = async (context, key, { tenantScoped = true } = {}) => {
   const fullKey = _generateKey(context, tenantScoped, key);
   if (config.redisEnabled) {
-    return await _checkLockExistsRedis(context, fullKey);
+    return !!(await _getLockValueRedis(context, fullKey));
   } else {
-    return await _checkLockExistsDb(context, fullKey);
+    return !!(await _getLockValueDb(context, fullKey));
+  }
+};
+
+const getValue = async (context, key, { tenantScoped = true } = {}) => {
+  const fullKey = _generateKey(context, tenantScoped, key);
+  if (config.redisEnabled) {
+    return await _getLockValueRedis(context, fullKey);
+  } else {
+    return await _getLockValueDb(context, fullKey);
   }
 };
 
@@ -106,12 +115,12 @@ const _renewLockRedis = async (context, fullKey, expiryTime, { value = "true" }
   return result === REDIS_COMMAND_OK;
 };
 
-const _checkLockExistsRedis = async (context, fullKey) => {
+const _getLockValueRedis = async (context, fullKey) => {
   const client = await redis.createMainClientAndConnect(config.redisOptions);
-  return await client.exists(fullKey);
+  return await client.get(fullKey);
 };
 
-const _checkLockExistsDb = async (context, fullKey) => {
+const _getLockValueDb = async (context, fullKey) => {
   let result;
   await cdsHelper.executeInNewTransaction(context, "distributedLock-checkExists", async (tx) => {
     result = await tx.run(SELECT.one.from(config.tableNameEventLock).where("code =", fullKey));
@@ -259,7 +268,8 @@ const shutdownHandler = async () => {
 module.exports = {
   acquireLock,
   releaseLock,
-  checkLockExistsAndReturnValue,
+  checkLockExists,
+  getValue,
   setValueWithExpire,
   shutdownHandler,
   renewLock,

package/srv/service/admin-service.cds

@@ -42,12 +42,4 @@ service EventQueueAdminService {
         @mandatory
         subType: String) returns Boolean;
     }
-
-   @readonly
-   @cds.persistence.skip
-   entity Tenant {
-      Key ID: String;
-      subdomain: String;
-      metadata: String;
-   }
 }

package/srv/service/admin-service.js

@@ -1,7 +1,6 @@
 "use strict";
 
 const cds = require("@sap/cds");
-const cdsHelper = require("../../src/shared/cdsHelper");
 const { EventProcessingStatus } = require("../../src");
 const config = require("../../src/config");
 const distributedLock = require("../../src/shared/distributedLock");
@@ -9,7 +8,7 @@ const redisPub = require("../../src/redis/redisPub");
 
 module.exports = class AdminService extends cds.ApplicationService {
   async init() {
-    const { Event: EventService, Tenant, Lock: LockService } = this.entities();
+    const { Event: EventService, Lock: LockService } = this.entities();
     const { Event: EventDb } = cds.db.entities("sap.eventqueue");
     const { landscape, space } = this.getLandscapeAndSpace();
 
@@ -18,9 +17,6 @@ module.exports = class AdminService extends cds.ApplicationService {
         req.reject(403, "Admin service is disabled by configuration");
       }
 
-      if (req.target.name === Tenant.name) {
-        return;
-      }
       const headers = Object.assign({}, req.headers, req.req?.headers);
       const tenant = headers["z-id"] ?? req.data.tenant;
 
@@ -61,11 +57,6 @@ module.exports = class AdminService extends cds.ApplicationService {
       }));
     });
 
-    this.on("READ", Tenant, async () => {
-      const tenants = await cdsHelper.getAllTenantWithMetadata();
-      return tenants ?? [];
-    });
-
     this.on("setStatusAndAttempts", async (req) => {
       const tenant = req.headers["z-id"];
       cds.log("eventQueue").info("Restarting processing for event queue");