Skip to content

Comments

Add in test vectors#77

Open
wbl wants to merge 3 commits intotlswg:masterfrom
wbl:watson/test-vecs
Open

Add in test vectors#77
wbl wants to merge 3 commits intotlswg:masterfrom
wbl:watson/test-vecs

Conversation

@wbl
Copy link

@wbl wbl commented Jul 8, 2020

No description provided.

@claucece claucece mentioned this pull request Nov 4, 2020
4 tasks
Copy link
Contributor

@chris-wood chris-wood left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@wbl, can we get others to verify these vectors?

cc @siyengar, @claucece, @xvzcf

66vHmWi7L52EDCUGYwIgawvUo3EplSqfdTrYM1YbjuN7aiyqeljkE5G0hihUh0w=
-----END CERTIFICATE-----
~~~~~~~
and its private key
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
and its private key
The corresponding private key is as follows.

@wbl
Copy link
Author

wbl commented Dec 3, 2020

I think these vectors actually have a misencoded extension. There is a PR internally to fix this that's been open for a while.

@claucece
Copy link
Contributor

claucece commented Dec 3, 2020

Yes @chris-wood , I saw them and tried to play with them; but it wasn't working...

@chris-wood
Copy link
Contributor

Okay, thanks! @wbl, will you have cycles to update these?

@wbl
Copy link
Author

wbl commented Dec 8, 2020

Unlikely at this point... too many things flying around.

@claucece
Copy link
Contributor

claucece commented Dec 9, 2020

Maybe we could use what the dc are now generating in the golang or boring-ssl experiment..

@chris-wood
Copy link
Contributor

Yep, that's a fine idea. Let's discuss offline.

@seanturner
Copy link
Contributor

@chris-wood any chance you can work some magic to get these updated?

grittygrease and others added 2 commits March 7, 2022 07:37
Co-authored-by: Christopher Wood <caw@heapingbits.net>
Co-authored-by: Christopher Wood <caw@heapingbits.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants