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

[FEATURE REQUEST] Improved POSTs requests (2): automatic POST HTTP verb when using --data #1118

Closed
JulianGR opened this issue Apr 3, 2024 · 6 comments · Fixed by #1114
Closed
Assignees
Labels
enhancement New feature or request

Comments

@JulianGR
Copy link
Contributor

JulianGR commented Apr 3, 2024

Sister issue with #1117

FEATURE REQUEST 2: automatic POST HTTP verb when using --data

Is your feature request related to a problem? Please describe.

When using the --data flag to add data to the body of the request, the HTTP verb that remains is GET (default). Everytime I use --data flag, has to be used together with -m POST

Describe the solution you'd like

I'd like that, when using --data flag, the HTTP verb automatically updates to POST. Also, -m will now only accept POST or PUT. As far as I'm concerned, GET requests are allowed to have body, but the server parses them useless.

Describe alternatives you've considered
N/A

Additional context
N/A

@epi052
Copy link
Owner

epi052 commented Apr 15, 2024

howdy!

this is an easy fix and makes sense to me. i'm going to add this into a PR i already have in progress. thanks!

@JulianGR
Copy link
Contributor Author

nice, thank you!

@epi052
Copy link
Owner

epi052 commented Apr 23, 2024

It's not merged yet, but will be this week likely

@JulianGR
Copy link
Contributor Author

ooops sorry

@JulianGR JulianGR reopened this Apr 24, 2024
@epi052
Copy link
Owner

epi052 commented Apr 27, 2024

@all-contributors add @JulianGR for ideas

Copy link
Contributor

@epi052

I've put up a pull request to add @JulianGR! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants