Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

(PE-36076) Remove content type from certificate reject #2769

Merged
merged 1 commit into from
Aug 2, 2023

Conversation

artlawson
Copy link
Contributor

Previously, when a DELETE request was sent to puppet-ca/v1/certificate_status/<nodename> a 204 response was sent with a malformed content-type header. This commit removes the header.

@artlawson artlawson requested a review from a team as a code owner July 31, 2023 18:21
@artlawson artlawson force-pushed the pe_36076 branch 4 times, most recently from 46fef94 to 1a5b302 Compare July 31, 2023 19:11
@@ -350,7 +348,6 @@
msg-matcher (re-pattern (str "Path " expected-path " exists but.*"))]
(is (= 500 (:status response)))
(is (re-matches msg-matcher (:body response)))
(is (= "text/plain" (get-in response [:headers "Content-Type"])))
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems odd. The 500 should return some sort of error and a Content-Type along with it. It may be that you'll need to add the content type specifically to the 500 error (which has a body).

@artlawson artlawson force-pushed the pe_36076 branch 6 times, most recently from 11c5c53 to eca725b Compare August 1, 2023 20:26
Previously, when a `DELETE` request was  sent to `puppet-ca/v1/certificate_status/<nodename>`
a 204 response was sent with a malformed content-type header. This commit removes the header.
Copy link
Contributor

@steveax steveax left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@steveax steveax merged commit fd64d69 into puppetlabs:7.x Aug 2, 2023
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants