New issue
Advanced search Search tips
Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Dec 17
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug



Sign in to add a comment
link

Issue 514468: QUIC - enable ProofTest.Verify

Reported by rtenneti@chromium.org, Jul 27 2015 Project Member

Issue description

Fix  ProofTest.Verify and enable it.

[ RUN      ] ProofTest.Verify
** Message: Skipping credential gecc because its status is NEED_KEY
../../net/quic/crypto/proof_test.cc:126: Failure
Value of: source->GetProof(server_ip, hostname, server_config, false , &first_certs, &first_signature)
  Actual: false
Expected: true
[  FAILED  ] ProofTest.Verify (1274 ms)
 

Comment 1 by rtenneti@chromium.org, Apr 24 2017

Owner: ----
Status: Available (was: Assigned)
Would appreciate if a new owner could be assigned to these bugs. Moved to a different team. Some of these bugs could be invalid now. Thanks.

Comment 2 by sheriffbot@chromium.org, Apr 25 2018

Project Member
Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 3 by bugdroid1@chromium.org, Dec 15

Project Member
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b28c476f724774d786a63a9e9e37e2ed925240c1

commit b28c476f724774d786a63a9e9e37e2ed925240c1
Author: David Benjamin <davidben@chromium.org>
Date: Sat Dec 15 04:47:15 2018

Enable ProofTest.Verify.

Inspired by pkasting's email, I noticed this...

It seems to work these days. Maybe the BoringSSL switch? The bug has
some error coming out of NSS.

Bug:  514468 
Change-Id: Ic0ba0af9b19019fea230e9ca171e2daf09fccbb2
Reviewed-on: https://chromium-review.googlesource.com/c/1378817
Reviewed-by: Ryan Hamilton <rch@chromium.org>
Commit-Queue: Ryan Hamilton <rch@chromium.org>
Cr-Commit-Position: refs/heads/master@{#616955}
[modify] https://crrev.com/b28c476f724774d786a63a9e9e37e2ed925240c1/net/quic/crypto/proof_test_chromium.cc

Comment 4 by davidben@chromium.org, Dec 17

Owner: davidben@chromium.org
Status: Fixed (was: Untriaged)

Sign in to add a comment