Skip to content

Commit 41fd6a7

Browse files
committed
fix(revoke-cert): Adds code removed by error
The following PR removed some logic on revoke_cert by mistake. https://github.com/Venafi/vcert-python/pull/142/files#diff-1c9593e2ac8cef81817b447b1aa42b4385daaefd12f5ca5478c1a425ec387b18L342
1 parent f703ef5 commit 41fd6a7

1 file changed

Lines changed: 4 additions & 0 deletions

File tree

vcert/connection_tpp_abstract.py

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -331,17 +331,21 @@ def renew_cert(self, request, reuse_key=False):
331331
def revoke_cert(self, request):
332332
if not (request.id or request.thumbprint):
333333
raise ClientBadData
334+
334335
d = {
335336
'Disable': request.disable
336337
}
337338
if request.reason:
338339
d['Reason'] = request.reason
340+
if request.comments:
341+
d['Comments'] = request.comments
339342
if request.id:
340343
d['CertificateDN'] = request.id
341344
elif request.thumbprint:
342345
d['Thumbprint'] = request.thumbprint
343346
else:
344347
raise ClientBadData
348+
345349
# TODO: Change _post() with post(args)
346350
status, data = self._post(URLS.CERTIFICATE_REVOKE, data=d)
347351
if status in (HTTPStatus.OK, HTTPStatus.ACCEPTED):

0 commit comments

Comments
 (0)