-
Notifications
You must be signed in to change notification settings - Fork 1.2k
http2: don't leak streams on broken body #18
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
Conversation
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed (or fixed any issues), please reply here (e.g. What to do if you already signed the CLAIndividual signers
Corporate signers
|
Ping cla bot |
CLAs look good, thanks! |
Message from Gerrit User 5976: Patch Set 1: Congratulations on opening your first change. Thank you for your contribution! Next steps: Most changes in the Go project go through a few rounds of revision. This can be During May-July and Nov-Jan the Go project is in a code freeze, during which Please don’t reply on this GitHub thread. Visit golang.org/cl/132715. |
Message from Gerrit User 5065: Patch Set 1: FYI: I'm just starting a few week vacation so I likely won't find time for this until I'm back. Please don’t reply on this GitHub thread. Visit golang.org/cl/132715. |
Message from Gerrit User 5065: Patch Set 1: Run-TryBot+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/132715. |
Message from Gerrit User 5976: Patch Set 1: TryBots beginning. Status page: https://farmer.golang.org/try?commit=6ff1ed4b Please don’t reply on this GitHub thread. Visit golang.org/cl/132715. |
Message from Gerrit User 5976: Patch Set 1: TryBot-Result+1 TryBots are happy. Please don’t reply on this GitHub thread. Visit golang.org/cl/132715. |
Message from Gerrit User 28859: Patch Set 1:
I hope you've enjoyed your vacation, so friendly ping :) Please don’t reply on this GitHub thread. Visit golang.org/cl/132715. |
Message from Gerrit User 5065: Patch Set 1: Code-Review+2 Please don’t reply on this GitHub thread. Visit golang.org/cl/132715. |
Updates golang/go#27208 Change-Id: I5d9a643f33d27d33b24f670c98f5a51aa6000967 GitHub-Last-Rev: 3ac4a57 GitHub-Pull-Request: #18 Reviewed-on: https://go-review.googlesource.com/c/132715 Run-TryBot: Brad Fitzpatrick <[email protected]> TryBot-Result: Gobot Gobot <[email protected]> Reviewed-by: Brad Fitzpatrick <[email protected]>
This PR is being closed because golang.org/cl/132715 has been merged. |
Updates golang/go#28673 Change-Id: I5d9a643f33d27d33b24f670c98f5a51aa6000967 GitHub-Last-Rev: 3ac4a57 GitHub-Pull-Request: #18 Reviewed-on: https://go-review.googlesource.com/c/132715 Run-TryBot: Brad Fitzpatrick <[email protected]> TryBot-Result: Gobot Gobot <[email protected]> Reviewed-by: Brad Fitzpatrick <[email protected]> (cherry picked from commit 1c5f79c) Reviewed-on: https://go-review.googlesource.com/c/154237
Updates golang/go#27208 Change-Id: I5d9a643f33d27d33b24f670c98f5a51aa6000967 GitHub-Last-Rev: 3ac4a573b62846ef4944599085218e119819383c GitHub-Pull-Request: golang/net#18 Reviewed-on: https://go-review.googlesource.com/c/132715 Run-TryBot: Brad Fitzpatrick <[email protected]> TryBot-Result: Gobot Gobot <[email protected]> Reviewed-by: Brad Fitzpatrick <[email protected]>
Updates golang/go#27208