Better LP/Broker API RPC Errors

[dandanlen]

Pull Request

Closes: PRO-808

Checklist

Please conduct a thorough self-review before opening the PR.

• I am confident that the code works.
• I have updated documentation where appropriate.

Summary

• Updated jsonrpsee because this PR changes the way errors are used.
• Added a wrapper around anyhow::Error to allow conversion to jsonrpsee::types::ErrorObjectOwned including context and source.

[linear]

PRO-808 Better API errors.

We currently use the default error conversion from anyhow::Error.

By default, this uses the "{}" formatter, which only uses the outermost error or context. This means that we often drop useful error messages and only display the context.

For example, lp_registerAccount just shows Could not register account role for account and drops the inner runtime error.

We should include the runtime error message.

[codecov]

Codecov Report

Merging #3931 (a997a57) into main (0ff654f) will decrease coverage by 0%.
The diff coverage is 0%.

@@          Coverage Diff          @@
##            main   #3931   +/-   ##
=====================================
- Coverage     72%     72%   -0%     
=====================================
  Files        366     366           
  Lines      56783   56789    +6     
  Branches   56783   56789    +6     
=====================================
- Hits       40949   40945    -4     
- Misses     13747   13756    +9     
- Partials    2087    2088    +1     

Files Changed	Coverage Δ
api/lib/src/lp.rs	0% <0%> (ø)
engine/src/dot/http_rpc.rs	0% <0%> (ø)
engine/src/eth/rpc.rs	0% <0%> (ø)
utilities/src/with_std.rs	50% <0%> (-1%)	⬇️

... and 3 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[dandanlen]

Thanks @j4m1ef0rd , LGTM.