Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(514)

Issue 81260045: code review 81260045: crypto/tls: Added dynamic alternative to NameToCertific...

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years ago by ox
Modified:
9 years, 9 months ago
Visibility:
Public.

Description

crypto/tls: Added dynamic alternative to NameToCertificate map for SNI Superceded by https://codereview.appspot.com/107400043

Patch Set 1 #

Patch Set 2 : diff -r 5f71b23a0a4b https://code.google.com/p/go #

Patch Set 3 : diff -r ed3e89f781c1 https://code.google.com/p/go #

Unified diffs Side-by-side diffs Delta from patch set Stats (+110 lines, -17 lines) Patch
M src/pkg/crypto/tls/common.go View 1 4 chunks +18 lines, -5 lines 0 comments Download
M src/pkg/crypto/tls/conn_test.go View 1 1 chunk +14 lines, -5 lines 0 comments Download
M src/pkg/crypto/tls/handshake_server.go View 1 1 chunk +4 lines, -1 line 0 comments Download
M src/pkg/crypto/tls/handshake_server_test.go View 1 4 chunks +74 lines, -6 lines 0 comments Download

Messages

Total messages: 6
ox
Hello golang-codereviews@googlegroups.com, I'd like you to review this change to https://code.google.com/p/go
10 years ago (2014-03-27 11:44:29 UTC) #1
minux1
Sorry, but currently the tree is closed for new features not marked Go1.3, please ping ...
10 years ago (2014-03-27 18:44:28 UTC) #2
ox
On 2014/03/27 18:44:28, minux wrote: > Sorry, but currently the tree is closed for new ...
10 years ago (2014-03-27 21:03:47 UTC) #3
bradfitz
You can keep it open and just email a ping after go 1.3. On Mar ...
10 years ago (2014-03-27 21:11:28 UTC) #4
gobot
R=agl@golang.org (assigned by minux.ma@gmail.com)
10 years ago (2014-03-28 04:30:58 UTC) #5
rsc
9 years, 11 months ago (2014-04-17 02:36:23 UTC) #6
R=close

Please run 'hg mail' after Go 1.3 is out and it will reopen this CL for us.
Thanks.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b