Merge pull request '1.5.0 Release' (#5) from develop into main

Reviewed-on: Infendro/mac-kt#5

.gitea/workflows/ci.yaml

@@ -1,92 +0,0 @@
-on:
-  push:
-    branches: [ develop, main ]
-  pull_request:
-    branches: [ main ]
-
-jobs:
-  test:
-    runs-on: linux
-    steps:
-      - name: Checkout
-        uses: actions/checkout@v4
-
-      - name: Setup Java
-        uses: actions/setup-java@v4
-        with:
-          java-version: '24'
-          distribution: 'temurin'
-
-      - name: Cache Gradle
-        uses: actions/cache@v4
-        with:
-          key: gradle-${{ hashFiles('**/libs.versions.toml') }}
-          restore-keys: gradle-
-          path: |
-            ~/.gradle/caches/
-            ~/.gradle/wrapper/
-
-      - name: Cache Konan
-        uses: actions/cache@v4
-        with:
-          key: konan-${{ hashFiles('**/libs.versions.toml') }}
-          restore-keys: konan-
-          path: |
-            ~/.konan/
-
-      - name: Cache Node
-        uses: actions/cache@v4
-        with:
-          key: node-${{ hashFiles('**/libs.versions.toml') }}
-          restore-keys: node-
-          path: |
-            ~/.gradle/nodejs
-            ~/.gradle/yarn
-
-      - name: Test
-        run: ./gradlew allTests
-
-  publish:
-    if: ${{ github.event_name == 'push' && github.ref_name == 'main' }}
-    needs: test
-    runs-on: linux
-    steps:
-      - name: Checkout
-        uses: actions/checkout@v4
-
-      - name: Setup Java
-        uses: actions/setup-java@v4
-        with:
-          java-version: '24'
-          distribution: 'temurin'
-
-      - name: Cache Gradle
-        uses: actions/cache@v4
-        with:
-          key: gradle-${{ hashFiles('**/libs.versions.toml') }}
-          restore-keys: gradle-
-          path: |
-            ~/.gradle/caches/
-            ~/.gradle/wrapper/
-
-      - name: Cache Konan
-        uses: actions/cache@v4
-        with:
-          key: konan-${{ hashFiles('**/libs.versions.toml') }}
-          restore-keys: konan-
-          path: |
-            ~/.konan/
-
-      - name: Cache Node
-        uses: actions/cache@v4
-        with:
-          key: node-${{ hashFiles('**/libs.versions.toml') }}
-          restore-keys: node-
-          path: |
-            ~/.gradle/nodejs
-            ~/.gradle/yarn
-
-      - name: Publish
-        run: ./gradlew publish
-        env:
-          INFENDRO__TOKEN: ${{ secrets.TOKEN }}

.gitea/workflows/publish.yaml

@@ -0,0 +1,47 @@
+on:
+  push:
+    branches: [ main ]
+
+jobs:
+  publish:
+    runs-on: linux
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+
+      - name: Setup Java
+        uses: actions/setup-java@v4
+        with:
+          java-version: '24'
+          distribution: 'temurin'
+
+      - name: Cache Gradle
+        uses: actions/cache@v4
+        with:
+          key: gradle-${{ hashFiles('**/libs.versions.toml') }}
+          restore-keys: gradle
+          path: |
+            ~/.gradle/caches/
+            ~/.gradle/wrapper/
+
+      - name: Cache Konan
+        uses: actions/cache@v4
+        with:
+          key: konan-${{ hashFiles('**/libs.versions.toml') }}
+          restore-keys: konan
+          path: |
+            ~/.konan/
+
+      - name: Cache Node
+        uses: actions/cache@v4
+        with:
+          key: node-${{ hashFiles('**/libs.versions.toml') }}
+          restore-keys: node
+          path: |
+            ~/.gradle/nodejs
+            ~/.gradle/yarn
+
+      - name: Publish
+        run: ./gradlew publish
+        env:
+          INFENDRO__TOKEN: ${{ secrets.TOKEN }}

.gitea/workflows/test.yaml

@@ -0,0 +1,28 @@
+on:
+  pull_request:
+    branches: [ main ]
+
+jobs:
+  test:
+    runs-on: linux
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+
+      - name: Setup Java
+        uses: actions/setup-java@v4
+        with:
+          java-version: '24'
+          distribution: 'temurin'
+
+      - name: Cache Gradle
+        uses: actions/cache@v4
+        with:
+          key: gradle-${{ hashFiles('**/libs.versions.toml') }}
+          restore-keys: gradle
+          path: |
+            ~/.gradle/caches/
+            ~/.gradle/wrapper/
+
+      - name: Test
+        run: ./gradlew jvmTest

README.md

@@ -15,7 +15,7 @@ repositories {
 }
 
 dependencies {
-    implementation("com.infendro:mac:1.3.0")
+    implementation("com.infendro:mac:1.5.0")
 }
 ```
 
@@ -24,7 +24,7 @@ dependencies {
 ```kotlin
 fun main() {
     // create an HMAC instance using SHA1
-    val hmac = HMAC(SHA1)
+    val hmac = HMAC(SHA1())
 
     // generate MAC
     val key = "secret".encodeToByteArray()

build.gradle.kts

@@ -5,7 +5,7 @@ import com.infendro.plugin.token
 import org.jetbrains.kotlin.gradle.ExperimentalWasmDsl
 
 group = "com.infendro"
-version = "1.3.0"
+version = "1.5.0"
 
 plugins {
     alias(libs.plugins.infendro)
@@ -31,17 +31,13 @@ kotlin {
 
     sourceSets {
         commonMain.dependencies {
-            implementation(libs.hash)
             implementation(libs.bytes)
+            implementation(libs.hash)
         }
         commonTest.dependencies {
             implementation(libs.test)
         }
     }
-
-    compilerOptions {
-        freeCompilerArgs.add("-opt-in=kotlin.ExperimentalUnsignedTypes")
-    }
 }
 
 publishing.repositories {

gradle/libs.versions.toml

@@ -1,14 +1,14 @@
 [versions]
 infendro = "1.1.0"
 kotlin = "2.3.0"
-hash = "1.5.1"
-bytes = "1.3.3"
+bytes = "1.4.1"
+hash = "1.7.0"
 
 [plugins]
 infendro = { id = "com.infendro.plugin", version.ref = "infendro" }
 multiplatform = { id = "org.jetbrains.kotlin.multiplatform", version.ref = "kotlin" }
 
 [libraries]
-hash = { module = "com.infendro:hash", version.ref = "hash" }
 bytes = { module = "com.infendro:bytes", version.ref = "bytes" }
+hash = { module = "com.infendro:hash", version.ref = "hash" }
 test = { module = "org.jetbrains.kotlin:kotlin-test", version.ref = "kotlin" }

src/commonMain/kotlin/com/infendro/mac/HMAC.kt

@@ -5,28 +5,59 @@ import com.infendro.hash.HashFunction
 
 class HMAC(
     val function: HashFunction,
-) : MAC {
-    override val bytes: Int
-        get() = function.bytes
+) : MAC(function.bytes) {
+    private var initialized = false
+    private val innerPadding: ByteArray = ByteArray(function.block)
+    private val outerPadding: ByteArray = ByteArray(function.block)
+    private val inner: ByteArray = ByteArray(function.bytes)
 
-    override fun hashInto(key: ByteArray, value: ByteArray, destination: ByteArray, destinationOffset: Int) {
-        val paddedKey = pad(key)
-        val inner = ByteArray(function.block + value.size).also {
-            paddedKey.xorInto(0x36, it)
-            value.copyInto(it, function.block)
-        }
-        val outer = ByteArray(function.block + function.bytes).also {
-            paddedKey.xorInto(0x5C, it)
-            function.hashInto(inner, it, function.block)
-        }
-        function.hashInto(outer, destination, destinationOffset)
+    override fun reset() {
+        function.reset()
+        initialized = false
+        innerPadding.fill(0x00)
+        outerPadding.fill(0x00)
     }
 
-    private fun pad(key: ByteArray): ByteArray =
-        ByteArray(function.block).also {
+    override fun init(key: ByteArray) {
+        val padded = pad(key)
+        padded.xorInto(0x36, innerPadding)
+        padded.xorInto(0x5C, outerPadding)
+
+        function.reset()
+        function.update(innerPadding)
+
+        initialized = true
+    }
+
+    override fun update(byte: Byte) {
+        require(initialized)
+
+        function.update(byte)
+    }
+
+    override fun update(bytes: ByteArray) {
+        require(initialized)
+
+        function.update(bytes)
+    }
+
+    override fun digestInto(destination: ByteArray, destinationOffset: Int) {
+        require(initialized)
+
+        function.digestInto(inner)
+        function.update(outerPadding)
+        function.update(inner)
+        function.digestInto(destination, destinationOffset)
+
+        function.update(innerPadding)
+    }
+
+    private fun pad(key: ByteArray): ByteArray {
+        val padded = ByteArray(function.block)
         when {
-                key.size > function.block -> function.hashInto(key, it)
-                else -> key.copyInto(it)
-            }
+            key.size > function.block -> function.hashInto(key, padded)
+            else -> key.copyInto(padded)
+        }
+        return padded
     }
 }

src/commonMain/kotlin/com/infendro/mac/MAC.kt

@@ -1,24 +1,38 @@
 package com.infendro.mac
 
-interface MAC {
-    val bytes: Int
+abstract class MAC(
+    val bytes: Int,
+) {
     val bits: Int
         get() = bytes * 8
 
-    fun hashInto(key: ByteArray, value: ByteArray, destination: ByteArray, destinationOffset: Int = 0)
+    abstract fun reset()
 
-    fun hashInto(key: ByteArray, value: ByteArray, destination: UByteArray, destinationOffset: Int = 0) =
-        hashInto(key, value, destination.asByteArray(), destinationOffset)
+    abstract fun init(key: ByteArray)
 
-    fun hashInto(key: UByteArray, value: UByteArray, destination: ByteArray, destinationOffset: Int = 0) =
-        hashInto(key.asByteArray(), value.asByteArray(), destination, destinationOffset)
+    abstract fun update(byte: Byte)
+    abstract fun update(bytes: ByteArray)
 
-    fun hashInto(key: UByteArray, value: UByteArray, destination: UByteArray, destinationOffset: Int = 0) =
-        hashInto(key.asByteArray(), value.asByteArray(), destination.asByteArray(), destinationOffset)
+    abstract fun digestInto(destination: ByteArray, destinationOffset: Int = 0)
 
-    fun hash(key: ByteArray, value: ByteArray): ByteArray =
-        ByteArray(bytes).also { hashInto(key, value, it) }
-
-    fun hash(key: UByteArray, value: UByteArray): UByteArray =
-        UByteArray(bytes).also { hashInto(key, value, it) }
+    fun digest(): ByteArray {
+        val result = ByteArray(bytes)
+        digestInto(result)
+        return result
+    }
+
+    fun hashInto(key: ByteArray, value: ByteArray, destination: ByteArray, destinationOffset: Int = 0) {
+        init(key)
+        update(value)
+        digestInto(destination, destinationOffset)
+        reset()
+    }
+
+    fun hash(key: ByteArray, value: ByteArray): ByteArray {
+        init(key)
+        update(value)
+        val result = digest()
+        reset()
+        return result
+    }
 }

src/commonTest/kotlin/com/infendro/mac/HMAC.kt

@@ -6,7 +6,7 @@ import kotlin.test.assertContentEquals
 import kotlin.test.assertEquals
 
 class `HMAC Test` {
-    val hmac = HMAC(`SHA-256`)
+    val hmac = HMAC(`SHA-256`())
 
     @Test
     fun bytes() {
@@ -16,16 +16,12 @@ class `HMAC Test` {
 
     @Test
     fun example() {
-        val expected = ubyteArrayOf(
-            0xf7U, 0xbcU, 0x83U, 0xf4U,
-            0x30U, 0x53U, 0x84U, 0x24U,
-            0xb1U, 0x32U, 0x98U, 0xe6U,
-            0xaaU, 0x6fU, 0xb1U, 0x43U,
-            0xefU, 0x4dU, 0x59U, 0xa1U,
-            0x49U, 0x46U, 0x17U, 0x59U,
-            0x97U, 0x47U, 0x9dU, 0xbcU,
-            0x2dU, 0x1aU, 0x3cU, 0xd8U,
-        ).asByteArray()
+        val expected = byteArrayOf(
+            -0x09, -0x44, -0x7d, -0x0c, +0x30, +0x53, -0x7c, +0x24,
+            -0x4f, +0x32, -0x68, -0x1a, -0x56, +0x6f, -0x4f, +0x43,
+            -0x11, +0x4d, +0x59, -0x5f, +0x49, +0x46, +0x17, +0x59,
+            -0x69, +0x47, -0x63, -0x44, +0x2d, +0x1a, +0x3c, -0x28,
+        )
         val actual = hmac.hash(
             "key".encodeToByteArray(),
             "The quick brown fox jumps over the lazy dog".encodeToByteArray(),