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

Issue 68240043: Adding CtExtensions to STH (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
8 years, 4 months ago by Eran
Modified:
8 years, 2 months ago
CC:
ctlog-opensource-review_google.com
Visibility:
Public.

Description

Adding CtExtensions to STH. As discussed offline, CtExtensions in the Signed Tree Head will allow the log to include additional data in the future. This is https://code.google.com/p/certificate-transparency/issues/detail?id=28

Patch Set 1 #

Total comments: 2

Patch Set 2 : Adding CtSthExtensions #

Patch Set 3 : Documenting extensions. #

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

Messages

Total messages: 5
Eran
8 years, 4 months ago (2014-02-24 17:37:52 UTC) #1
Ben Laurie (Google)
https://codereview.appspot.com/68240043/diff/1/doc/rfc6962-bis.xml File doc/rfc6962-bis.xml (right): https://codereview.appspot.com/68240043/diff/1/doc/rfc6962-bis.xml#newcode629 doc/rfc6962-bis.xml:629: CtExtensions extensions; I suspect this should have a different ...
8 years, 4 months ago (2014-02-26 15:49:28 UTC) #2
Eran
Addressed comment, PTAL. https://codereview.appspot.com/68240043/diff/1/doc/rfc6962-bis.xml File doc/rfc6962-bis.xml (right): https://codereview.appspot.com/68240043/diff/1/doc/rfc6962-bis.xml#newcode629 doc/rfc6962-bis.xml:629: CtExtensions extensions; On 2014/02/26 15:49:29, Ben ...
8 years, 4 months ago (2014-03-04 14:33:57 UTC) #3
Ben Laurie (Google)
LGTM
8 years, 2 months ago (2014-05-02 11:24:51 UTC) #4
Eran
8 years, 2 months ago (2014-05-02 12:27:44 UTC) #5

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