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

Issue 1684051: code review 1684051: crypto/tls, http: Make HTTPS servers easier. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
13 years, 9 months ago by agl1
Modified:
13 years, 9 months ago
Reviewers:
r, adg, rsc1
CC:
adg, rsc, golang-dev
Visibility:
Public.

Description

crypto/tls, http: Make HTTPS servers easier.

Patch Set 1 #

Patch Set 2 : code review 1684051: crypto/tls, http/https: Make HTTPS servers easier. #

Patch Set 3 : code review 1684051: crypto/tls, http/https: Make HTTPS servers easier. #

Total comments: 4

Patch Set 4 : code review 1684051: crypto/tls, http/https: Make HTTPS servers easier. #

Patch Set 5 : code review 1684051: crypto/tls, http/https: Make HTTPS servers easier. #

Patch Set 6 : code review 1684051: crypto/tls, http: Make HTTPS servers easier. #

Patch Set 7 : code review 1684051: crypto/tls, http: Make HTTPS servers easier. #

Total comments: 11

Patch Set 8 : code review 1684051: crypto/tls, http: Make HTTPS servers easier. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+185 lines, -1 line) Patch
A src/pkg/crypto/tls/generate_cert.go View 6 7 1 chunk +79 lines, -0 lines 0 comments Download
M src/pkg/crypto/tls/tls.go View 1 2 3 4 5 2 chunks +54 lines, -1 line 0 comments Download
M src/pkg/http/server.go View 6 7 3 chunks +52 lines, -0 lines 0 comments Download

Messages

Total messages: 19
agl1
13 years, 9 months ago (2010-07-01 21:57:04 UTC) #1
adg
LGTM A couple of nits below. I'd like to write more of these examples/ directories ...
13 years, 9 months ago (2010-07-01 22:08:23 UTC) #2
agl1
Happy to make the directory 'example' if people like that. http://codereview.appspot.com/1684051/diff/5001/6001 File src/pkg/crypto/tls/examples/generate_cert.go (right): http://codereview.appspot.com/1684051/diff/5001/6001#newcode1 ...
13 years, 9 months ago (2010-07-01 22:11:42 UTC) #3
rsc1
I'm inclined to drop the examples directory and just leave the generate_cert program in the ...
13 years, 9 months ago (2010-07-01 23:24:07 UTC) #4
rsc1
13 years, 9 months ago (2010-07-01 23:24:15 UTC) #5
r2
I think 'examples' is an ok name but singular is fine too. What's important is ...
13 years, 9 months ago (2010-07-01 23:25:04 UTC) #6
r2
On Jul 1, 2010, at 4:24 PM, rsc@google.com wrote: > I'm inclined to drop the ...
13 years, 9 months ago (2010-07-01 23:26:08 UTC) #7
rsc1
I'd like to discuss the example thing separately. Can we leave the example in tls ...
13 years, 9 months ago (2010-07-01 23:29:21 UTC) #8
r2
On Jul 1, 2010, at 4:29 PM, Russ Cox wrote: > I'd like to discuss ...
13 years, 9 months ago (2010-07-01 23:30:16 UTC) #9
rsc1
okay, so i think: move ListenAndServeTLS into package http, dropping http/https move generate_certs.go up one ...
13 years, 9 months ago (2010-07-01 23:38:31 UTC) #10
agl1
On Thu, Jul 1, 2010 at 7:38 PM, <rsc@google.com> wrote: > move generate_certs.go up one ...
13 years, 9 months ago (2010-07-02 15:17:58 UTC) #11
r2
On Jul 2, 2010, at 8:17 AM, Adam Langley wrote: > On Thu, Jul 1, ...
13 years, 9 months ago (2010-07-02 15:27:16 UTC) #12
agl1
On Fri, Jul 2, 2010 at 11:26 AM, Rob 'Commander' Pike <r@google.com> wrote: > Yes ...
13 years, 9 months ago (2010-07-02 15:45:26 UTC) #13
rsc1
LGTM thanks for adding this http://codereview.appspot.com/1684051/diff/23001/24001 File src/pkg/crypto/tls/generate_cert.go (right): http://codereview.appspot.com/1684051/diff/23001/24001#newcode1 src/pkg/crypto/tls/generate_cert.go:1: // Generate a self-signed ...
13 years, 9 months ago (2010-07-02 16:43:23 UTC) #14
agl1
On Fri, Jul 2, 2010 at 12:43 PM, <rsc@google.com> wrote: > LGTM Thanks. Have fixed ...
13 years, 9 months ago (2010-07-02 17:00:02 UTC) #15
agl1
Hello r, adg, rsc (cc: golang-dev@googlegroups.com), I'd like you to review this change.
13 years, 9 months ago (2010-07-02 17:00:17 UTC) #16
agl1
*** Submitted as http://code.google.com/p/go/source/detail?r=cf7566bf82b6 *** crypto/tls, http: Make HTTPS servers easier. R=r, adg, rsc CC=golang-dev ...
13 years, 9 months ago (2010-07-02 17:00:24 UTC) #17
r
http://codereview.appspot.com/1684051/diff/23001/24001 File src/pkg/crypto/tls/generate_cert.go (right): http://codereview.appspot.com/1684051/diff/23001/24001#newcode20 src/pkg/crypto/tls/generate_cert.go:20: } you might want to use the flags package ...
13 years, 9 months ago (2010-07-02 17:02:11 UTC) #18
agl1
13 years, 9 months ago (2010-07-02 17:04:20 UTC) #19
On Fri, Jul 2, 2010 at 1:02 PM,  <r@golang.org> wrote:
> http://codereview.appspot.com/1684051/diff/23001/24001
> File src/pkg/crypto/tls/generate_cert.go (right):

Even if I wait a while you still manage to comment two minutes after I submit :)

Will fix.


AGL
Sign in to reply to this message.

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