google / google/go-tpm-tools

Replace server.GroupedError with some standard multierr package

Open
#169 0 comments 0 reactions 0 assignees View on GitHub
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.