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

Issue 83200047: Draft text for http://trac.tools.ietf.org/wg/trans/trac/ticket/1 (Closed)

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

Description

Draft text for http://trac.tools.ietf.org/wg/trans/trac/ticket/1

Patch Set 1 #

Total comments: 14

Patch Set 2 : Address comments from Ben #

Total comments: 7

Patch Set 3 : Address more comments from Ben #

Patch Set 4 : CN-IDs MUST remain unredacted #

Patch Set 5 : Include all edits #

Unified diffs Side-by-side diffs Delta from patch set Stats (+145 lines, -3 lines) Patch
M doc/rfc6962-bis.xml View 1 2 3 4 6 chunks +145 lines, -3 lines 0 comments Download

Messages

Total messages: 9
Rob Stradling
9 years, 12 months ago (2014-04-02 12:13:14 UTC) #1
Ben Laurie (Google)
https://codereview.appspot.com/83200047/diff/1/doc/rfc6962-bis.xml File doc/rfc6962-bis.xml (right): https://codereview.appspot.com/83200047/diff/1/doc/rfc6962-bis.xml#newcode419 doc/rfc6962-bis.xml:419: them to be secured using publicly-trusted TLS server certificates. ...
9 years, 12 months ago (2014-04-02 12:52:00 UTC) #2
Rob Stradling
https://codereview.appspot.com/83200047/diff/1/doc/rfc6962-bis.xml File doc/rfc6962-bis.xml (right): https://codereview.appspot.com/83200047/diff/1/doc/rfc6962-bis.xml#newcode419 doc/rfc6962-bis.xml:419: them to be secured using publicly-trusted TLS server certificates. ...
9 years, 12 months ago (2014-04-04 14:41:05 UTC) #3
Ben Laurie (Google)
https://codereview.appspot.com/83200047/diff/20001/doc/rfc6962-bis.xml File doc/rfc6962-bis.xml (right): https://codereview.appspot.com/83200047/diff/20001/doc/rfc6962-bis.xml#newcode450 doc/rfc6962-bis.xml:450: Precertificate. Do we say anything about the CN-ID in ...
9 years, 11 months ago (2014-04-07 10:22:51 UTC) #4
Rob Stradling
https://codereview.appspot.com/83200047/diff/20001/doc/rfc6962-bis.xml File doc/rfc6962-bis.xml (right): https://codereview.appspot.com/83200047/diff/20001/doc/rfc6962-bis.xml#newcode450 doc/rfc6962-bis.xml:450: Precertificate. On 2014/04/07 10:22:51, Ben Laurie (Google) wrote: > ...
9 years, 11 months ago (2014-04-07 13:50:25 UTC) #5
Ben Laurie (Google)
LGTM, but if you want to do the CN-ID thing, happy to review again. https://codereview.appspot.com/83200047/diff/20001/doc/rfc6962-bis.xml ...
9 years, 11 months ago (2014-04-07 14:43:11 UTC) #6
Rob Stradling
On 2014/04/07 14:43:11, Ben Laurie (Google) wrote: > LGTM, but if you want to do ...
9 years, 11 months ago (2014-04-10 13:12:47 UTC) #7
Ben Laurie (Google)
LGTM
9 years, 11 months ago (2014-04-11 14:57:00 UTC) #8
Rob Stradling
9 years, 10 months ago (2014-05-23 12:06:34 UTC) #9
Pushed as b33d6561ab32cd8e50e7c2c6560fa8acc5899a1f
Sign in to reply to this message.

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