Opened 6 years ago

Closed 5 years ago

#181 closed defect (fixed)

Consolidate "Add Chain" and "Add PreCertChain" endpoints

Reported by: rlb@… Owned by: eranm@…
Priority: major Milestone: review
Component: rfc6962-bis Version:
Severity: - Keywords:
Cc:

Description

There is no need for separate endpoints here. The server can tell which is
which based on the presence of the "precertificate" parameter.

Change History (10)

comment:1 Changed 6 years ago by rob.stradling@…

  • Component changed from client-behavior to to-be-decided

comment:2 Changed 6 years ago by rob.stradling@…

Let's call the consolidated endpoint: /ct/v2/add-entry

comment:3 Changed 5 years ago by eranm@…

  • Component changed from to-be-decided to rfc6962-bis

Per in-person discussion and trans wg meeting, seems like a good idea (I didn't hear anyone objecting) since the precertificate parameter pretty clearly indicates that.

Change out for review in https://github.com/google/certificate-transparency-rfcs/pull/238

comment:4 Changed 5 years ago by eranm@…

comment:5 Changed 5 years ago by eranm@…

  • Milestone set to review

Change merged - chairs, please review.

comment:6 Changed 5 years ago by eranm@…

  • Owner changed from draft-ietf-trans-rfc6962-bis@… to eranm@…
  • Status changed from new to assigned

comment:7 Changed 5 years ago by melinda.shore@…

  • Resolution set to fixed
  • Status changed from assigned to closed

comment:8 Changed 5 years ago by eranm@…

  • Milestone review deleted
  • Resolution fixed deleted
  • Status changed from closed to reopened

My bad - that has not been resolved yet and the PR not actually merged. Reverting resolution.

Andrew Ayer objected to that, in https://www.ietf.org/mail-archive/web/trans/current/msg02874.html, I'll follow up with him to better understand the objection and we'll bring it back to the list once we have a clear suggestion.

comment:9 Changed 5 years ago by eranm@…

  • Milestone set to review

Please review the solution Rob has implemented in
https://github.com/google/certificate-transparency-rfcs/pull/262.

This was supported on the list by:

(and I support it too, obviously)

comment:10 Changed 5 years ago by melinda.shore@…

  • Resolution set to fixed
  • Status changed from reopened to closed
Note: See TracTickets for help on using tickets.