Replace server.GroupedError with some standard multierr package
Open
- Dominant language
- C
- Stars
- 309
- Forks
- 118
- Avg merge
- 1d 16h
- Merged PRs (30d)
- 33
Description
Potential packages to use:
- https://pkg.go.dev/go.uber.org/multierr
- https://pkg.go.dev/github.com/hashicorp/go-multierror
This would make it easier for both us and our users to deal with functions that may return multiple errors.
Examples include:
- `client.Attest`
- `server.VerifyAttestation`
Contributor guide
Research direction
Review the existing server.GroupedError implementation and trace the client.Attest and server.VerifyAttestation entry points. Compare the two proposed multierr packages and determine the compatibility impact of replacing the current error type; done means the affected APIs use an agreed standard multi-error package.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- go
- Domain
- api
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 38/100