Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Multiple certificates in response #1

Open
dbrgn opened this issue Apr 11, 2016 · 2 comments
Open

Multiple certificates in response #1

dbrgn opened this issue Apr 11, 2016 · 2 comments

Comments

@dbrgn
Copy link
Contributor

dbrgn commented Apr 11, 2016

Does ocspbuilder already support multiple certificates in the response? To quote RFC 6960:

The response MUST include a SingleResponse for each certificate in the request.

@wbond
Copy link
Owner

wbond commented Apr 11, 2016

No, currently the builder does not support multiple certificates. The ASN.1 definitions from asn1crypto.ocsp do, but the API for this builder would need to be augmented to support multiple certificates in a few places.

@andrea-f
Copy link

andrea-f commented Nov 13, 2018

@dbrgn and @wbond (thanks for the repo :) I added multi cert response support to OCSPBuilder to conform with RFC6060, code is here: https://github.com/andrea-f/ocspbuilder, the tests in test_ocsp_response_builder.py are passing. I can't raise a PR but let me know what you think!!

For example a query consisting of:

$ openssl ocsp -issuer ./ca-cert.pem -cert ./1.pem -cert ./2.pem -cert ./3.pem -no_nonce -url http://localhost -noverify -header "Host" "localhost"

Yields:

./1.pem: good
	This Update: Nov 12 23:17:07 2018 GMT
	Next Update: Nov 19 23:17:07 2018 GMT
./2.pem: good
	This Update: Nov 12 23:17:07 2018 GMT
	Next Update: Nov 19 23:17:07 2018 GMT
./3.pem: good
	This Update: Nov 12 23:17:07 2018 GMT
	Next Update: Nov 19 23:17:07 2018 GMT

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

No branches or pull requests

3 participants