LGTM, see suggestion for additional check. https://codereview.appspot.com/13334045/diff/1/src/python/ct/crypto/cert_test.py File src/python/ct/crypto/cert_test.py (right): https://codereview.appspot.com/13334045/diff/1/src/python/ct/crypto/cert_test... src/python/ct/crypto/cert_test.py:147: self.assertEqual(0, c.version()) Also check that calling get_extensions does not throw
https://codereview.appspot.com/13334045/diff/1/src/python/ct/crypto/cert_test.py File src/python/ct/crypto/cert_test.py (right): https://codereview.appspot.com/13334045/diff/1/src/python/ct/crypto/cert_test... src/python/ct/crypto/cert_test.py:147: self.assertEqual(0, c.version()) On 2013/08/30 13:36:47, Eran wrote: > Also check that calling get_extensions does not throw There's no get_extensions yet because I haven't decided on that API, but I've added a check for basicConstraints.