close
The Wayback Machine - https://web.archive.org/web/20201114121513/https://github.com/letsencrypt/boulder/pull/4229
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

Consistently used stdlib context package #4229

Merged
merged 4 commits into from May 28, 2019
Merged

Consistently used stdlib context package #4229

merged 4 commits into from May 28, 2019

Conversation

@rolandshoemaker
Copy link
Contributor

@rolandshoemaker rolandshoemaker commented May 24, 2019

No idea why this works now when @cpu's original attempt didn't, ¯\_(ツ)_/¯.

Fixes #2335.

@rolandshoemaker rolandshoemaker requested a review from letsencrypt/boulder-developers as a code owner May 24, 2019
@cpu
Copy link
Contributor

@cpu cpu commented May 24, 2019

No idea why this works now when @cpu's original attempt didn't, ¯\_(ツ)_/¯.

😆 I don't even remember trying this...

@@ -63,8 +63,12 @@ func dnsi(hostname string) identifier.ACMEIdentifier {

var ctx context.Context

func init() {
ctx, _ = context.WithTimeout(context.Background(), 10*time.Minute)
func TestMain(m *testing.M) {

This comment has been minimized.

@cpu

cpu May 24, 2019
Contributor

Using TestMain for this was a good idea 👍

@cpu
Copy link
Contributor

@cpu cpu commented May 28, 2019

Merging with one 🔍 based on low risk (switch to stdlib context, should be functionally equiv and breakage will be easy to spot) and available dev resources.

@cpu
cpu approved these changes May 28, 2019
@cpu cpu merged commit 6f93942 into master May 28, 2019
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@cpu cpu deleted the stdlib-context branch May 28, 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.

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