Skip to content

Commit

Permalink
ecs_certificate: allow to request renewal without csr (#740) (#752)
Browse files Browse the repository at this point in the history
* renew request CSR validation

* Create 740-ecs_certificate-renewal-without-csr

* Rename 740-ecs_certificate-renewal-without-csr to 740-ecs_certificate-renewal-without-csr.yml

---------

Co-authored-by: flovecchio <flovecchio@sorint.com>
(cherry picked from commit 29ac3cb)

Co-authored-by: francescolovecchio <francescolovecchio97@gmail.com>
  • Loading branch information
patchback[bot] and francescolovecchio authored May 9, 2024
1 parent 0ef6494 commit 29ed12e
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
bugfixes:
- "ecs_certificate - fixed ``csr`` option to be empty and allow renewal of a specific certificate according to the Renewal Information specification (https://github.com/ansible-collections/community.crypto/pull/740)."
4 changes: 2 additions & 2 deletions plugins/modules/ecs_certificate.py
Original file line number Diff line number Diff line change
Expand Up @@ -922,8 +922,8 @@ def main():
module.fail_json(msg='The cert_expiry field is invalid when request_type="reissue".')
elif module.params['cert_lifetime']:
module.fail_json(msg='The cert_lifetime field is invalid when request_type="reissue".')
# Only a reissued request can omit the CSR
else:
# Reissued or renew request can omit the CSR
elif module.params['request_type'] != 'renew':
module_params_csr = module.params['csr']
if module_params_csr is None:
module.fail_json(msg='The csr field is required when request_type={0}'.format(module.params['request_type']))
Expand Down

0 comments on commit 29ed12e

Please sign in to comment.