Dockerfile

@@ -21,7 +21,7 @@ RUN chown -R $APPLICATION_USER /app
 USER $APPLICATION_USER
 
 COPY --from=jdkbuilder /myjdk /myjdk
-COPY app/build/libs/app-with-dependencies*.jar /app/simplenotes.jar
+COPY simplenotes-app/build/libs/simplenotes-app-with-dependencies*.jar /app/simplenotes.jar
 WORKDIR /app
 
 CMD [ \

app/test/Index.kt

@@ -1 +0,0 @@
-package be.simplenotes.app

buildSrc/src/main/kotlin/be/simplenotes/CssTask.kt

@@ -15,15 +15,15 @@ open class CssTask : DefaultTask() {
 
     private val viewsProject = project
         .parent
-        ?.project(":views")
+        ?.project(":simplenotes-views")
-        ?: error("Missing :views")
+        ?: error("Missing :simplenotes-views")
 
     @get:InputDirectory
     val templatesDir = viewsProject.extensions
         .getByType<SourceSetContainer>()
         .asMap.getOrElse("main") { error("main sources not found") }
         .allSource.srcDirs
-        .find { it.endsWith("src") }
+        .find { it.endsWith("kotlin") }
         ?: error("kotlin sources not found")
 
     private val yarnRoot = File(project.rootDir, "css")

buildSrc/src/main/kotlin/be/simplenotes/java-convention.gradle.kts

@@ -24,6 +24,3 @@ java {
 tasks.withType<JavaCompile> {
     options.encoding = "UTF-8"
 }
-
-sourceSets["main"].resources.srcDirs("resources")
-sourceSets["test"].resources.srcDirs("testresources")

buildSrc/src/main/kotlin/be/simplenotes/junit-convention.gradle.kts

@@ -10,5 +10,5 @@ tasks.withType<Test> {
 
 sourceSets {
     val test by getting
-    test.resources.srcDir("${rootProject.projectDir}/testresources/src/test/resources")
+    test.resources.srcDir("${rootProject.projectDir}/simplenotes-test-resources/src/test/resources")
 }

buildSrc/src/main/kotlin/be/simplenotes/kotlin-convention.gradle.kts

@@ -23,6 +23,3 @@ tasks.withType<KotlinCompile> {
         )
     }
 }
-
-kotlin.sourceSets["main"].kotlin.srcDirs("src")
-kotlin.sourceSets["test"].kotlin.srcDirs("test")

domain/src/Index.kt

@@ -1 +0,0 @@
-package be.simplenotes.domain

domain/src/security/JwtMapper.kt

@@ -1,30 +0,0 @@
-package be.simplenotes.domain.security
-
-import be.simplenotes.types.LoggedInUser
-import com.auth0.jwt.JWTCreator
-import com.auth0.jwt.interfaces.DecodedJWT
-import javax.inject.Singleton
-
-interface JwtMapper<T> {
-    fun extract(decodedJWT: DecodedJWT): T?
-    fun build(builder: JWTCreator.Builder, value: T)
-}
-
-@Singleton
-class UserJwtMapper : JwtMapper<LoggedInUser> {
-    private val userIdField = "i"
-    private val usernameField = "u"
-
-    override fun extract(decodedJWT: DecodedJWT): LoggedInUser? {
-        val id = decodedJWT.getClaim(userIdField).asInt() ?: null
-        val username = decodedJWT.getClaim(usernameField).asString() ?: null
-        return if (id != null && username != null)
-            LoggedInUser(id, username)
-        else null
-    }
-
-    override fun build(builder: JWTCreator.Builder, value: LoggedInUser) {
-        builder.withClaim(userIdField, value.userId)
-            .withClaim(usernameField, value.username)
-    }
-}

domain/test/Index.kt

@@ -1 +0,0 @@
-package be.simplenotes.domain

settings.gradle.kts

@@ -1,8 +1,8 @@
 rootProject.name = "simplenotes"
-include(":config")
+include(":simplenotes-config")
-include(":views")
+include(":simplenotes-views")
-include(":app")
+include(":simplenotes-app")
-include(":domain")
+include(":simplenotes-domain")
-include(":search")
+include(":simplenotes-search")
-include(":types")
+include(":simplenotes-types")
-include(":persistance")
+include(":simplenotes-persistance")

simplenotes-app/build.gradle.kts

@@ -10,10 +10,10 @@ plugins {
 }
 
 dependencies {
-    implementation(project(":domain"))
+    implementation(project(":simplenotes-domain"))
-    implementation(project(":types"))
+    implementation(project(":simplenotes-types"))
-    implementation(project(":config"))
+    implementation(project(":simplenotes-config"))
-    implementation(project(":views"))
+    implementation(project(":simplenotes-views"))
 
     implementation(Libs.arrowCoreData)
     implementation(Libs.konform)

simplenotes-app/src/main/kotlin/be/simplenotes/app/filters/auth/OptionalAuthFilter.kt

@@ -1,14 +1,13 @@
 package be.simplenotes.app.filters.auth
 
 import be.simplenotes.app.filters.auth.JwtSource.Cookie
-import be.simplenotes.domain.security.SimpleJwt
+import be.simplenotes.domain.security.JwtPayloadExtractor
-import be.simplenotes.types.LoggedInUser
 import org.http4k.core.Filter
 import org.http4k.core.HttpHandler
 import org.http4k.core.with
 
 class OptionalAuthFilter(
-    private val simpleJwt: SimpleJwt<LoggedInUser>,
+    private val extractor: JwtPayloadExtractor,
     private val lens: OptionalAuthLens,
     private val source: JwtSource = Cookie,
 ) : Filter {
@@ -18,6 +17,6 @@ class OptionalAuthFilter(
             Cookie -> it.bearerTokenCookie()
         }
 
-        next(it.with(lens of token?.let { simpleJwt.extract(it) }))
+        next(it.with(lens of token?.let { extractor(it) }))
     }
 }

simplenotes-app/src/main/kotlin/be/simplenotes/app/filters/auth/RequiredAuthFilter.kt

@@ -1,8 +1,7 @@
 package be.simplenotes.app.filters.auth
 
 import be.simplenotes.app.extensions.redirect
-import be.simplenotes.domain.security.SimpleJwt
+import be.simplenotes.domain.security.JwtPayloadExtractor
-import be.simplenotes.types.LoggedInUser
 import org.http4k.core.Filter
 import org.http4k.core.HttpHandler
 import org.http4k.core.Response
@@ -10,7 +9,7 @@ import org.http4k.core.Status.Companion.UNAUTHORIZED
 import org.http4k.core.with
 
 class RequiredAuthFilter(
-    private val simpleJwt: SimpleJwt<LoggedInUser>,
+    private val extractor: JwtPayloadExtractor,
     private val lens: RequiredAuthLens,
     private val source: JwtSource = JwtSource.Cookie,
     private val redirect: Boolean = true,
@@ -20,7 +19,7 @@ class RequiredAuthFilter(
             JwtSource.Header -> it.bearerTokenHeader()
             JwtSource.Cookie -> it.bearerTokenCookie()
         }
-        val jwtPayload = token?.let { simpleJwt.extract(token) }
+        val jwtPayload = token?.let { extractor(token) }
 
         if (jwtPayload != null) next(it.with(lens of jwtPayload))
         else {

simplenotes-app/src/main/kotlin/be/simplenotes/app/modules/AuthModule.kt

@@ -1,8 +1,7 @@
 package be.simplenotes.app.modules
 
 import be.simplenotes.app.filters.auth.*
-import be.simplenotes.domain.security.SimpleJwt
+import be.simplenotes.domain.security.JwtPayloadExtractor
-import be.simplenotes.types.LoggedInUser
 import io.micronaut.context.annotation.Factory
 import io.micronaut.context.annotation.Primary
 import org.http4k.core.RequestContexts
@@ -22,21 +21,21 @@ class AuthModule {
     fun requiredAuthLens(ctx: RequestContexts): RequiredAuthLens = RequestContextKey.required(ctx)
 
     @Singleton
-    fun optionalAuth(simpleJwt: SimpleJwt<LoggedInUser>, @Named("optional") lens: OptionalAuthLens) =
+    fun optionalAuth(extractor: JwtPayloadExtractor, @Named("optional") lens: OptionalAuthLens) =
-        OptionalAuthFilter(simpleJwt, lens)
+        OptionalAuthFilter(extractor, lens)
 
     @Primary
     @Singleton
-    fun requiredAuth(simpleJwt: SimpleJwt<LoggedInUser>, @Named("required") lens: RequiredAuthLens) =
+    fun requiredAuth(extractor: JwtPayloadExtractor, @Named("required") lens: RequiredAuthLens) =
-        RequiredAuthFilter(simpleJwt, lens)
+        RequiredAuthFilter(extractor, lens)
 
     @Singleton
     @Named("api")
     internal fun apiAuthFilter(
-        simpleJwt: SimpleJwt<LoggedInUser>,
+        jwtPayloadExtractor: JwtPayloadExtractor,
         @Named("required") lens: RequiredAuthLens,
     ) = RequiredAuthFilter(
-        simpleJwt = simpleJwt,
+        extractor = jwtPayloadExtractor,
         lens = lens,
         source = JwtSource.Header,
         redirect = false

simplenotes-app/src/main/resources/logback.xml

@@ -14,5 +14,5 @@
     <logger name="com.zaxxer.hikari" level="INFO"/>
     <logger name="org.flywaydb.core" level="INFO"/>
     <logger name="io.micronaut" level="INFO"/>
-    <logger name="io.micronaut.context.lifecycle" level="INFO"/>
+    <logger name="io.micronaut.context.lifecycle" level="DEBUG"/>
 </configuration>

simplenotes-app/src/test/kotlin/be/simplenotes/app/filters/RequiredAuthFilterTest.kt

@@ -6,7 +6,6 @@ import be.simplenotes.app.filters.auth.RequiredAuthFilter
 import be.simplenotes.app.filters.auth.RequiredAuthLens
 import be.simplenotes.config.JwtConfig
 import be.simplenotes.domain.security.SimpleJwt
-import be.simplenotes.domain.security.UserJwtMapper
 import be.simplenotes.types.LoggedInUser
 import com.natpryce.hamkrest.assertion.assertThat
 import io.micronaut.context.BeanContext
@@ -33,7 +32,7 @@ internal class RequiredAuthFilterTest {
 
     // region setup
     private val jwtConfig = JwtConfig("secret", 1, TimeUnit.HOURS)
-    private val simpleJwt = SimpleJwt(jwtConfig, UserJwtMapper())
+    private val simpleJwt = SimpleJwt(jwtConfig)
 
     private val beanCtx = BeanContext.build()
         .registerSingleton(jwtConfig)

simplenotes-domain/build.gradle.kts

@@ -7,10 +7,10 @@ plugins {
 }
 
 dependencies {
-    implementation(project(":config"))
+    implementation(project(":simplenotes-config"))
-    implementation(project(":types"))
+    implementation(project(":simplenotes-types"))
-    implementation(project(":persistance"))
+    implementation(project(":simplenotes-persistance"))
-    implementation(project(":search"))
+    implementation(project(":simplenotes-search"))
 
     implementation(Libs.micronaut)
     kapt(Libs.micronautProcessor)

simplenotes-domain/src/main/kotlin/be/simplenotes/domain/security/JwtPayloadExtractor.kt

@@ -0,0 +1,19 @@
+package be.simplenotes.domain.security
+
+import be.simplenotes.types.LoggedInUser
+import com.auth0.jwt.exceptions.JWTVerificationException
+import javax.inject.Singleton
+
+@Singleton
+class JwtPayloadExtractor(private val jwt: SimpleJwt) {
+    operator fun invoke(token: String): LoggedInUser? = try {
+        val decodedJWT = jwt.verifier.verify(token)
+        val id = decodedJWT.getClaim(userIdField).asInt() ?: null
+        val username = decodedJWT.getClaim(usernameField).asString() ?: null
+        id?.let { username?.let { LoggedInUser(id, username) } }
+    } catch (e: JWTVerificationException) {
+        null
+    } catch (e: IllegalArgumentException) {
+        null
+    }
+}

simplenotes-domain/src/main/kotlin/be/simplenotes/domain/security/SimpleJwt.kt

@@ -1,33 +1,28 @@
 package be.simplenotes.domain.security
 
 import be.simplenotes.config.JwtConfig
+import be.simplenotes.types.LoggedInUser
 import com.auth0.jwt.JWT
 import com.auth0.jwt.JWTVerifier
 import com.auth0.jwt.algorithms.Algorithm
-import com.auth0.jwt.exceptions.JWTVerificationException
 import java.util.*
 import java.util.concurrent.TimeUnit
 import javax.inject.Singleton
 
+internal const val userIdField = "i"
+internal const val usernameField = "u"
+
 @Singleton
-class SimpleJwt<T>(jwtConfig: JwtConfig, private val mapper: JwtMapper<T>) {
+class SimpleJwt(jwtConfig: JwtConfig) {
     private val validityInMs = TimeUnit.MILLISECONDS.convert(jwtConfig.validity, jwtConfig.timeUnit)
     private val algorithm = Algorithm.HMAC256(jwtConfig.secret)
-    private val verifier: JWTVerifier = JWT.require(algorithm).build()
 
-    fun sign(value: T): String = JWT.create()
+    val verifier: JWTVerifier = JWT.require(algorithm).build()
-        .apply { mapper.build(this, value) }
+    fun sign(loggedInUser: LoggedInUser): String = JWT.create()
+        .withClaim(userIdField, loggedInUser.userId)
+        .withClaim(usernameField, loggedInUser.username)
         .withExpiresAt(getExpiration())
         .sign(algorithm)
 
-    fun extract(token: String): T? = try {
-        val decodedJWT = verifier.verify(token)
-        mapper.extract(decodedJWT)
-    } catch (e: JWTVerificationException) {
-        null
-    } catch (e: IllegalArgumentException) {
-        null
-    }
-
     private fun getExpiration() = Date(System.currentTimeMillis() + validityInMs)
 }

simplenotes-domain/src/main/kotlin/be/simplenotes/domain/usecases/users/login/LoginUseCaseImpl.kt

@@ -16,7 +16,7 @@ import javax.inject.Singleton
 internal class LoginUseCaseImpl(
     private val userRepository: UserRepository,
     private val passwordHash: PasswordHash,
-    private val jwt: SimpleJwt<LoggedInUser>
+    private val jwt: SimpleJwt
 ) : LoginUseCase {
     override fun login(form: LoginForm) = either.eager<LoginError, Token> {
         val user = !UserValidations.validateLogin(form)

simplenotes-domain/src/test/kotlin/be/simplenotes/domain/security/LoggedInUserExtractorTest.kt

@@ -16,13 +16,14 @@ import java.util.stream.Stream
 
 internal class LoggedInUserExtractorTest {
     private val jwtConfig = JwtConfig("a secret", 1, TimeUnit.HOURS)
-    private val mapper = UserJwtMapper()
+    private val simpleJwt = SimpleJwt(jwtConfig)
-    private val simpleJwt = SimpleJwt(jwtConfig, mapper)
+    private val jwtPayloadExtractor = JwtPayloadExtractor(simpleJwt)
 
     private fun createToken(username: String? = null, id: Int? = null, secret: String = jwtConfig.secret): Token {
         val algo = Algorithm.HMAC256(secret)
         return JWT.create().apply {
-            if (username != null && id != null) mapper.build(this, LoggedInUser(id, username))
+            username?.let { withClaim(usernameField, it) }
+            id?.let { withClaim(userIdField, it) }
         }.sign(algo)
     }
 
@@ -39,12 +40,12 @@ internal class LoggedInUserExtractorTest {
     @ParameterizedTest(name = "[{index}] token `{0}` should be invalid")
     @MethodSource("invalidTokens")
     fun `parse invalid tokens`(token: String) {
-        assertThat(simpleJwt.extract(token), absent())
+        assertThat(jwtPayloadExtractor(token), absent())
     }
 
     @Test
     fun `parse valid token`() {
         val token = createToken(username = "someone", id = 1)
-        assertThat(simpleJwt.extract(token), equalTo(LoggedInUser(1, "someone")))
+        assertThat(jwtPayloadExtractor(token), equalTo(LoggedInUser(1, "someone")))
     }
 }

simplenotes-domain/src/test/kotlin/be/simplenotes/domain/usecases/users/login/LoginUseCaseImplTest.kt

@@ -3,7 +3,6 @@ package be.simplenotes.domain.usecases.users.login
 import be.simplenotes.config.JwtConfig
 import be.simplenotes.domain.security.BcryptPasswordHash
 import be.simplenotes.domain.security.SimpleJwt
-import be.simplenotes.domain.security.UserJwtMapper
 import be.simplenotes.domain.testutils.isLeftOfType
 import be.simplenotes.domain.testutils.isRight
 import be.simplenotes.persistance.repositories.UserRepository
@@ -19,7 +18,7 @@ internal class LoginUseCaseImplTest {
     private val mockUserRepository = mockk<UserRepository>()
     private val passwordHash = BcryptPasswordHash(test = true)
     private val jwtConfig = JwtConfig("a secret", 1, TimeUnit.HOURS)
-    private val simpleJwt = SimpleJwt(jwtConfig, UserJwtMapper())
+    private val simpleJwt = SimpleJwt(jwtConfig)
     private val loginUseCase = LoginUseCaseImpl(mockUserRepository, passwordHash, simpleJwt)
 
     @BeforeEach

simplenotes-persistance/build.gradle.kts

@@ -7,8 +7,8 @@ plugins {
 }
 
 dependencies {
-    implementation(project(":types"))
+    implementation(project(":simplenotes-types"))
-    implementation(project(":config"))
+    implementation(project(":simplenotes-config"))
 
     implementation(Libs.mariadbClient)
     implementation(Libs.h2)
@@ -29,9 +29,9 @@ dependencies {
     testImplementation(Libs.logbackClassic)
     testImplementation(Libs.mariaTestContainer)
 
-    testFixturesImplementation(project(":types"))
+    testFixturesImplementation(project(":simplenotes-types"))
-    testFixturesImplementation(project(":config"))
+    testFixturesImplementation(project(":simplenotes-config"))
-    testFixturesImplementation(project(":persistance"))
+    testFixturesImplementation(project(":simplenotes-persistance"))
 
     testFixturesImplementation(Libs.micronaut)
     kaptTestFixtures(Libs.micronautProcessor)
@@ -48,5 +48,3 @@ dependencies {
     testFixturesImplementation(Libs.ktormCore)
     testFixturesImplementation(Libs.hikariCP)
 }
-
-kotlin.sourceSets["testFixtures"].kotlin.srcDirs("testfixtures")

simplenotes-persistance/src/main/kotlin/be/simplenotes/persistance/extensions/KtormExtensions.kt

@@ -1,7 +1,6 @@
 package be.simplenotes.persistance.extensions
 
-import me.liuwj.ktorm.schema.BaseTable
+import me.liuwj.ktorm.schema.*
-import me.liuwj.ktorm.schema.SqlType
 import java.nio.ByteBuffer
 import java.sql.PreparedStatement
 import java.sql.ResultSet