cloudflare / cloudflare/boringtun

Error doesn't implement StdError or Display

Open
#339 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Rust
Stars
7.2k
Forks
532
PR merge metrics
No merged PRs in 30d

Description

boringtun's device::Error type isn't actually a standard Rust error, it doesn't implement StdError or Display. We use thiserror, and if device::Error was implemented as expected we'd use:

#[error("boringtun error {0} while {1}")]
Boringtun(#[source] boringtun::device::Error, String),

instead we have to use

#[error("boringtun error {0:?} while {1}")]
Boringtun(boringtun::device::Error, String),

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.