Don't process response when the body is empty.

lazaroofarrill commented 1 year ago

bug

What does this PR do?

Return from process_response if the body is empty. Don't process response when the body is empty.

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

This PR fixes the issue that would cause the filter to throw when receiving an empty body in the response. At the moment, when an server returns an empty body, a common example being the result of a DELETE request, the pipeline crashes. When the body is empty the process_response function should return immediately.

Checklist

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

Related issues

Closes #25

cla-checker-service[bot] commented 1 year ago

💚 CLA has been signed

jsvd commented 8 months ago

@logstashmachine bump patch

logstash-plugins / logstash-filter-http