From c04f964855aba4aa5ebc94872191c1d1ea89eb74 Mon Sep 17 00:00:00 2001 From: "renovate[bot]" <29139614+renovate[bot]@users.noreply.github.com> Date: Tue, 10 Jun 2025 07:45:36 +0000 Subject: [PATCH 1/4] fix(deps): update dependency com.github.spotbugs.snom:spotbugs-gradle-plugin to v6.2.0 Signed-off-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> --- gradle/verification-metadata.xml | 27 +++++++++++++++++++++++++++ library/build.gradle | 2 +- sample_client/build.gradle | 2 +- 3 files changed, 29 insertions(+), 2 deletions(-) diff --git a/gradle/verification-metadata.xml b/gradle/verification-metadata.xml index df99027c0..cd69ee472 100644 --- a/gradle/verification-metadata.xml +++ b/gradle/verification-metadata.xml @@ -70,6 +70,7 @@ + @@ -215,6 +216,7 @@ + @@ -10827,6 +10829,11 @@ + + + + + @@ -10856,6 +10863,11 @@ + + + + + @@ -10892,6 +10904,11 @@ + + + + + @@ -10924,6 +10941,11 @@ + + + + + @@ -10953,6 +10975,11 @@ + + + + + diff --git a/library/build.gradle b/library/build.gradle index 855951107..1ca81f937 100644 --- a/library/build.gradle +++ b/library/build.gradle @@ -19,7 +19,7 @@ buildscript { } dependencies { classpath 'com.android.tools.build:gradle:8.10.1' - classpath 'com.github.spotbugs.snom:spotbugs-gradle-plugin:6.1.13' + classpath 'com.github.spotbugs.snom:spotbugs-gradle-plugin:6.2.0' classpath "org.jetbrains.kotlin:kotlin-gradle-plugin:$kotlin_version" classpath "io.gitlab.arturbosch.detekt:detekt-gradle-plugin:1.23.8" classpath "org.jacoco:org.jacoco.core:$jacoco_version" diff --git a/sample_client/build.gradle b/sample_client/build.gradle index 7185be334..62f0ef680 100644 --- a/sample_client/build.gradle +++ b/sample_client/build.gradle @@ -12,7 +12,7 @@ buildscript { dependencies { classpath 'com.android.tools.build:gradle:8.10.1' - classpath 'com.github.spotbugs.snom:spotbugs-gradle-plugin:6.1.13' + classpath 'com.github.spotbugs.snom:spotbugs-gradle-plugin:6.2.0' classpath "org.jetbrains.kotlin:kotlin-gradle-plugin:$kotlin_version" classpath "io.gitlab.arturbosch.detekt:detekt-gradle-plugin:1.23.8" classpath "commons-httpclient:commons-httpclient:3.1@jar" // remove after entire switch to lib v2 From 74ebfaf05f027c9614d9a4e547a48a3a346753a8 Mon Sep 17 00:00:00 2001 From: Andy Scherzinger Date: Tue, 10 Jun 2025 11:53:44 +0200 Subject: [PATCH 2/4] style(spotbugs): remove useless suppression statement Signed-off-by: Andy Scherzinger --- .../resources/notifications/GetNotificationRemoteOperation.java | 1 - 1 file changed, 1 deletion(-) diff --git a/library/src/main/java/com/owncloud/android/lib/resources/notifications/GetNotificationRemoteOperation.java b/library/src/main/java/com/owncloud/android/lib/resources/notifications/GetNotificationRemoteOperation.java index 4730e4575..c1ee1a455 100644 --- a/library/src/main/java/com/owncloud/android/lib/resources/notifications/GetNotificationRemoteOperation.java +++ b/library/src/main/java/com/owncloud/android/lib/resources/notifications/GetNotificationRemoteOperation.java @@ -43,7 +43,6 @@ public GetNotificationRemoteOperation(int id) { this.id = id; } - @SuppressFBWarnings("HTTP_PARAMETER_POLLUTION") @Override public RemoteOperationResult run(NextcloudClient client) { RemoteOperationResult result; From 4ae1a9ac9ef540dfa73a76c8fc732966c61f13e2 Mon Sep 17 00:00:00 2001 From: Andy Scherzinger Date: Tue, 10 Jun 2025 11:57:40 +0200 Subject: [PATCH 3/4] style(spotbugs): Return more specific interface Signed-off-by: Andy Scherzinger --- .../android/lib/common/network/AdvancedX509KeyManager.java | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/library/src/main/java/com/owncloud/android/lib/common/network/AdvancedX509KeyManager.java b/library/src/main/java/com/owncloud/android/lib/common/network/AdvancedX509KeyManager.java index 7d4304e44..f204ed195 100644 --- a/library/src/main/java/com/owncloud/android/lib/common/network/AdvancedX509KeyManager.java +++ b/library/src/main/java/com/owncloud/android/lib/common/network/AdvancedX509KeyManager.java @@ -287,8 +287,8 @@ private void removeKeyChain(AKMAlias filter) throws IllegalArgumentException { * @param filter AKMAlias object used as filter * @return all aliases from KEYCHAIN_ALIASES which satisfy alias.matches(filter) */ - private static Collection filterAliases(Collection aliases, AKMAlias filter) { - Collection filtered = new LinkedList<>(); + private static LinkedList filterAliases(Collection aliases, AKMAlias filter) { + LinkedList filtered = new LinkedList<>(); for (Object alias : aliases) { if (new AKMAlias(alias.toString()).matches(filter)) { filtered.add(((String) alias)); @@ -308,7 +308,7 @@ private static Collection filterAliases(Collection aliases, AKMAl private @NonNull String[] getAliases(Set keyTypes, Principal[] issuers, String hostname, Integer port) { // Check keychain aliases AKMAlias filter = new AKMAlias(KEYCHAIN, null, hostname, port); - List validAliases = new LinkedList<>(filterAliases(sharedPreferences.getStringSet(KEYCHAIN_ALIASES, new HashSet<>()), filter)); + List validAliases = filterAliases(sharedPreferences.getStringSet(KEYCHAIN_ALIASES, new HashSet<>()), filter); Log_OC.d(TAG, "getAliases(keyTypes=" + (keyTypes != null ? Arrays.toString(keyTypes.toArray()) : null) + ", issuers=" + Arrays.toString(issuers) From a89a1e489917aee92bdb6c0a87455a25e089ad77 Mon Sep 17 00:00:00 2001 From: Andy Scherzinger Date: Tue, 10 Jun 2025 12:03:36 +0200 Subject: [PATCH 4/4] ci(chksm): Add meta-data Signed-off-by: Andy Scherzinger --- gradle/verification-metadata.xml | 998 ++++++++++++++++++++++++++ scripts/analysis/findbugs-results.txt | 2 +- 2 files changed, 999 insertions(+), 1 deletion(-) diff --git a/gradle/verification-metadata.xml b/gradle/verification-metadata.xml index cd69ee472..deb7292cc 100644 --- a/gradle/verification-metadata.xml +++ b/gradle/verification-metadata.xml @@ -511,6 +511,14 @@ + + + + + + + + @@ -599,6 +607,14 @@ + + + + + + + + @@ -1141,6 +1157,14 @@ + + + + + + + + @@ -1229,6 +1253,14 @@ + + + + + + + + @@ -1317,6 +1349,14 @@ + + + + + + + + @@ -1405,6 +1445,14 @@ + + + + + + + + @@ -1493,6 +1541,14 @@ + + + + + + + + @@ -1581,6 +1637,14 @@ + + + + + + + + @@ -1669,6 +1733,14 @@ + + + + + + + + @@ -1741,6 +1813,14 @@ + + + + + + + + @@ -1829,6 +1909,14 @@ + + + + + + + + @@ -1917,6 +2005,14 @@ + + + + + + + + @@ -2005,6 +2101,14 @@ + + + + + + + + @@ -2093,6 +2197,14 @@ + + + + + + + + @@ -2181,6 +2293,14 @@ + + + + + + + + @@ -2269,6 +2389,14 @@ + + + + + + + + @@ -2357,6 +2485,14 @@ + + + + + + + + @@ -2397,6 +2533,14 @@ + + + + + + + + @@ -2485,6 +2629,14 @@ + + + + + + + + @@ -2573,6 +2725,14 @@ + + + + + + + + @@ -2661,6 +2821,14 @@ + + + + + + + + @@ -2749,6 +2917,14 @@ + + + + + + + + @@ -2837,6 +3013,14 @@ + + + + + + + + @@ -2925,6 +3109,14 @@ + + + + + + + + @@ -3061,6 +3253,14 @@ + + + + + + + + @@ -3149,6 +3349,14 @@ + + + + + + + + @@ -3237,6 +3445,14 @@ + + + + + + + + @@ -3325,6 +3541,14 @@ + + + + + + + + @@ -3437,6 +3661,14 @@ + + + + + + + + @@ -3525,6 +3757,14 @@ + + + + + + + + @@ -3613,6 +3853,14 @@ + + + + + + + + @@ -3685,6 +3933,14 @@ + + + + + + + + @@ -3757,6 +4013,14 @@ + + + + + + + + @@ -3829,6 +4093,14 @@ + + + + + + + + @@ -3917,6 +4189,14 @@ + + + + + + + + @@ -3989,6 +4269,14 @@ + + + + + + + + @@ -4061,6 +4349,14 @@ + + + + + + + + @@ -4133,6 +4429,14 @@ + + + + + + + + @@ -4205,6 +4509,14 @@ + + + + + + + + @@ -4293,6 +4605,14 @@ + + + + + + + + @@ -4381,6 +4701,14 @@ + + + + + + + + @@ -4469,6 +4797,14 @@ + + + + + + + + @@ -4557,6 +4893,14 @@ + + + + + + + + @@ -4645,6 +4989,14 @@ + + + + + + + + @@ -4733,6 +5085,14 @@ + + + + + + + + @@ -4773,6 +5133,14 @@ + + + + + + + + @@ -4813,6 +5181,14 @@ + + + + + + + + @@ -4901,6 +5277,14 @@ + + + + + + + + @@ -4989,6 +5373,14 @@ + + + + + + + + @@ -5077,6 +5469,14 @@ + + + + + + + + @@ -5165,6 +5565,14 @@ + + + + + + + + @@ -5253,6 +5661,14 @@ + + + + + + + + @@ -5341,6 +5757,14 @@ + + + + + + + + @@ -5429,6 +5853,14 @@ + + + + + + + + @@ -5517,6 +5949,14 @@ + + + + + + + + @@ -5605,6 +6045,14 @@ + + + + + + + + @@ -5693,6 +6141,14 @@ + + + + + + + + @@ -5781,6 +6237,14 @@ + + + + + + + + @@ -6045,6 +6509,14 @@ + + + + + + + + @@ -6133,6 +6605,14 @@ + + + + + + + + @@ -6221,6 +6701,14 @@ + + + + + + + + @@ -6411,16 +6899,31 @@ + + + + + + + + + + + + + + + @@ -6545,6 +7048,14 @@ + + + + + + + + @@ -6553,6 +7064,14 @@ + + + + + + + + @@ -6585,6 +7104,14 @@ + + + + + + + + @@ -6630,6 +7157,14 @@ + + + + + + + + @@ -6726,6 +7261,14 @@ + + + + + + + + @@ -6749,6 +7292,11 @@ + + + + + @@ -6807,11 +7355,27 @@ + + + + + + + + + + + + + + + + @@ -6820,6 +7384,14 @@ + + + + + + + + @@ -6840,16 +7412,31 @@ + + + + + + + + + + + + + + + @@ -6886,6 +7473,12 @@ + + + + + + @@ -6973,6 +7566,11 @@ + + + + + @@ -6989,6 +7587,14 @@ + + + + + + + + @@ -7005,6 +7611,14 @@ + + + + + + + + @@ -7036,6 +7650,11 @@ + + + + + @@ -7581,6 +8200,14 @@ + + + + + + + + @@ -8079,6 +8706,14 @@ + + + + + + + + @@ -8100,6 +8735,14 @@ + + + + + + + + @@ -8116,6 +8759,22 @@ + + + + + + + + + + + + + + + + @@ -8132,6 +8791,14 @@ + + + + + + + + @@ -8148,6 +8815,14 @@ + + + + + + + + @@ -8164,6 +8839,14 @@ + + + + + + + + @@ -8188,6 +8871,22 @@ + + + + + + + + + + + + + + + + @@ -8198,6 +8897,14 @@ + + + + + + + + @@ -8206,6 +8913,14 @@ + + + + + + + + @@ -8214,6 +8929,14 @@ + + + + + + + + @@ -8222,6 +8945,14 @@ + + + + + + + + @@ -8238,6 +8969,14 @@ + + + + + + + + @@ -8254,6 +8993,14 @@ + + + + + + + + @@ -8262,6 +9009,14 @@ + + + + + + + + @@ -8270,6 +9025,14 @@ + + + + + + + + @@ -8286,11 +9049,24 @@ + + + + + + + + + + + + + @@ -8299,6 +9075,14 @@ + + + + + + + + @@ -8307,6 +9091,14 @@ + + + + + + + + @@ -8347,6 +9139,14 @@ + + + + + + + + @@ -8514,6 +9314,14 @@ + + + + + + + + @@ -8571,6 +9379,11 @@ + + + + + @@ -8581,6 +9394,14 @@ + + + + + + + + @@ -8605,6 +9426,14 @@ + + + + + + + + @@ -8658,11 +9487,26 @@ + + + + + + + + + + + + + + + @@ -8671,6 +9515,22 @@ + + + + + + + + + + + + + + + + @@ -8786,11 +9646,21 @@ + + + + + + + + + + @@ -8799,11 +9669,24 @@ + + + + + + + + + + + + + @@ -8812,6 +9695,14 @@ + + + + + + + + @@ -8844,6 +9735,14 @@ + + + + + + + + @@ -8868,6 +9767,14 @@ + + + + + + + + @@ -8892,6 +9799,14 @@ + + + + + + + + @@ -8967,6 +9882,11 @@ + + + + + @@ -10727,11 +11647,40 @@ + + + + + + + + + + + + + + + + + + + + + + + + + + + + + @@ -10830,6 +11779,9 @@ + + + @@ -10864,6 +11816,9 @@ + + + @@ -10905,6 +11860,9 @@ + + + @@ -10942,6 +11900,9 @@ + + + @@ -10976,6 +11937,9 @@ + + + @@ -11033,6 +11997,14 @@ + + + + + + + + @@ -11046,6 +12018,11 @@ + + + + + @@ -11076,6 +12053,11 @@ + + + + + @@ -11089,6 +12071,14 @@ + + + + + + + + @@ -11128,6 +12118,14 @@ + + + + + + + + diff --git a/scripts/analysis/findbugs-results.txt b/scripts/analysis/findbugs-results.txt index 2efea5198..a3090d211 100644 --- a/scripts/analysis/findbugs-results.txt +++ b/scripts/analysis/findbugs-results.txt @@ -1 +1 @@ -167 \ No newline at end of file +168 \ No newline at end of file