From 0bb15e16f96d7167c3d2c8ff28fc75385caa86d4 Mon Sep 17 00:00:00 2001 From: Herman Slatman Date: Fri, 8 Apr 2022 16:10:26 +0200 Subject: [PATCH] Fix missing ACME provisioner option --- authority/provisioner/acme.go | 1 + authority/provisioner/acme_test.go | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/authority/provisioner/acme.go b/authority/provisioner/acme.go index 2bcaeef2..219176fd 100644 --- a/authority/provisioner/acme.go +++ b/authority/provisioner/acme.go @@ -139,6 +139,7 @@ func (p *ACME) AuthorizeOrderIdentifier(ctx context.Context, identifier ACMEIden // on the resulting certificate. func (p *ACME) AuthorizeSign(ctx context.Context, token string) ([]SignOption, error) { opts := []SignOption{ + p, // modifiers / withOptions newProvisionerExtensionOption(TypeACME, p.Name, ""), newForceCNOption(p.ForceCN), diff --git a/authority/provisioner/acme_test.go b/authority/provisioner/acme_test.go index 73342d79..33cbbc75 100644 --- a/authority/provisioner/acme_test.go +++ b/authority/provisioner/acme_test.go @@ -176,7 +176,7 @@ func TestACME_AuthorizeSign(t *testing.T) { } } else { if assert.Nil(t, tc.err) && assert.NotNil(t, opts) { - assert.Len(t, 6, opts) // number of SignOptions returned + assert.Equals(t, 7, len(opts)) // number of SignOptions returned for _, o := range opts { switch v := o.(type) { case *ACME: