Looks good to me. --Sean
On 06/07/2012 07:37 AM, Weijun Wang wrote:
Hi All Please take a review at this http://cr.openjdk.java.net/~weijun/7175041/webrev.00/ The original check is added [1] in JDK 8 so there is no need for a backport. I also do not intend to introduce the updated check to JDK 7 or earlier. Noreg-trivial. Thanks Max [1] http://hg.openjdk.java.net/jdk8/tl/jdk/diff/57eb9136b73b/src/share/classes/sun/security/timestamp/HttpTimestamper.java -------- Original Message -------- 7175041: HttpTimestamper should accept https URI Product: java Category: java Subcategory: classes_security Type: Defect === *Description* ============================================================ sun.security.timestamp.HttpTimestamper has: public HttpTimestamper(URI tsaURI) { if (!tsaURI.getScheme().equalsIgnoreCase("http")) throw new IllegalArgumentException("TSA must be an HTTP URI"); this.tsaURI = tsaURI; } It should also support TSA with a https URI.
