Skip to content

Commit c1af4aa

Browse files
committed
Replace openssl argument with comment explaining why we don't need it
`-nameopt compat` basically has the opposite behavior in openssl 1.0 vs 1.1 but the field we're parsing is unaffected by it.
1 parent 8615bdb commit c1af4aa

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

osgtest/tests/test_407_voms.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,9 @@ def test_08_voms_proxy_check(self):
8080

8181
pwd_entry = pwd.getpwnam(core.options.username)
8282
cert_path = os.path.join(pwd_entry.pw_dir, '.globus', 'usercert.pem')
83-
command = ['openssl', 'x509', '-in', cert_path, '-nameopt', 'compat', '-text']
83+
# Note: We're only looking for the "Signature Algorithm" line which has the same output format
84+
# regardless of openssl version.
85+
command = ['openssl', 'x509', '-in', cert_path, '-text']
8486
signature_re = re.compile(r'Signature Algorithm:\s+(\w+)\s')
8587
stdout = core.check_system(command, 'Check X.509 certificate algorithm', user=True)[0]
8688
match = signature_re.search(stdout)

0 commit comments

Comments
 (0)