Include cert ID in warning.
This prevents the warning to be not shown for different certificates in the same playbook due to warning de-duplication.pull/844/head
parent
bf70f8d717
commit
3951e6ceb4
|
@ -124,14 +124,15 @@ class Order(object):
|
||||||
replaces_cert_id is not None and
|
replaces_cert_id is not None and
|
||||||
not (exc.error_code == 409 and exc.error_type == 'urn:ietf:params:acme:error:alreadyReplaced')
|
not (exc.error_code == 409 and exc.error_type == 'urn:ietf:params:acme:error:alreadyReplaced')
|
||||||
):
|
):
|
||||||
replaces_cert_id = None
|
|
||||||
if message_callback:
|
if message_callback:
|
||||||
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,
|
code=exc.error_code,
|
||||||
type=exc.error_type,
|
type=exc.error_type,
|
||||||
|
replaces=replaces_cert_id,
|
||||||
)
|
)
|
||||||
)
|
)
|
||||||
|
replaces_cert_id = None
|
||||||
continue
|
continue
|
||||||
|
|
||||||
raise
|
raise
|
||||||
|
|
Loading…
Reference in New Issue