@celerispay/hazelcast-client 3.12.5-5 → 3.12.5-7

package/lib/invocation/ClusterService.d.ts

@@ -47,6 +47,10 @@ export declare class ClusterService {
      */
     connectToCluster(): Promise<void>;
     getPossibleMemberAddresses(): Promise<string[]>;
+    /**
+     * Returns the owner connection if available
+     */
+    getOwnerConnection(): ClientConnection | null;
     /**
      * Returns the list of members in the cluster.
      * @returns
@@ -63,11 +67,6 @@ export declare class ClusterService {
      * @returns {ClientInfo}
      */
     getClientInfo(): ClientInfo;
-    /**
-     * Returns the connection associated with owner node of this client.
-     * @returns {ClientConnection}
-     */
-    getOwnerConnection(): ClientConnection;
     /**
      * Adds MembershipListener to listen for membership updates. There is no check for duplicate registrations,
      * so if you register the listener twice, it will get events twice.

package/lib/invocation/ClusterService.js

@@ -103,6 +103,12 @@ var ClusterService = /** @class */ (function () {
             return Array.from(new Set(Array.from(addresses).concat(Array.from(providerAddresses))));
         });
     };
+    /**
+     * Returns the owner connection if available
+     */
+    ClusterService.prototype.getOwnerConnection = function () {
+        return this.ownerConnection;
+    };
     /**
      * Returns the list of members in the cluster.
      * @returns
@@ -144,16 +150,15 @@ var ClusterService = /** @class */ (function () {
     ClusterService.prototype.getClientInfo = function () {
         var info = new ClientInfo_1.ClientInfo();
         info.uuid = this.uuid;
-        info.localAddress = this.getOwnerConnection().getLocalAddress();
+        var ownerConnection = this.getOwnerConnection();
+        if (ownerConnection) {
+            info.localAddress = ownerConnection.getLocalAddress();
+        }
+        else {
+            info.localAddress = null;
+        }
         return info;
     };
-    /**
-     * Returns the connection associated with owner node of this client.
-     * @returns {ClientConnection}
-     */
-    ClusterService.prototype.getOwnerConnection = function () {
-        return this.ownerConnection;
-    };
     /**
      * Adds MembershipListener to listen for membership updates. There is no check for duplicate registrations,
      * so if you register the listener twice, it will get events twice.
@@ -185,7 +190,11 @@ var ClusterService = /** @class */ (function () {
             var handleAttributeChange = _this.handleMemberAttributeChange.bind(_this);
             ClientAddMembershipListenerCodec_1.ClientAddMembershipListenerCodec.handle(m, handleMember, handleMemberList, handleAttributeChange, null);
         };
-        return this.client.getInvocationService().invokeOnConnection(this.getOwnerConnection(), request, handler)
+        var ownerConnection = this.getOwnerConnection();
+        if (!ownerConnection) {
+            return Promise.reject(new Error('Cannot initialize membership listener: no owner connection available'));
+        }
+        return this.client.getInvocationService().invokeOnConnection(ownerConnection, request, handler)
             .then(function (resp) {
             _this.logger.trace('ClusterService', 'Registered listener with id '
                 + ClientAddMembershipListenerCodec_1.ClientAddMembershipListenerCodec.decodeResponse(resp).response);
@@ -603,7 +612,7 @@ var ClusterService = /** @class */ (function () {
         // Check if we're already trying to connect to this address
         var connectionManager = this.client.getConnectionManager();
         var establishedConnections = connectionManager.getEstablishedConnections();
-        var pendingConnections = Object.keys(connectionManager.getPendingConnections || {}).length;
+        var pendingConnections = Object.keys(connectionManager.getPendingConnections()).length;
         if (pendingConnections > 0) {
             this.logger.debug('ClusterService', "Already have pending connections, skipping reconnection to " + addressStr);
             return;

package/lib/proxy/ProxyManager.js

@@ -91,6 +91,9 @@ var ProxyManager = /** @class */ (function () {
         if (createAtServer) {
             this.createProxy(newProxy).then(function () {
                 deferred.resolve(newProxy);
+            }).catch(function (error) {
+                _this.logger.error('ProxyManager', 'Failed to create proxy for ' + name + ': ' + error);
+                deferred.reject(error);
             });
         }
         this.proxies[fullName] = deferred.promise;
@@ -130,20 +133,32 @@ var ProxyManager = /** @class */ (function () {
     };
     ProxyManager.prototype.findNextAddress = function () {
         var members = this.client.getClusterService().getMembers();
+        // If no members available, return null but log the issue
+        if (!members || members.length === 0) {
+            this.logger.warn('ProxyManager', 'No cluster members available for proxy creation');
+            return null;
+        }
         var liteMember = null;
+        var dataMember = null;
         for (var _i = 0, members_1 = members; _i < members_1.length; _i++) {
             var member = members_1[_i];
             if (member != null && member.isLiteMember === false) {
-                return member.address;
+                dataMember = member;
+                break; // Prefer data members
             }
             else if (member != null && member.isLiteMember) {
                 liteMember = member;
             }
         }
-        if (liteMember != null) {
+        // Return data member if available, otherwise lite member, otherwise null
+        if (dataMember != null) {
+            return dataMember.address;
+        }
+        else if (liteMember != null) {
             return liteMember.address;
         }
         else {
+            this.logger.warn('ProxyManager', 'No valid members found for proxy creation');
             return null;
         }
     };
@@ -151,6 +166,12 @@ var ProxyManager = /** @class */ (function () {
         var _this = this;
         if (Date.now() <= deadline) {
             var address = this.findNextAddress();
+            if (!address) {
+                var error = new Error('No cluster members available for proxy creation: ' + proxyObject.getName());
+                this.logger.error('ProxyManager', error.message);
+                promise.reject(error);
+                return;
+            }
             var request = ClientCreateProxyCodec_1.ClientCreateProxyCodec.encodeRequest(proxyObject.getName(), proxyObject.getServiceName(), address);
             var invocation = new InvocationService_1.Invocation(this.client, request);
             invocation.address = address;
@@ -160,10 +181,13 @@ var ProxyManager = /** @class */ (function () {
                 if (_this.isRetryable(error)) {
                     _this.logger.warn('ProxyManager', 'Create proxy request for ' + proxyObject.getName() +
                         ' failed. Retrying in ' + _this.invocationRetryPauseMillis + 'ms. ' + error);
-                    setTimeout(_this.initializeProxy.bind(_this, proxyObject, promise, deadline), _this.invocationRetryPauseMillis);
+                    setTimeout(function () {
+                        _this.initializeProxy(proxyObject, promise, deadline);
+                    }, _this.invocationRetryPauseMillis);
                 }
                 else {
-                    _this.logger.warn('ProxyManager', 'Create proxy request for ' + proxyObject.getName() + ' failed ' + error);
+                    _this.logger.error('ProxyManager', 'Create proxy request for ' + proxyObject.getName() + ' failed ' + error);
+                    promise.reject(error);
                 }
             });
         }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@celerispay/hazelcast-client",
-  "version": "3.12.5-5",
+  "version": "3.12.5-7",
   "description": "Hazelcast - open source In-Memory Data Grid - client for NodeJS with critical connection failover fixes",
   "main": "./lib/index.js",
   "scripts": {