Skip to content

Commit 1f350aa

Browse files
committed
refactor code style
1 parent 8afa810 commit 1f350aa

File tree

6 files changed

+10
-24
lines changed

6 files changed

+10
-24
lines changed

cosec-jwt/src/main/kotlin/me/ahoo/cosec/jwt/Jwts.kt

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -53,12 +53,11 @@ object Jwts {
5353

5454
@JvmStatic
5555
fun parseAccessToken(authorization: String?): SimpleAccessToken? {
56-
if (authorization != null && authorization.startsWith(TOKEN_PREFIX)
57-
) {
58-
val accessToken = authorization.substring(TOKEN_PREFIX.length)
59-
return SimpleAccessToken(accessToken)
56+
if (authorization?.startsWith(TOKEN_PREFIX) != true) {
57+
return null
6058
}
61-
return null
59+
val accessToken = authorization.substring(TOKEN_PREFIX.length)
60+
return SimpleAccessToken(accessToken)
6261
}
6362

6463
fun decode(token: String): DecodedJWT {

cosec-spring-boot-starter/src/test/kotlin/me/ahoo/cosec/spring/boot/starter/authorization/CoSecAuthorizationAutoConfigurationTest.kt

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,6 @@ import me.ahoo.cosec.spring.boot.starter.authentication.CoSecAuthenticationAutoC
2020
import me.ahoo.cosec.spring.boot.starter.authorization.cache.CoSecCacheAutoConfiguration
2121
import me.ahoo.cosec.spring.boot.starter.jwt.CoSecJwtAutoConfiguration
2222
import me.ahoo.cosec.spring.boot.starter.jwt.JwtProperties
23-
import me.ahoo.cosec.token.TokenCompositeAuthentication
2423
import me.ahoo.cosid.IdGenerator
2524
import me.ahoo.cosid.test.MockIdGenerator
2625
import org.assertj.core.api.AssertionsForInterfaceTypes.assertThat
@@ -36,7 +35,7 @@ internal class CoSecAuthorizationAutoConfigurationTest {
3635
fun contextLoads() {
3736
contextRunner
3837
.withPropertyValues(
39-
"${JwtProperties.PREFIX}.secret=FyN0Igd80Gas8stTavArGKOYnS9uLwGA_",
38+
"${JwtProperties.PREFIX}.secret=FyN0Igd80Gas8stTavArGKOYnS9uLwGA_"
4039
)
4140
.withBean(IdGenerator::class.java, { MockIdGenerator.INSTANCE })
4241
.withUserConfiguration(

cosec-spring-boot-starter/src/test/kotlin/me/ahoo/cosec/spring/boot/starter/authorization/gateway/CoSecGatewayAuthorizationAutoConfigurationTest.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ internal class CoSecGatewayAuthorizationAutoConfigurationTest {
3434
fun contextLoads() {
3535
contextRunner
3636
.withPropertyValues(
37-
"${JwtProperties.PREFIX}.secret=FyN0Igd80Gas8stTavArGKOYnS9uLwGA_",
37+
"${JwtProperties.PREFIX}.secret=FyN0Igd80Gas8stTavArGKOYnS9uLwGA_"
3838
)
3939
.withBean(IdGenerator::class.java, { MockIdGenerator.INSTANCE })
4040
.withUserConfiguration(

cosec-spring-boot-starter/src/test/kotlin/me/ahoo/cosec/spring/boot/starter/jwt/CoSecJwtAutoConfigurationTest.kt

Lines changed: 3 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -14,25 +14,14 @@
1414
package me.ahoo.cosec.spring.boot.starter.jwt
1515

1616
import com.auth0.jwt.algorithms.Algorithm
17-
import me.ahoo.cache.spring.boot.starter.CoCacheAutoConfiguration
18-
import me.ahoo.cosec.api.authorization.Authorization
19-
import me.ahoo.cosec.servlet.AuthorizationFilter
2017
import me.ahoo.cosec.spring.boot.starter.authentication.CoSecAuthenticationAutoConfiguration
21-
import me.ahoo.cosec.spring.boot.starter.authentication.oauth.OAuthClientAuthenticationProperties
22-
import me.ahoo.cosec.spring.boot.starter.authorization.AuthorizationProperties
23-
import me.ahoo.cosec.spring.boot.starter.authorization.CoSecAuthorizationAutoConfiguration
24-
import me.ahoo.cosec.spring.boot.starter.authorization.cache.CoSecCacheAutoConfiguration
2518
import me.ahoo.cosec.token.TokenCompositeAuthentication
2619
import me.ahoo.cosec.token.TokenConverter
2720
import me.ahoo.cosec.token.TokenVerifier
2821
import me.ahoo.cosid.IdGenerator
2922
import me.ahoo.cosid.test.MockIdGenerator
3023
import org.assertj.core.api.AssertionsForInterfaceTypes
31-
import org.junit.jupiter.api.Assertions.*
32-
import org.hamcrest.MatcherAssert.assertThat
33-
import org.hamcrest.Matchers.*
3424
import org.junit.jupiter.api.Test
35-
import org.springframework.boot.autoconfigure.data.redis.RedisAutoConfiguration
3625
import org.springframework.boot.test.context.assertj.AssertableApplicationContext
3726
import org.springframework.boot.test.context.runner.ApplicationContextRunner
3827

@@ -43,7 +32,7 @@ class CoSecJwtAutoConfigurationTest {
4332
fun contextLoads() {
4433
contextRunner
4534
.withPropertyValues(
46-
"${JwtProperties.PREFIX}.secret=FyN0Igd80Gas8stTavArGKOYnS9uLwGA_",
35+
"${JwtProperties.PREFIX}.secret=FyN0Igd80Gas8stTavArGKOYnS9uLwGA_"
4736
)
4837
.withBean(IdGenerator::class.java, { MockIdGenerator.INSTANCE })
4938
.withUserConfiguration(
@@ -63,7 +52,7 @@ class CoSecJwtAutoConfigurationTest {
6352
fun contextLoadsWhenDisable() {
6453
contextRunner
6554
.withPropertyValues(
66-
"${JwtProperties.PREFIX}.enabled=false",
55+
"${JwtProperties.PREFIX}.enabled=false"
6756
)
6857
.withUserConfiguration(
6958
CoSecJwtAutoConfiguration::class.java
@@ -76,4 +65,4 @@ class CoSecJwtAutoConfigurationTest {
7665
.doesNotHaveBean(TokenVerifier::class.java)
7766
}
7867
}
79-
}
68+
}

cosec-webflux/src/test/kotlin/me/ahoo/cosec/webflux/SecurityContextParserSpec.kt

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,6 @@ abstract class SecurityContextParserSpec {
3838
}
3939
}
4040

41-
4241
val jwtTokenVerifier = JwtTokenVerifier(algorithm)
4342
abstract fun createSecurityContextParser(): SecurityContextParser<ServerWebExchange>
4443

gradle.properties

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
# limitations under the License.
1212
#
1313
group=me.ahoo.cosec
14-
version=1.5.2
14+
version=1.5.3
1515
description=RBAC-based And Policy-based Multi-Tenant Reactive Security Framework
1616
website=https://github.com/Ahoo-Wang/CoSec
1717
issues=https://github.com/Ahoo-Wang/CoSec/issues

0 commit comments

Comments
 (0)