search

campbellC / third-wheel

A rust implementation of a man-in-the-middle proxy
MIT License
71 stars 19 forks source link

Sort out the error handling #17

Closed campbellC closed 4 years ago

campbellC commented 4 years ago

The error handling is currently pretty awful - we've got this SafeResult type which litters the code base but doesn't fit with more idiomatic error handling like in anyhow. This should be replace with proper error types and the errors should be handled or propagated properly in a consistent and predictable way for client code. Probably this should be done before removing all the unwraps.

campbellC commented 4 years ago

This is being done as part of https://github.com/campbellC/third-wheel/issues/12