@onlineapps/conn-orch-registry 1.1.49 → 1.1.50

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@onlineapps/conn-orch-registry",
-  "version": "1.1.49",
+  "version": "1.1.50",
   "license": "MIT",
   "description": "Connector-registry-client provides the core communication mechanism for microservices in this environment. It enables them to interact with a services_registry to receive and fulfill tasks by submitting heartbeats or their API descriptions.",
   "keywords": [

package/src/registryClient.js

@@ -37,7 +37,7 @@ class ServiceRegistryClient extends EventEmitter {
    */
   constructor({ amqpUrl, serviceName, version, specificationEndpoint = '/api/v1/specification',
                 heartbeatInterval = 10000, apiQueue = 'api_services_queuer', registryQueue = 'registry.register',
-                redis = null, storageConfig = {}, validationProof = null }) {
+                registryUrl = null, redis = null, storageConfig = {}, validationProof = null }) {
     super();
     if (!amqpUrl || !serviceName || !version) {
       throw new Error('amqpUrl, serviceName, and version are required');
@@ -49,6 +49,7 @@ class ServiceRegistryClient extends EventEmitter {
     this.heartbeatInterval = heartbeatInterval;
     this.apiQueue = apiQueue;
     this.registryQueue = registryQueue;
+    this.registryUrl = registryUrl;
     this.queueManager = new QueueManager(amqpUrl, serviceName);
     this.heartbeatTimer = null;
 
@@ -706,27 +707,41 @@ class ServiceRegistryClient extends EventEmitter {
    * @returns {Promise<{serviceName: string, status: 'active'|'inactive', version?: string, fingerprint?: string, bucket?: string, path?: string, updatedAt?: string} | null>}
    */
   async getService(serviceName) {
-    if (!this.eventConsumer) {
-      throw new Error('[RegistryClient] getService - Event consumer not initialized. Fix: call subscribeToChanges() before service discovery.');
-    }
     if (!serviceName || typeof serviceName !== 'string') {
       throw new Error('[RegistryClient] getService - serviceName is required and must be a string');
     }
 
-    const info = this.eventConsumer.getServiceInfo(serviceName);
-    if (!info) {
+    if (!this.registryUrl || typeof this.registryUrl !== 'string') {
+      throw new Error('[RegistryClient] getService - Missing required config: registryUrl. Fix: pass wrapper.registry.url into ServiceRegistryClient.');
+    }
+
+    // Primary source of truth for service availability is the central registry HTTP API.
+    // This avoids relying on optional event subscriptions and ensures deterministic routing.
+    const url = `${this.registryUrl.replace(/\/$/, '')}/services`;
+    let response;
+    try {
+      response = await fetch(url, { method: 'GET' });
+    } catch (err) {
+      throw new Error(`[RegistryClient] getService - Registry HTTP request failed (${url}). Problem: ${err.message}`);
+    }
+
+    if (!response.ok) {
+      throw new Error(`[RegistryClient] getService - Registry HTTP request failed (${url}). Status: ${response.status}`);
+    }
+
+    const data = await response.json();
+    const entry = data && data.services ? data.services[serviceName] : null;
+    if (!entry) {
       return null;
     }
 
-    const normalizedStatus = info.status === 'ACTIVE' ? 'active' : 'inactive';
+    // Registry stores status as 'active'/'inactive' (lowercase).
     return {
       serviceName,
-      status: normalizedStatus,
-      version: info.version,
-      fingerprint: info.fingerprint,
-      bucket: info.bucket,
-      path: info.path,
-      updatedAt: info.updatedAt
+      status: entry.status,
+      version: entry.version,
+      lastHeartbeatAt: entry.lastHeartbeatAt,
+      isAvailable: entry.isAvailable
     };
   }