close
The Wayback Machine - https://web.archive.org/web/20201114121517/https://github.com/letsencrypt/boulder/pull/4227
Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement suberrors & subproblems #4227

Merged
merged 4 commits into from May 24, 2019
Merged

Conversation

@cpu
Copy link
Contributor

@cpu cpu commented May 23, 2019

Updates #4193

Updating relevant Boulder locations to use WithSubErrors and WithSubProblems will be done in a separate follow-up PR.

Daniel added 3 commits May 23, 2019
Daniel
Daniel
Daniel
…oblems
@cpu cpu requested a review from letsencrypt/boulder-developers as a code owner May 23, 2019
@cpu cpu self-assigned this May 23, 2019
Copy link
Contributor

@jsha jsha left a comment

Looks great, one nit!

probs/probs.go Outdated Show resolved Hide resolved
Daniel
@jsha
jsha approved these changes May 24, 2019
@rolandshoemaker rolandshoemaker merged commit ecd1ea6 into master May 24, 2019
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@cpu cpu deleted the cpu-my-problems-have-problems branch May 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.