Better error handling #10

New Issue

2026-06-26T15:59:26Z

decentral1se commented

2026-06-26 15:59:26 +00:00

It would be nice if we could avoid the thing we're doing as it's a bit clunky:

if err != nil {
  irohErr := err.(*iroh.IrohError)
  panic(irohErr.Message())
}

And instead just expose the error types directly:

https://github.com/n0-computer/iroh-ffi/blob/87509053e1dbe1e561c4abb0cd08481441c0edcb/src/error.rs#L39-L57

Then people can errors.Is(...) (or whatever) directly to check without the casting. I don't actually see this exposed in the API via the docs: https://godocs.io/git.coopcloud.tech/decentral1se/iroh-go so there might need to be some automagic conversion done 🤔

It would be nice if we could avoid the thing we're doing as it's a bit clunky: ```go if err != nil { irohErr := err.(*iroh.IrohError) panic(irohErr.Message()) } ``` And instead just expose the error types directly: > https://github.com/n0-computer/iroh-ffi/blob/87509053e1dbe1e561c4abb0cd08481441c0edcb/src/error.rs#L39-L57 Then people can `errors.Is(...)` (or whatever) directly to check without the casting. I don't actually see this exposed in the API via the docs: https://godocs.io/git.coopcloud.tech/decentral1se/iroh-go so there might need to be some automagic conversion done 🤔

cblgh commented

2026-06-26 20:07:44 +00:00

yeah i was thinking of this too! big fan of being able to use errors.Is if possible

🎉 1

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: decentral1se/iroh-go#10