Skip to content

handlers refactor poc - #19

Open
bit-aloo wants to merge 9 commits into
2025-06-08-noise-streamfrom
2025-07-08-handlers-refactor
Open

handlers refactor poc#19
bit-aloo wants to merge 9 commits into
2025-06-08-noise-streamfrom
2025-07-08-handlers-refactor

Conversation

@bit-aloo

@bit-aloo bit-aloo commented Jul 8, 2025

Copy link
Copy Markdown
Owner

No description provided.

@bit-aloo
bit-aloo force-pushed the 2025-07-08-handlers-refactor branch from b3dabe2 to d8fa5c5 Compare July 9, 2025 07:56
@bit-aloo
bit-aloo force-pushed the 2025-06-08-noise-stream branch from e990a5c to de5dc34 Compare July 9, 2025 07:58
@bit-aloo
bit-aloo force-pushed the 2025-07-08-handlers-refactor branch 2 times, most recently from 934c357 to 0a711f8 Compare July 9, 2025 15:33
@bit-aloo
bit-aloo force-pushed the 2025-06-08-noise-stream branch 3 times, most recently from b76df93 to e51a33b Compare July 15, 2025 08:19
@bit-aloo
bit-aloo force-pushed the 2025-07-08-handlers-refactor branch 9 times, most recently from 933eb97 to 00c4019 Compare July 20, 2025 13:02
@bit-aloo
bit-aloo force-pushed the 2025-07-08-handlers-refactor branch from 18202de to df689e2 Compare July 29, 2025 15:03
@bit-aloo
bit-aloo force-pushed the 2025-06-08-noise-stream branch from 0cb48fb to 0ac90d5 Compare July 29, 2025 15:06
GitGab19 and others added 7 commits July 30, 2025 12:22
Previously, `HandlerError` only supported a fixed set of internal error types.
This made it difficult for downstream users implementing library traits to
propagate their own custom error types.

This commit introduces a new `External(Box<dyn Error + Send + Sync>)` variant
to allow wrapping arbitrary error values. This enables implementors to return
non-library errors without requiring changes to the core `HandlerError` enum.
…de how to go about authorization of a connection
@bit-aloo
bit-aloo force-pushed the 2025-07-08-handlers-refactor branch from b709c97 to 52626ca Compare July 30, 2025 13:35
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