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 REST API to the httpjson plugin #394

Closed
panda87 opened this issue Nov 25, 2015 · 5 comments · Fixed by #712
Closed

Add REST API to the httpjson plugin #394

panda87 opened this issue Nov 25, 2015 · 5 comments · Fixed by #712
Labels
help wanted Request for community participation, code, contribution

Comments

@panda87
Copy link

panda87 commented Nov 25, 2015

Hi

I found the httpjson plugin as a very good idea, but today as many of the API's are REST based, this will be very useful to add REST capabilities to this plugin.

Thanks
D.

@sparrc
Copy link
Contributor

sparrc commented Dec 1, 2015

@panda87 could you be more specific? What features would you like to see?

@panda87
Copy link
Author

panda87 commented Dec 1, 2015

Today, the plugin is good only for simple GET \ POST.
But, if I want to send more complicated POST as REST API like:
{"type":"test","host":"home"}
This are not possible, since instead of sending the the json request in the body (-d) you serialize it in the url request.

So POST requests to REST API are not working as expected.

@sparrc sparrc added the help wanted Request for community participation, code, contribution label Dec 3, 2015
@miketonks
Copy link
Contributor

Not sure I can see the use case here.

With a REST webservice, GET requests should fetch data and POST requests should (generally) send data to / modify data on the remote server.

This is an input plugin so we want to fetch data. Where's the need to support POST?

@gabelev
Copy link
Contributor

gabelev commented Feb 11, 2016

I agree with this feature request. I think this is a good idea to be able to fetch data also via a POST.

sparrc pushed a commit that referenced this issue Feb 18, 2016
sparrc pushed a commit that referenced this issue Feb 18, 2016
sparrc pushed a commit that referenced this issue Feb 18, 2016
@panda87
Copy link
Author

panda87 commented Feb 18, 2016

Thanks @gabelev

geodimm pushed a commit to miketonks/telegraf that referenced this issue Mar 10, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Request for community participation, code, contribution
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants