diff --git a/acceptance/suites/pre_suite/foss/70_install_puppet.rb b/acceptance/suites/pre_suite/foss/70_install_puppet.rb index 166d741a6..33ed74a82 100644 --- a/acceptance/suites/pre_suite/foss/70_install_puppet.rb +++ b/acceptance/suites/pre_suite/foss/70_install_puppet.rb @@ -21,10 +21,10 @@ nss_package_name="nss" end if nss_package_name - if master['platform'] != 'el-8-x86_64' - master.upgrade_package(nss_package_name) - else + if master['platform'] == 'el-8-x86_64' || master['platform'] == 'el-9-x86_64' master.install_package(nss_package_name) + else + master.upgrade_package(nss_package_name) end else logger.warn("Don't know what nss package to use for #{variant} so not installing one") diff --git a/src/clj/puppetlabs/services/ca/certificate_authority_core.clj b/src/clj/puppetlabs/services/ca/certificate_authority_core.clj index 872c8fffc..d89934524 100644 --- a/src/clj/puppetlabs/services/ca/certificate_authority_core.clj +++ b/src/clj/puppetlabs/services/ca/certificate_authority_core.clj @@ -140,9 +140,12 @@ ca-settings :- ca/CaSettings] (let [response (ca/delete-certificate-request! ca-settings subject) outcomes->codes {:success 204 :not-found 404 :error 500}] - (-> (rr/response (:message response)) - (rr/status ((response :outcome) outcomes->codes)) - (rr/content-type "text/plain")))) + (if (not= (response :outcome) :success) + (-> (rr/response (:message response)) + (rr/status ((response :outcome) outcomes->codes)) + (rr/content-type "text/plain")) + (-> (rr/response (:message response)) + (rr/status ((response :outcome) outcomes->codes)))))) (schema/defn handle-get-ca-expirations [ca-settings :- ca/CaSettings] diff --git a/test/unit/puppetlabs/services/ca/certificate_authority_core_test.clj b/test/unit/puppetlabs/services/ca/certificate_authority_core_test.clj index 841ca87c6..9c149c560 100644 --- a/test/unit/puppetlabs/services/ca/certificate_authority_core_test.clj +++ b/test/unit/puppetlabs/services/ca/certificate_authority_core_test.clj @@ -327,7 +327,6 @@ (is (false? (fs/exists? expected-path))) (is (= 204 (:status response))) (is (re-matches msg-matcher (:body response))) - (is (= "text/plain" (get-in response [:headers "Content-Type"]))) (is (logged? msg-matcher :debug))) (finally (fs/delete expected-path))))))