chore: prepare release 2.5.2

Author: andrei-cristea

CHANGELOG.md

@@ -5,8 +5,17 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
 ## [Unreleased]
+
+## [2.5.2] - 2026-02-20
 ### Changed
+- Normalized logging and error messages using Keyple coding standards.
 - Migrated the CI pipeline from Jenkins to GitHub Actions.
+### Upgraded
+- `keyple-common-java-api` from `2.0.1` to `2.0.2` (source code not impacted)
+- `keyple-distributed-remote-java-api` from `3.1.0` to `3.1.1` (source code not impacted)
+- `keyple-distributed-network-java-lib` from `2.5.1` to `2.5.2` (source code not impacted)
+- `keyple-util-java-lib` from `2.4.0` to `2.4.1` (source code not impacted)
+- `slf4j-api` from `1.7.32` to `1.7.36` (`compileOnly`)
 
 ## [2.5.1] - 2024-09-19
 ### Fixed
@@ -92,7 +101,8 @@ It follows the extraction of Keyple 1.0 components contained in the `eclipse-key
 repositories.
 It also brings many major API changes.
 
-[unreleased]: https://github.com/eclipse-keyple/keyple-distributed-remote-java-lib/compare/2.5.1...HEAD
+[unreleased]: https://github.com/eclipse-keyple/keyple-distributed-remote-java-lib/compare/2.5.2...HEAD
+[2.5.2]: https://github.com/eclipse-keyple/keyple-distributed-remote-java-lib/compare/2.5.1...2.5.2
 [2.5.1]: https://github.com/eclipse-keyple/keyple-distributed-remote-java-lib/compare/2.5.0...2.5.1
 [2.5.0]: https://github.com/eclipse-keyple/keyple-distributed-remote-java-lib/compare/2.4.0...2.5.0
 [2.4.0]: https://github.com/eclipse-keyple/keyple-distributed-remote-java-lib/compare/2.3.1...2.4.0

build.gradle.kts

@@ -14,13 +14,14 @@ plugins {
 ///////////////////////////////////////////////////////////////////////////////
 
 dependencies {
-  implementation("org.eclipse.keyple:keyple-common-java-api:2.0.1")
-  implementation("org.eclipse.keyple:keyple-distributed-remote-java-api:3.1.0")
-  implementation("org.eclipse.keyple:keyple-distributed-network-java-lib:2.5.1")
-  implementation("org.eclipse.keyple:keyple-util-java-lib:2.4.0")
+  implementation("org.eclipse.keyple:keyple-common-java-api:2.0.2")
+  implementation("org.eclipse.keyple:keyple-distributed-remote-java-api:3.1.1")
+  implementation("org.eclipse.keyple:keyple-distributed-network-java-lib:2.5.2")
+  implementation("org.eclipse.keyple:keyple-util-java-lib:2.4.1")
   implementation("com.google.code.gson:gson:2.10.1")
-  implementation("org.slf4j:slf4j-api:1.7.32")
-  testImplementation("org.slf4j:slf4j-simple:1.7.32")
+  compileOnly("org.slf4j:slf4j-api:1.7.36")
+
+  testImplementation("org.slf4j:slf4j-simple:1.7.36")
   testImplementation(platform("org.junit:junit-bom:5.10.2"))
   testImplementation("org.junit.jupiter:junit-jupiter")
   testImplementation("org.junit.vintage:junit-vintage-engine")

src/main/java/org/eclipse/keyple/distributed/AbstractRemotePluginClientAdapter.java

@@ -52,9 +52,9 @@ public final AsyncNodeClient getAsyncNode() {
       return (AsyncNodeClient) getNode();
     }
     throw new IllegalStateException(
-        String.format(
-            "Remote plugin [%s] is not configured with an asynchronous network protocol",
-            getName()));
+        "Remote plugin '"
+            + getName()
+            + "' is not configured with an asynchronous network protocol");
   }
 
   /**

src/main/java/org/eclipse/keyple/distributed/ObservableRemotePluginServerAdapter.java

@@ -69,8 +69,7 @@ public SyncNodeServer getSyncNode() {
       return (SyncNodeServer) getNode();
     }
     throw new IllegalStateException(
-        String.format(
-            "Remote plugin [%s] is not configured with a synchronous network protocol", getName()));
+        "Remote plugin '" + getName() + "' is not configured with a synchronous network protocol");
   }
 
   /**
@@ -84,9 +83,9 @@ public AsyncNodeServer getAsyncNode() {
       return (AsyncNodeServer) getNode();
     }
     throw new IllegalStateException(
-        String.format(
-            "Remote plugin [%s] is not configured with an asynchronous network protocol",
-            getName()));
+        "Remote plugin '"
+            + getName()
+            + "' is not configured with an asynchronous network protocol");
   }
 
   /**
@@ -103,8 +102,7 @@ public void endRemoteService(String remoteReaderName, Object outputData) {
     RemoteReaderServerAdapter reader = readers.remove(remoteReaderName);
 
     if (reader == null) {
-      throw new IllegalArgumentException(
-          String.format("No reader exists with name [%s]", remoteReaderName));
+      throw new IllegalArgumentException("No reader exists with name '" + remoteReaderName + "'");
     }
 
     // Unregister the remote reader.
@@ -235,7 +233,8 @@ public void onStopObservation() {
   void onMessage(MessageDto message) {
 
     if (!Action.EXECUTE_REMOTE_SERVICE.name().equals(message.getAction())) {
-      throw new IllegalStateException(String.format("Message not supported : %s", message));
+      throw new IllegalStateException(
+          "Message action '" + message.getAction() + "' not supported: " + message);
     }
 
     // Creates a remote reader based on the incoming message.
@@ -303,14 +302,6 @@ void onMessage(MessageDto message) {
     // Other fields
     String remoteReaderName = UUID.randomUUID().toString();
 
-    logger.info(
-        "Plugin [{}] create new remote reader (remoteReaderName: {}, serviceId: {}, sessionId: {}, clientNodeId: {})",
-        getName(),
-        remoteReaderName,
-        serviceId,
-        message.getSessionId(),
-        message.getClientNodeId());
-
     RemoteReaderServerAdapter remoteReader =
         new RemoteReaderServerAdapter(
             clientDistributedApiLevel,
@@ -331,5 +322,13 @@ void onMessage(MessageDto message) {
 
     // Register the remote reader and notify observers.
     observableRemotePluginApi.addRemoteReader(remoteReader, clientCoreApiLevel);
+
+    logger.info(
+        "[plugin={}] New 'RemoteReaderServer' created [remoteReaderName={}, serviceId={}, sessionId={}, clientNodeId={}]",
+        getName(),
+        remoteReaderName,
+        serviceId,
+        message.getSessionId(),
+        message.getClientNodeId());
   }
 }

src/main/java/org/eclipse/keyple/distributed/RemotePluginClientFactoryAdapter.java

@@ -96,6 +96,10 @@ public AbstractRemotePluginSpi getRemotePlugin() {
 
     // Bind the node.
     if (syncEndpointClientSpi != null) {
+
+      remotePlugin.bindSyncNodeClient(
+          syncEndpointClientSpi, syncPluginObservationStrategy, syncReaderObservationStrategy);
+
       String pluginObservationStrategy =
           syncPluginObservationStrategy != null
               ? syncPluginObservationStrategy.getType().name()
@@ -111,25 +115,23 @@ public AbstractRemotePluginSpi getRemotePlugin() {
                   + "_millis"
               : null;
       logger.info(
-          "Create new 'RemotePluginClient' (name: {}, nodeType: SyncNodeClient, isPluginObservationEnabled: {}, syncPluginObservationStrategy: {}, isReaderObservationEnabled: {}, syncReaderObservationStrategy: {})",
+          "New 'RemotePluginClient' created [name={}, nodeType=SyncNodeClient, isPluginObservationEnabled={}, syncPluginObservationStrategy={}, isReaderObservationEnabled={}, syncReaderObservationStrategy={}]",
           getRemotePluginName(),
           isPluginObservationEnabled,
           pluginObservationStrategy,
           isReaderObservationEnabled,
           readerObservationStrategy);
 
-      remotePlugin.bindSyncNodeClient(
-          syncEndpointClientSpi, syncPluginObservationStrategy, syncReaderObservationStrategy);
-
     } else {
+
+      remotePlugin.bindAsyncNodeClient(asyncEndpointClientSpi, asyncNodeClientTimeoutSeconds);
+
       logger.info(
-          "Create new 'RemotePluginClient' (name: {}, nodeType: AsyncNodeClient, timeoutSeconds: {}, isPluginObservationEnabled: {}, isReaderObservationEnabled: {})",
+          "New 'RemotePluginClient' created [name={}, nodeType=AsyncNodeClient, timeoutSeconds={}, isPluginObservationEnabled={}, isReaderObservationEnabled={}]",
           getRemotePluginName(),
           asyncNodeClientTimeoutSeconds,
           isPluginObservationEnabled,
           isReaderObservationEnabled);
-
-      remotePlugin.bindAsyncNodeClient(asyncEndpointClientSpi, asyncNodeClientTimeoutSeconds);
     }
 
     return remotePlugin;

src/main/java/org/eclipse/keyple/distributed/RemotePluginServerFactoryAdapter.java

@@ -65,18 +65,16 @@ public AbstractRemotePluginSpi getRemotePlugin() {
         new ObservableRemotePluginServerAdapter(getRemotePluginName(), executorService);
 
     // Bind the node.
-    String nodeType = asyncEndpointServerSpi != null ? "AsyncNodeServer" : "SyncNodeServer";
-    logger.info(
-        "Create new 'RemotePluginServer' (name: {}, nodeType: {}, timeoutSeconds: {})",
-        getRemotePluginName(),
-        nodeType,
-        timeoutSeconds);
-
     if (asyncEndpointServerSpi == null) {
       remotePlugin.bindSyncNodeServer(timeoutSeconds);
     } else {
       remotePlugin.bindAsyncNodeServer(asyncEndpointServerSpi, timeoutSeconds);
     }
+    logger.info(
+        "New 'RemotePluginServer' created [name={}, nodeType={}, timeoutSeconds={}]",
+        getRemotePluginName(),
+        asyncEndpointServerSpi != null ? "AsyncNodeServer" : "SyncNodeServer",
+        timeoutSeconds);
 
     return remotePlugin;
   }

src/main/java/org/eclipse/keyple/distributed/RemoteReaderServerAdapter.java

@@ -98,7 +98,8 @@ public Object getInitialCardContent() {
         Class<?> classOfInitialCardContent = Class.forName(initialCardContentClassName);
         return JsonUtil.getParser().fromJson(initialCardContentJson, classOfInitialCardContent);
       } catch (ClassNotFoundException e) {
-        logger.error("Class not found: {}", initialCardContentClassName, e);
+        logger.error(
+            "Class not found [initialCardContentClassName={}]", initialCardContentClassName, e);
       }
     }
     return null;