Android AIA fetching does not parse PEM-encoded certificates |
|||
Issue descriptionThe AIA fetching logic in CertVerifyProcAndroid doesn't parse PEM-encoded certificates from AIA endpoints, such as http://certificates.godaddy.com/repository/gdig2.crt I can't seem to find any way to distinguish whether to expect PEM or not (e.g. there is no Content-Type header on http://certificates.godaddy.com/repository/gdig2.crt), so perhaps we should just try parsing with PEMTokenizer if ParsedCertificate fails to parse the fetched certificate on the first try.
,
May 22 2017
Ok, would you like to do the reaching-out yourself or do you want me to? (I'm fine either way but would need an intro and/or contacts if you want me to do it.) They are also not the only CA doing this; I'll attempt to put together a somewhat comprehensive list.
,
May 22 2017
Just did :) And yeah, that's something that I suppose we should have done sooner. It's not hard to get the AIA list from CT and check the compliance to these aspects. Happy to nag everyone, to see if there's some use case or interoperability concern we've missed. If it's a large number, we might introduce a compat shim to allow it for a time, and I'll ratchet up pressure on CAs publicly and privately so that we could ultimately remove it. Unfortunately, there's no good "AIA horked" reporting at the moment - although we could always suggest it to be added to certlint and/or crt.sh relatively easily (PRs welcome!). That might be a more effective long-term mitigation strategy.
,
May 22 2017
This list is probably not comprehensive but looks like it accounts for most of the failures that we get in Chrome: AlphaSSL: http://secure2.alphassl.com/cacert/gsalphasha2g2r1.crt Starfield: http://certificates.starfieldtech.com/repository/sfig2.crt Cybertrust: http://sureseries-crl.cybertrust.ne.jp/SureServer/2021_ev/ctjevcag2_sha256.crt (can pull a more comprehensive list later this afternoon)
,
May 23 2017
We updated the AlphaSSL CA certificate to be DER.
,
May 23 2017
Thanks Doug! Let us know if there are any issues, and we can certainly discuss in the CA/Browser Forum if there's concrete data about clients that expect PEM rather than DER - and we can look to express that both in the BRs and our client code, much like the discussions around nameConstraints :)
,
May 23 2017
DigiCert has confirmed they've worked with Cybertrust Japan to correct http://sureseries-crl.cybertrust.ne.jp/SureServer/2021_ev/ctjevcag2_sha256.crt GoDaddy has indicated they expect to make the change in 3 weeks.
,
Mar 16 2018
,
Aug 2
|
|||
►
Sign in to add a comment |
|||
Comment 1 by rsleevi@chromium.org
, May 22 2017