-
Notifications
You must be signed in to change notification settings - Fork 22
Conversation
src/errors.rs
Outdated
pub enum Error { | ||
/// Access is denied for a given requester | ||
/// Access denied for supplied PublicKey | ||
#[error("Access Denied")] | ||
AccessDenied, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't we want to provide the pk here so the consumer knows which pk was being used when the error was thrown? I know this is probably out of scope of this PR, but just thinking the direction we should take these errors to and start working on some consistency.
src/messaging/mod.rs
Outdated
@@ -640,7 +640,7 @@ mod tests { | |||
} | |||
|
|||
#[test] | |||
fn try_from() -> Result<()> { | |||
fn try_from() -> anyhow::Result<()> { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perhaps simply use anyhow::{Result, anyhow}
for the whole test
module?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I did try that and things got wiiild w/ compiler errors. I'll double check that's not sorted now though
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
that wildness is only in the seq file. the rest ill update
No description provided.