Add support for PUT requests - Githubissues

logstash-plugins / logstash-filter-http

HTTP Filter Plugin for Logstash

Apache License 2.0

12 stars 29 forks source link

Add support for PUT requests #34

Closed fylie closed 2 years ago

fylie commented 3 years ago

What does this PR do?

This PR adds support for the PUT HTTP request method.

Why is it important/What is the impact to the user?

Supporting PUT is important as some APIs (for example the Elasticsearch API) don't support POST/PUT interchangeably.

I noticed that there were multiple other PRs that attempted to add PUT functionality https://github.com/logstash-plugins/logstash-filter-http/pull/32 and https://github.com/logstash-plugins/logstash-filter-http/pull/28, but they haven't been merged due to reasons unrelated to this functionality.

Checklist

[x] My code follows the style guidelines of this project
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] I have made corresponding change to the default configuration files (and/or docker env variables)
[x] I have added tests that prove my fix is effective or that my feature works

Related issues

cla-checker-service[bot] commented 3 years ago

💚 CLA has been signed

jsvd commented 2 years ago

Published as logstash-filter-http-1.1.0