Skip to content

Comments

MIC authentication implementation (part 2)#32

Merged
amirna2 merged 2 commits intomainfrom
implement-mic-part-2
Feb 22, 2026
Merged

MIC authentication implementation (part 2)#32
amirna2 merged 2 commits intomainfrom
implement-mic-part-2

Conversation

@amirna2
Copy link
Owner

@amirna2 amirna2 commented Jul 27, 2025

Summary

  • Replaces the initial MIC authentication implementation (reverted in main) with a corrected version
  • Adds MicService abstraction and reorganized AesCmac under src/core/protocol/src/crypto/cmac/
  • Adds MIC computation/verification to the packet send, receive, and relay paths in PacketRouter
  • Updates RadioMeshPacket with MIC extraction and append support, reducing max payload from 221 to 217 bytes

Type of Change

  • New feature
  • Breaking change (max payload size reduced from 221 to 217 bytes)

Test plan

  • Verify AES-CMAC computation with known test vectors
  • Test end-to-end packet flow with MIC enabled
  • Verify MIC recomputation on relay path
  • Test invalid MIC rejection
  • Confirm backward compatibility with v3 packets

🤖 Generated with Claude Code

Signed-off-by: Amir Nathoo <amirna2@users.noreply.github.com>
Signed-off-by: Amir Nathoo <amirna2@users.noreply.github.com>
@amirna2 amirna2 force-pushed the implement-mic-part-2 branch from 630a515 to f9293f5 Compare February 22, 2026 21:59
@amirna2 amirna2 changed the title Fixing MIC MIC authentication implementation (part 2) Feb 22, 2026
@amirna2 amirna2 merged commit ee67d1f into main Feb 22, 2026
4 checks passed
@amirna2 amirna2 deleted the implement-mic-part-2 branch February 22, 2026 22:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant