Skip to content

chore(iroh): Update noq commit, stop exposing AeadKey#4003

Merged
matheus23 merged 2 commits intomainfrom
matheus23/update-noq
Mar 9, 2026
Merged

chore(iroh): Update noq commit, stop exposing AeadKey#4003
matheus23 merged 2 commits intomainfrom
matheus23/update-noq

Conversation

@matheus23
Copy link
Copy Markdown
Member

Description

Updates to the latest noq main branch.

noq recently stopped exposing trait AeadKey, because the trait HandshakeTokenKey doesn't have a function returning Box<dyn AeadKey> anymore (instead it is left up to the implementation to use appropriate AEAD constructions with or without an HKDF in front).

(Also fix the iroh unit test suite to allow compiling without the metrics feature.)

Breaking Changes

  • Stop exposing trait AeadKey and change trait HandshakeTokenKey to implement seal and open on its own, leaving choice of using an HKDF to the user.

@matheus23 matheus23 added this to the iroh: v0.97 milestone Mar 9, 2026
@matheus23 matheus23 self-assigned this Mar 9, 2026
@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 9, 2026

Documentation for this PR has been generated and is available at: https://n0-computer.github.io/iroh/pr/4003/docs/iroh/

Last updated: 2026-03-09T09:46:00Z

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 9, 2026

Netsim report & logs for this PR have been generated and is available at: LOGS
This report will remain available for 3 days.

Last updated for commit: f65cd0b

@dignifiedquire
Copy link
Copy Markdown
Contributor

this is a breaking change, no? please document if so properly

@matheus23
Copy link
Copy Markdown
Member Author

this is a breaking change, no? please document if so properly

There already is a section in the breaking changes list.

@matheus23 matheus23 added this pull request to the merge queue Mar 9, 2026
Merged via the queue into main with commit bf07082 Mar 9, 2026
28 of 29 checks passed
@matheus23 matheus23 deleted the matheus23/update-noq branch March 9, 2026 10:55
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.

2 participants