-
Notifications
You must be signed in to change notification settings - Fork 2k
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
crypto/ocsp: Add Raw field to ocsp.Response #115
base: master
Are you sure you want to change the base?
crypto/ocsp: Add Raw field to ocsp.Response #115
Conversation
This PR (HEAD: bdf47ae) has been imported to Gerrit for code review. Please visit https://go-review.googlesource.com/c/crypto/+/216423 to see it. Tip: You can toggle comments from me using the |
Message from Gobot Gobot: 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/216423. |
bdf47ae
to
7b4e24e
Compare
This PR (HEAD: 7b4e24e) has been imported to Gerrit for code review. Please visit https://go-review.googlesource.com/c/crypto/+/216423 to see it. Tip: You can toggle comments from me using the |
7b4e24e
to
9ac6681
Compare
This PR (HEAD: 9ac6681) has been imported to Gerrit for code review. Please visit https://go-review.googlesource.com/c/crypto/+/216423 to see it. Tip: You can toggle comments from me using the |
9ac6681
to
f888226
Compare
This PR (HEAD: f888226) has been imported to Gerrit for code review. Please visit https://go-review.googlesource.com/c/crypto/+/216423 to see it. Tip: You can toggle comments from me using the |
This PR (HEAD: 18876f6) has been imported to Gerrit for code review. Please visit https://go-review.googlesource.com/c/crypto/+/216423 to see it. Tip: You can toggle comments from me using the |
18876f6
to
b9db5ad
Compare
This PR (HEAD: b9db5ad) has been imported to Gerrit for code review. Please visit https://go-review.googlesource.com/c/crypto/+/216423 to see it. Tip: You can toggle comments from me using the |
b9db5ad
to
fffe4d8
Compare
This PR (HEAD: fffe4d8) has been imported to Gerrit for code review. Please visit https://go-review.googlesource.com/c/crypto/+/216423 to see it. Tip: You can toggle comments from me using the |
This PR (HEAD: 87ee9ea) has been imported to Gerrit for code review. Please visit https://go-review.googlesource.com/c/crypto/+/216423 to see it. Tip: You can toggle comments from me using the |
@FiloSottile I updated the PR, which de-scoped it quite a bit. Let me know if this is closer to what you had in mind, and if so I can squash the commits. |
This PR (HEAD: 56c74ec) has been imported to Gerrit for code review. Please visit https://go-review.googlesource.com/c/crypto/+/216423 to see it. Tip: You can toggle comments from me using the |
Message from Filippo Valsorda: Patch Set 9: Run-TryBot+1 Code-Review+2 (1 comment) Thanks! To update the commit message you need to update the title and body of the PR. See https://golang.org/wiki/CommitMessage Please don’t reply on this GitHub thread. Visit golang.org/cl/216423. |
Message from Gobot Gobot: Patch Set 9: TryBots beginning. Status page: https://farmer.golang.org/try?commit=a216d472 Please don’t reply on this GitHub thread. Visit golang.org/cl/216423. |
Message from Gobot Gobot: Patch Set 9: TryBot-Result+1 TryBots are happy. Please don’t reply on this GitHub thread. Visit golang.org/cl/216423. |
This PR (HEAD: da25f67) has been imported to Gerrit for code review. Please visit https://go-review.googlesource.com/c/crypto/+/216423 to see it. Tip: You can toggle comments from me using the |
The Raw field contains the DER encoded repreentation of the OCSP response
da25f67
to
bba6e2e
Compare
This PR (HEAD: bba6e2e) has been imported to Gerrit for code review. Please visit https://go-review.googlesource.com/c/crypto/+/216423 to see it. Tip: You can toggle comments from me using the |
@FiloSottile sorry this slipped through the cracks, hopefully I have addressed your concerns. |
@FiloSottile bump |
Message from Go Bot: 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/216423. |
Message from Go Bot: Patch Set 9: TryBots beginning. Status page: https://farmer.golang.org/try?commit=a216d472 Please don’t reply on this GitHub thread. Visit golang.org/cl/216423. |
Message from Go Bot: Patch Set 9: TryBot-Result+1 TryBots are happy. Please don’t reply on this GitHub thread. Visit golang.org/cl/216423. |
Message from Emmanuel Odeke: Patch Set 12: Trust+1 (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/216423. |
Message from Emmanuel Odeke: Patch Set 12: Run-TryBot+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/216423. |
Message from Go Bot: Patch Set 12: TryBots beginning. Status page: https://farmer.golang.org/try?commit=5f741069 Please don’t reply on this GitHub thread. Visit golang.org/cl/216423. |
Message from Go Bot: Patch Set 12: TryBot-Result+1 TryBots are happy. Please don’t reply on this GitHub thread. Visit golang.org/cl/216423. |
Would be nice to have this. It would clean up my code a little. Also, kinda expected this already because |
The Raw field contains the DER encoded repreentation of the OCSP response