fix: properly validate hmac strings

frytg · Dec 17, 2024 · 74a7ed0 · 74a7ed0
1 parent 7268e4a
commit 74a7ed0
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/crypto/hmac-buffer.test.ts b/crypto/hmac-buffer.test.ts
@@ -90,14 +90,16 @@ test('bufferFromHex - converts hex strings to Buffer correctly', () => {
 	}
 })
 
-test('bufferFromHex - validates hex strings correctly', () => {
+test('bufferFromHex - validates valid hex strings correctly', () => {
 	// Valid hex strings should work
 	const validHexes = ['0123456789abcdef', 'ABCDEF', '', '00', 'ff', 'deadbeef']
 
 	for (const hex of validHexes) {
 		assertEquals(typeof bufferFromHex(hex), 'object', `bufferFromHex should accept valid hex string "${hex}"`)
 	}
+})
 
+test('bufferFromHex - validates invalid hex strings correctly', () => {
 	// Invalid hex strings should throw
 	const invalidHexes = [
 		'0123456789abcdefg', // invalid hex char

diff --git a/crypto/hmac.ts b/crypto/hmac.ts
@@ -11,6 +11,7 @@
  */
 
 // load packages
+// deno-fmt-ignore
 import { Buffer, atob } from 'node:buffer'
 import { createHmac } from 'node:crypto'