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

Add LIST to khttpmethod.c #1107

Closed
wants to merge 2 commits into from
Closed

Add LIST to khttpmethod.c #1107

wants to merge 2 commits into from

Conversation

mterron
Copy link
Contributor

@mterron mterron commented Feb 14, 2024

Currently Redbean's Fetch() API does not seem to support arbitrary HTTP methods. This change will enable the use of LIST as one of the supported HTTP verbs to be able to interact with the Hashicorp Vault HTTP API.

Enable LIST HTTP method
@pkulchenko
Copy link
Collaborator

@mterron, the PR looks good to me. Can you send the copyright assignment to @jart per these instructions?

@mterron
Copy link
Contributor Author

mterron commented Feb 15, 2024

@jart , is there a reason for not allowing arbitrary http methods?

Copy link
Owner

@jart jart left a comment

Choose a reason for hiding this comment

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

Approval for copyright.

net/http/khttpmethod.c Show resolved Hide resolved
@jart jart closed this in ad3944a Feb 22, 2024
@mterron mterron deleted the patch-1 branch February 22, 2024 22:49
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.

3 participants