net/data/name_constraints_unittest/san-permitted.pem - Issue 1685023002: Fix API mismatch between NameConstraints::IsPermittedCert's subjectAltName param and ParseExtension.

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Side by Side Diff: net/data/name_constraints_unittest/san-permitted.pem

Issue 1685023002: Fix API mismatch between NameConstraints::IsPermittedCert's subjectAltName param and ParseExtension. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@name-constraints-directorynameparsing

Patch Set: Created 4 years, 10 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View unified diff | Download patch

« net/cert/internal/name_constraints.h ('K') | « net/data/name_constraints_unittest/san-othername.pem ('k') | net/data/name_constraints_unittest/san-registeredid.pem » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

OLD	NEW
1 0:d=0 hl=2 l= 66 prim: OCTET STRING [HEX DUMP]:304082157065726D697474 65642E6578616D706C652E636F6D8704C0A80102A421301F310B3009060355040613025553311030 0E06035504080C074172697A6F6E61	1 0:d=0 hl=2 l= 64 cons: SEQUENCE
	eroman 2016/02/10 17:30:09 A nice consequence of this change is this pretty p A nice consequence of this change is this pretty prints better now :) mattm 2016/02/10 22:48:40 Yep :) Show quoted text On 2016/02/10 17:30:09, eroman wrote: > A nice consequence of this change is this pretty prints better now :) Yep :)
	2 2:d=1 hl=2 l= 21 prim: cont [ 2 ]

	3 25:d=1 hl=2 l= 4 prim: cont [ 7 ]

	4 31:d=1 hl=2 l= 33 cons: cont [ 4 ]

	5 33:d=2 hl=2 l= 31 cons: SEQUENCE

	6 35:d=3 hl=2 l= 11 cons: SET

	7 37:d=4 hl=2 l= 9 cons: SEQUENCE

	8 39:d=5 hl=2 l= 3 prim: OBJECT :countryName

	9 44:d=5 hl=2 l= 2 prim: PRINTABLESTRING :US

	10 48:d=3 hl=2 l= 16 cons: SET

	11 50:d=4 hl=2 l= 14 cons: SEQUENCE

	12 52:d=5 hl=2 l= 3 prim: OBJECT :stateOrProvinceName

	13 57:d=5 hl=2 l= 7 prim: UTF8STRING :Arizona

2 -----BEGIN SUBJECT ALTERNATIVE NAME-----	14 -----BEGIN SUBJECT ALTERNATIVE NAME-----

3 BEIwQIIVcGVybWl0dGVkLmV4YW1wbGUuY29thwTAqAECpCEwHzELMAkGA1UEBhMCVVMxEDAOBgNV	15 MECCFXBlcm1pdHRlZC5leGFtcGxlLmNvbYcEwKgBAqQhMB8xCzAJBgNVBAYTAlVTMRAwDgYDVQQI

4 BAgMB0FyaXpvbmE=	16 DAdBcml6b25h

5 -----END SUBJECT ALTERNATIVE NAME-----	17 -----END SUBJECT ALTERNATIVE NAME-----

OLD	NEW