chore: upgrade linter and style

Author: sellisd

build.gradle.kts

@@ -8,7 +8,7 @@ plugins {
   val kotlinVersion = "1.9.23"
   kotlin("jvm") version kotlinVersion
   id("com.diffplug.spotless") version "6.25.0"
-  id("org.springframework.boot") version "3.3.6" apply false
+  id("org.springframework.boot") version "3.4.0" apply false
   id("io.gitlab.arturbosch.detekt") version "1.23.7"
   id("pl.allegro.tech.build.axion-release") version "1.15.5"
   id("org.jetbrains.kotlinx.kover") version "0.7.6"
@@ -84,12 +84,12 @@ configure<SpotlessExtension> {
     licenseHeader(licenseHeaderComment)
   }
   kotlin {
-    ktfmt("0.41")
+    ktfmt("0.50")
     target("**/*.kt")
     licenseHeader(licenseHeaderComment)
   }
   kotlinGradle {
-    ktfmt("0.41")
+    ktfmt("0.50")
     target("**/*.kts")
     //      licenseHeader(licenseHeaderComment, "import")
   }

src/main/kotlin/com/cosmotech/api/config/CsmApiConfiguration.kt

@@ -45,6 +45,7 @@ open class CsmApiConfiguration {
 @Order(Ordered.HIGHEST_PRECEDENCE)
 open class CsmPlatformEnvironmentPostProcessor : EnvironmentPostProcessor {
   private val log = DeferredLog()
+
   override fun postProcessEnvironment(
       environment: ConfigurableEnvironment,
       application: SpringApplication

src/main/kotlin/com/cosmotech/api/config/CsmPlatformProperties.kt

@@ -114,6 +114,7 @@ data class CsmPlatformProperties(
     ) {
       data class CsmStorageUser(val username: String, val password: String)
     }
+
     data class CsmEventBus(
         /** Define if current API use event bus within internal result data service or not */
         val enabled: Boolean = true,
@@ -236,6 +237,7 @@ data class CsmPlatformProperties(
       /** Base Loki url */
       val baseUrl: String = "http://loki.default.svc.cluster.local:3100",
   )
+
   data class Argo(
       /** Argo service base Uri */
       val baseUri: String,

src/main/kotlin/com/cosmotech/api/rbac/CsmRbac.kt

@@ -244,8 +244,7 @@ open class CsmRbac(
   internal fun getUserRole(rbacSecurity: RbacSecurity, user: String): String {
     return rbacSecurity.accessControlList
         .firstOrNull { it.id.lowercase() == user.lowercase() }
-        ?.role
-        ?: rbacSecurity.default
+        ?.role ?: rbacSecurity.default
   }
 
   internal fun getAdminCount(rbacSecurity: RbacSecurity, rolesDefinition: RolesDefinition): Int {

src/main/kotlin/com/cosmotech/api/security/filters/ApiKeyAuthenticationFilter.kt

@@ -112,6 +112,7 @@ class ApiKeyAuthentication(
   init {
     this.isAuthenticated = true
   }
+
   override fun getCredentials(): Any? {
     return null
   }

src/main/kotlin/com/cosmotech/api/security/keycloak/KeycloakSecurityConfiguration.kt

@@ -146,6 +146,7 @@ class KeycloakJwtGrantedAuthoritiesConverter(
 ) : Converter<Jwt, Collection<GrantedAuthority>> {
 
   private val logger = LoggerFactory.getLogger(KeycloakJwtGrantedAuthoritiesConverter::class.java)
+
   override fun convert(jwt: Jwt): Collection<GrantedAuthority> {
     val extractAuthorities = mutableListOf<GrantedAuthority>()
     extractAuthorities.addAll(

src/main/kotlin/com/cosmotech/api/security/okta/OktaSecurityConfiguration.kt

@@ -141,6 +141,7 @@ class OktaJwtGrantedAuthoritiesConverter(private val csmPlatformProperties: CsmP
     Converter<Jwt, Collection<GrantedAuthority>> {
 
   private val logger = LoggerFactory.getLogger(OktaJwtGrantedAuthoritiesConverter::class.java)
+
   override fun convert(jwt: Jwt): Collection<GrantedAuthority> {
     val extractAuthorities = mutableListOf<GrantedAuthority>()
     extractAuthorities.addAll(

src/main/kotlin/com/cosmotech/api/utils/AnyExtensions.kt

@@ -41,11 +41,11 @@ fun <T> T.convertToMap(): Map<String, Any> =
  * @param new the new object against which comparison is performed
  * @param mutateIfChanged whether to mutate the current object if for fields that have changed
  * @param excludedFields an array of fields to exlude from comparison. Note that fields named 'id´
- * are automatically excluded, by convention.
+ *   are automatically excluded, by convention.
  * @throws UnsupportedOperationException if the object being compared are not instances of a data
- * class
+ *   class
  * @throws IllegalArgumentException if changes were detected and {@code mutateIfChanged = true}, but
- * the field is not mutable
+ *   the field is not mutable
  */
 @Suppress("NestedBlockDepth")
 inline fun <reified T> T.compareToAndMutateIfNeeded(

src/main/kotlin/com/cosmotech/api/utils/SecurityUtils.kt

@@ -26,7 +26,7 @@ fun getCurrentAuthenticatedUserName(configuration: CsmPlatformProperties): Strin
       val jwtClaimsSet = JWTParser.parse(it).jwtClaimsSet
       jwtClaimsSet.getStringClaim(configuration.authorization.principalJwtClaim)
           ?: jwtClaimsSet.getStringClaim(configuration.authorization.applicationIdJwtClaim)
-              ?: throw IllegalStateException("User Authentication not found in Security Context")
+          ?: throw IllegalStateException("User Authentication not found in Security Context")
     } catch (e: ParseException) {
       JSONObjectUtils.parse(it)[configuration.authorization.principalJwtClaim] as String
     }
@@ -63,8 +63,7 @@ fun getCurrentAuthenticatedRoles(configuration: CsmPlatformProperties): List<Str
     } catch (e: ParseException) {
       JSONObjectUtils.parse(it)[configuration.authorization.rolesJwtClaim] as List<String>
     }
-  }
-      ?: emptyList())
+  } ?: emptyList())
 }
 
 fun <T> getValueFromAuthenticatedToken(

src/main/kotlin/com/cosmotech/api/utils/StringExtensions.kt

@@ -58,6 +58,7 @@ fun String.formatQuery(map: Map<String, String>): String {
 
 /**
  * Extract the file name from a path for example: /path/to/file.txt -> file
+ *
  * @return the file name without the extension
  */
 fun String.extractFileNameFromPath(): String {