Skip to content

Commit

Permalink
Include cert ID in warning.
Browse files Browse the repository at this point in the history
This prevents the warning to be not shown for different certificates
in the same playbook due to warning de-duplication.
  • Loading branch information
felixfontein committed Jan 19, 2025
1 parent bf70f8d commit 3951e6c
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions plugins/module_utils/acme/orders.py
Original file line number Diff line number Diff line change
Expand Up @@ -124,14 +124,15 @@ def create_with_error_handling(
replaces_cert_id is not None and
not (exc.error_code == 409 and exc.error_type == 'urn:ietf:params:acme:error:alreadyReplaced')
):
replaces_cert_id = None
if message_callback:
message_callback(
'Stop passing `replaces` due to error {code} {type} when creating ACME order'.format(
'Stop passing `replaces={replaces}` due to error {code} {type} when creating ACME order'.format(
code=exc.error_code,
type=exc.error_type,
replaces=replaces_cert_id,
)
)
replaces_cert_id = None
continue

raise
Expand Down

0 comments on commit 3951e6c

Please sign in to comment.