Skip to content

/ go

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

Jump to bottom

x/net/http2: log.Printf is being used instead of sc.logf #12742

Closed
itsmontoya opened this issue Sep 24, 2015 · 1 comment
Closed

x/net/http2: log.Printf is being used instead of sc.logf #12742

itsmontoya opened this issue Sep 24, 2015 · 1 comment
Labels
FrozenDueToAge

Comments

@itsmontoya
Copy link

@itsmontoya itsmontoya commented Sep 24, 2015

I have a pull from the original repo with the fix here.

bradfitz/http2#72
@gopherbot

This comment has been minimized.

Sign in to view
Copy link

@gopherbot gopherbot commented Oct 13, 2015

CL https://golang.org/cl/15736 mentions this issue.
@golang golang locked and limited conversation to collaborators Oct 12, 2016
c3mb0 pushed a commit to c3mb0/net that referenced this issue Apr 2, 2018
@bradfitz
http2: change two debug log.Printf calls to vlogf
422e0a3 
Fixes golang/go#12742

Change-Id: I04d65b0c68ccb5da5ee90650c7b0593fbdb1d470
Reviewed-on: https://go-review.googlesource.com/15736
Reviewed-by: Andrew Gerrand <adg@golang.org>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
FrozenDueToAge
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
@itsmontoya @gopherbot
You can’t perform that action at this time.