From d4d9cf0aabcb361337c120ac1450c866b3605247 Mon Sep 17 00:00:00 2001 From: alexdbarrio Date: Fri, 21 Nov 2025 16:40:15 +0100 Subject: [PATCH 01/13] Testing java 17 upgrade --- pom.xml | 6 +++--- tool/pom.xml | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/pom.xml b/pom.xml index 9eb846b..678f488 100644 --- a/pom.xml +++ b/pom.xml @@ -145,8 +145,8 @@ 3.8.6 - 11 - ATTENTION: please use JDK11 or later + 17 + ATTENTION: please use JDK17 or later @@ -159,7 +159,7 @@ maven-compiler-plugin 3.10.1 - 11 + 17 -Xlint -Werror diff --git a/tool/pom.xml b/tool/pom.xml index 7d9256a..f1c0051 100644 --- a/tool/pom.xml +++ b/tool/pom.xml @@ -83,7 +83,7 @@ %JAVA_HOME%;%PATH% - 11.0 + 17.0 ${parsedVersion.majorVersion}.${parsedVersion.minorVersion}.${parsedVersion.incrementalVersion}.${parsedVersion.buildNumber} From 2d882140b3d9a9f451c4a110ec87f8b1a8f5658d Mon Sep 17 00:00:00 2001 From: alexdbarrio Date: Fri, 21 Nov 2025 16:41:15 +0100 Subject: [PATCH 02/13] Release v25.11.21 --- library/pom.xml | 2 +- pom.xml | 2 +- tool/pom.xml | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/library/pom.xml b/library/pom.xml index b8f4a2a..1e7998f 100644 --- a/library/pom.xml +++ b/library/pom.xml @@ -7,7 +7,7 @@ com.fidesmo fdsm-toolbox - 25.02.15-SNAPSHOT + 25.11.21 fdsm diff --git a/pom.xml b/pom.xml index 678f488..b37fd07 100644 --- a/pom.xml +++ b/pom.xml @@ -6,7 +6,7 @@ com.fidesmo fdsm-toolbox - 25.02.15-SNAPSHOT + 25.11.21 pom UTF-8 diff --git a/tool/pom.xml b/tool/pom.xml index f1c0051..f7e6028 100644 --- a/tool/pom.xml +++ b/tool/pom.xml @@ -7,7 +7,7 @@ com.fidesmo fdsm-toolbox - 25.02.15-SNAPSHOT + 25.11.21 fdsm-tool From f09a551f15c18ee530632cfec830ee36ce68a45e Mon Sep 17 00:00:00 2001 From: alexdbarrio Date: Fri, 21 Nov 2025 16:41:16 +0100 Subject: [PATCH 03/13] Set version to next snapshot --- library/pom.xml | 2 +- pom.xml | 2 +- tool/pom.xml | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/library/pom.xml b/library/pom.xml index 1e7998f..30241ac 100644 --- a/library/pom.xml +++ b/library/pom.xml @@ -7,7 +7,7 @@ com.fidesmo fdsm-toolbox - 25.11.21 + 25.11.22-SNAPSHOT fdsm diff --git a/pom.xml b/pom.xml index b37fd07..d81183d 100644 --- a/pom.xml +++ b/pom.xml @@ -6,7 +6,7 @@ com.fidesmo fdsm-toolbox - 25.11.21 + 25.11.22-SNAPSHOT pom UTF-8 diff --git a/tool/pom.xml b/tool/pom.xml index f7e6028..50f15a7 100644 --- a/tool/pom.xml +++ b/tool/pom.xml @@ -7,7 +7,7 @@ com.fidesmo fdsm-toolbox - 25.11.21 + 25.11.22-SNAPSHOT fdsm-tool From c15246f2eebedbb85985081b72d478903597d728 Mon Sep 17 00:00:00 2001 From: alexdbarrio Date: Mon, 24 Nov 2025 10:14:03 +0100 Subject: [PATCH 04/13] Upgraded to java 21 --- .github/workflows/ci.yml | 2 +- pom.xml | 6 +++--- tool/pom.xml | 2 +- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 54b91e0..1662769 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -11,7 +11,7 @@ jobs: runs-on: ubuntu-latest strategy: matrix: - java: [ 11, 17 ] + java: [ 11, 17, 21 ] name: Java ${{ matrix.java }} steps: - name: Check out code diff --git a/pom.xml b/pom.xml index d81183d..9327f89 100644 --- a/pom.xml +++ b/pom.xml @@ -145,8 +145,8 @@ 3.8.6 - 17 - ATTENTION: please use JDK17 or later + 21 + ATTENTION: please use JDK21 or later @@ -159,7 +159,7 @@ maven-compiler-plugin 3.10.1 - 17 + 21 -Xlint -Werror diff --git a/tool/pom.xml b/tool/pom.xml index 50f15a7..e531aa2 100644 --- a/tool/pom.xml +++ b/tool/pom.xml @@ -83,7 +83,7 @@ %JAVA_HOME%;%PATH% - 17.0 + 21.0 ${parsedVersion.majorVersion}.${parsedVersion.minorVersion}.${parsedVersion.incrementalVersion}.${parsedVersion.buildNumber} From 0a9147a2ed5d696cd6ff2e8fd0eb92b990a04457 Mon Sep 17 00:00:00 2001 From: alexdbarrio Date: Mon, 24 Nov 2025 11:21:54 +0100 Subject: [PATCH 05/13] Problem fixed --- pom.xml | 2 +- .../main/java/com/fidesmo/fdsm/CommandLineInterface.java | 6 ++++-- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/pom.xml b/pom.xml index 9327f89..c2dfe3e 100644 --- a/pom.xml +++ b/pom.xml @@ -170,7 +170,7 @@ com.github.spotbugs spotbugs-maven-plugin - 4.7.3.0 + 4.9.8.0 spotbugs.xml diff --git a/tool/src/main/java/com/fidesmo/fdsm/CommandLineInterface.java b/tool/src/main/java/com/fidesmo/fdsm/CommandLineInterface.java index af956cd..a4dd8f1 100644 --- a/tool/src/main/java/com/fidesmo/fdsm/CommandLineInterface.java +++ b/tool/src/main/java/com/fidesmo/fdsm/CommandLineInterface.java @@ -31,6 +31,8 @@ import java.io.IOException; import java.io.PrintStream; import java.net.MalformedURLException; +import java.net.URI; +import java.net.URISyntaxException; import java.net.URL; import java.util.Arrays; import java.util.List; @@ -114,8 +116,8 @@ protected static void inspectEnvironment(OptionSet args) { // API URL try { - apiurl = new URL(System.getenv().getOrDefault(ENV_FIDESMO_API_URL, FidesmoApiClient.APIv3)).toString(); - } catch (MalformedURLException e) { + apiurl = new URI(System.getenv().getOrDefault(ENV_FIDESMO_API_URL, FidesmoApiClient.APIv3)).toURL().toString(); + } catch (MalformedURLException | URISyntaxException e) { System.err.printf("Invalid $%s: %s%n", ENV_FIDESMO_API_URL, System.getenv(ENV_FIDESMO_API_URL)); } } From 8bdeedbcabb1244c27fb89eb119f506016d7117b Mon Sep 17 00:00:00 2001 From: alexdbarrio Date: Mon, 24 Nov 2025 11:23:40 +0100 Subject: [PATCH 06/13] Release v25.11.24 --- library/pom.xml | 2 +- pom.xml | 2 +- tool/pom.xml | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/library/pom.xml b/library/pom.xml index 30241ac..d5b4b87 100644 --- a/library/pom.xml +++ b/library/pom.xml @@ -7,7 +7,7 @@ com.fidesmo fdsm-toolbox - 25.11.22-SNAPSHOT + 25.11.24 fdsm diff --git a/pom.xml b/pom.xml index c2dfe3e..a1fdf38 100644 --- a/pom.xml +++ b/pom.xml @@ -6,7 +6,7 @@ com.fidesmo fdsm-toolbox - 25.11.22-SNAPSHOT + 25.11.24 pom UTF-8 diff --git a/tool/pom.xml b/tool/pom.xml index e531aa2..0494c9c 100644 --- a/tool/pom.xml +++ b/tool/pom.xml @@ -7,7 +7,7 @@ com.fidesmo fdsm-toolbox - 25.11.22-SNAPSHOT + 25.11.24 fdsm-tool From cee50a1a246551560592a09978481b3f23ee0e7a Mon Sep 17 00:00:00 2001 From: alexdbarrio Date: Mon, 24 Nov 2025 11:23:41 +0100 Subject: [PATCH 07/13] Set version to next snapshot --- library/pom.xml | 2 +- pom.xml | 2 +- tool/pom.xml | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/library/pom.xml b/library/pom.xml index d5b4b87..424f746 100644 --- a/library/pom.xml +++ b/library/pom.xml @@ -7,7 +7,7 @@ com.fidesmo fdsm-toolbox - 25.11.24 + 25.11.25-SNAPSHOT fdsm diff --git a/pom.xml b/pom.xml index a1fdf38..b0c7796 100644 --- a/pom.xml +++ b/pom.xml @@ -6,7 +6,7 @@ com.fidesmo fdsm-toolbox - 25.11.24 + 25.11.25-SNAPSHOT pom UTF-8 diff --git a/tool/pom.xml b/tool/pom.xml index 0494c9c..3fa5a7c 100644 --- a/tool/pom.xml +++ b/tool/pom.xml @@ -7,7 +7,7 @@ com.fidesmo fdsm-toolbox - 25.11.24 + 25.11.25-SNAPSHOT fdsm-tool From 85e8bbf254e79f2aded8d046a69f39c49a59ab12 Mon Sep 17 00:00:00 2001 From: alexdbarrio Date: Mon, 24 Nov 2025 11:25:00 +0100 Subject: [PATCH 08/13] Readme updated with java 21 --- README.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/README.md b/README.md index 456c3c5..bfd063c 100644 --- a/README.md +++ b/README.md @@ -5,7 +5,7 @@ FDSM is _the_ developer's Swiss Army Knife for working with a [Fidesmo device](h # Getting started You will need the following: -- Latest [Java 11](https://adoptopenjdk.net/) for running the app +- Latest [Java 21](https://adoptopenjdk.net/) for running the app - a [Fidesmo Developer account](https://fidesmo.com/book-demo/) for your `$FIDESMO_AUTH` token and `$FIDESMO_APPID` - a Fidesmo device (get one from the [shop](http://shop.fidesmo.com)) - a [PC/SC NFC reader](https://github.com/fidesmo/fdsm/wiki/Choosing-a-desktop-reader) From bd39487028f2ea5964c5b29481596e0408f4d421 Mon Sep 17 00:00:00 2001 From: alexdbarrio Date: Mon, 24 Nov 2025 11:25:49 +0100 Subject: [PATCH 09/13] Readme updated --- library/pom.xml | 2 +- pom.xml | 2 +- tool/pom.xml | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/library/pom.xml b/library/pom.xml index 424f746..d5b4b87 100644 --- a/library/pom.xml +++ b/library/pom.xml @@ -7,7 +7,7 @@ com.fidesmo fdsm-toolbox - 25.11.25-SNAPSHOT + 25.11.24 fdsm diff --git a/pom.xml b/pom.xml index b0c7796..a1fdf38 100644 --- a/pom.xml +++ b/pom.xml @@ -6,7 +6,7 @@ com.fidesmo fdsm-toolbox - 25.11.25-SNAPSHOT + 25.11.24 pom UTF-8 diff --git a/tool/pom.xml b/tool/pom.xml index 3fa5a7c..0494c9c 100644 --- a/tool/pom.xml +++ b/tool/pom.xml @@ -7,7 +7,7 @@ com.fidesmo fdsm-toolbox - 25.11.25-SNAPSHOT + 25.11.24 fdsm-tool From e9059ceb06322e9da7b9c34a553a35b734c61f92 Mon Sep 17 00:00:00 2001 From: alexdbarrio Date: Mon, 24 Nov 2025 11:28:45 +0100 Subject: [PATCH 10/13] test --- library/pom.xml | 2 +- pom.xml | 2 +- tool/pom.xml | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/library/pom.xml b/library/pom.xml index d5b4b87..424f746 100644 --- a/library/pom.xml +++ b/library/pom.xml @@ -7,7 +7,7 @@ com.fidesmo fdsm-toolbox - 25.11.24 + 25.11.25-SNAPSHOT fdsm diff --git a/pom.xml b/pom.xml index a1fdf38..b0c7796 100644 --- a/pom.xml +++ b/pom.xml @@ -6,7 +6,7 @@ com.fidesmo fdsm-toolbox - 25.11.24 + 25.11.25-SNAPSHOT pom UTF-8 diff --git a/tool/pom.xml b/tool/pom.xml index 0494c9c..3fa5a7c 100644 --- a/tool/pom.xml +++ b/tool/pom.xml @@ -7,7 +7,7 @@ com.fidesmo fdsm-toolbox - 25.11.24 + 25.11.25-SNAPSHOT fdsm-tool From 58f2086bebb3b302e153d397ff5b6f7a3689f743 Mon Sep 17 00:00:00 2001 From: alexdbarrio Date: Mon, 24 Nov 2025 12:41:20 +0100 Subject: [PATCH 11/13] Spotbugs problems fixed --- .../main/java/com/fidesmo/fdsm/ClientAuthentication.java | 6 +++--- library/src/main/java/com/fidesmo/fdsm/ClientInfo.java | 4 ++-- library/src/main/java/com/fidesmo/fdsm/FidesmoCard.java | 6 +++--- 3 files changed, 8 insertions(+), 8 deletions(-) diff --git a/library/src/main/java/com/fidesmo/fdsm/ClientAuthentication.java b/library/src/main/java/com/fidesmo/fdsm/ClientAuthentication.java index 4eb0ed3..4dc77c2 100644 --- a/library/src/main/java/com/fidesmo/fdsm/ClientAuthentication.java +++ b/library/src/main/java/com/fidesmo/fdsm/ClientAuthentication.java @@ -29,9 +29,6 @@ public class ClientAuthentication { private final String authentication; private ClientAuthentication(String auth) { - if (auth.split(":").length > 2) { - throw new IllegalArgumentException("Wrong authentication format"); - } this.authentication = auth; } @@ -47,6 +44,9 @@ public Optional getUsername() { } public static ClientAuthentication forToken(String token) { + if (token.split(":").length > 2) { + throw new IllegalArgumentException("Wrong authentication format"); + } return new ClientAuthentication(token); } diff --git a/library/src/main/java/com/fidesmo/fdsm/ClientInfo.java b/library/src/main/java/com/fidesmo/fdsm/ClientInfo.java index a2623e5..d342e3b 100644 --- a/library/src/main/java/com/fidesmo/fdsm/ClientInfo.java +++ b/library/src/main/java/com/fidesmo/fdsm/ClientInfo.java @@ -34,14 +34,14 @@ public static String stringify(Capability c) { private Set capabilities; private Locale locale; - public ClientInfo(String name, String version, Set capabilities, Locale locale) { + private ClientInfo(String name, String version, Set capabilities, Locale locale) { this.name = name; this.version = version; this.capabilities = Collections.unmodifiableSet(capabilities); this.locale = locale; } - public ClientInfo(String name, String version) { + private ClientInfo(String name, String version) { this(name, version, DEFAULT_CAPABILITIES, Locale.getDefault()); } diff --git a/library/src/main/java/com/fidesmo/fdsm/FidesmoCard.java b/library/src/main/java/com/fidesmo/fdsm/FidesmoCard.java index 21b082c..036104b 100644 --- a/library/src/main/java/com/fidesmo/fdsm/FidesmoCard.java +++ b/library/src/main/java/com/fidesmo/fdsm/FidesmoCard.java @@ -149,8 +149,7 @@ public static Optional detectPlatform(byte[] cplc) { /** Indicates that device is fully batched or requires batching operation otherwise */ private final boolean batched; - public FidesmoCard(byte[] fid, byte[] cplc, int batchId, boolean batched, Optional uid) { - if (fid == null) throw new NullPointerException("fid can't be null"); + private FidesmoCard(byte[] fid, byte[] cplc, int batchId, boolean batched, Optional uid) { this.cin = fid.clone(); this.cplc = cplc == null ? null : cplc.clone(); this.batchId = batchId; @@ -295,7 +294,8 @@ public static Optional detect(Map commands, Fides return Optional.empty(); } } - + + if (cin == null) throw new NullPointerException("fid can't be null"); return Optional.of(new FidesmoCard(cin, cplc, new BigInteger(1, batch).intValue(), true, uid)); } From ff4eeb7509c28ca1cfa803fd163b3eca9433aba0 Mon Sep 17 00:00:00 2001 From: alexdbarrio Date: Mon, 24 Nov 2025 12:47:57 +0100 Subject: [PATCH 12/13] Removed java 11 & 17 --- .github/workflows/ci.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 1662769..1f543f0 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -11,7 +11,7 @@ jobs: runs-on: ubuntu-latest strategy: matrix: - java: [ 11, 17, 21 ] + java: [ 21 ] name: Java ${{ matrix.java }} steps: - name: Check out code From dd9a62f2e168d502efce50938fe81d449ace0861 Mon Sep 17 00:00:00 2001 From: alexdbarrio Date: Mon, 24 Nov 2025 13:31:13 +0100 Subject: [PATCH 13/13] Removed unused import --- tool/src/main/java/com/fidesmo/fdsm/CommandLineInterface.java | 1 - 1 file changed, 1 deletion(-) diff --git a/tool/src/main/java/com/fidesmo/fdsm/CommandLineInterface.java b/tool/src/main/java/com/fidesmo/fdsm/CommandLineInterface.java index a4dd8f1..fc465af 100644 --- a/tool/src/main/java/com/fidesmo/fdsm/CommandLineInterface.java +++ b/tool/src/main/java/com/fidesmo/fdsm/CommandLineInterface.java @@ -33,7 +33,6 @@ import java.net.MalformedURLException; import java.net.URI; import java.net.URISyntaxException; -import java.net.URL; import java.util.Arrays; import java.util.List; import java.util.Locale;