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.

Sign up for GitHub

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

Improve copy to curl #546

Open

centic9 wants to merge 2 commits into lmenezes:main from centic9:improve_copy_to_curl

centic9 commented Jan 18, 2022

Copying to cURL has two issues:

if the hostname in the configuration ends with a slash, the resulting cURL has double slashes, which renders it invalid
HTTP GET can also have a body as part of the request, e.g. when performing Elasticsearch queries

This PR solves both issues.

centic9 added 2 commits

January 18, 2022 09:31


When copying cURL cut off trailing slash from hostname

414e117

This would produce invalid URLs with double slashes.


Include body for GET in copy as cURL

11389a2

This is possible and used e.g. for sending Elasticsearch queries via REST API

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment