'HEAD' verb case handled - Githubissues

logstash-plugins / logstash-filter-http

HTTP Filter Plugin for Logstash

Apache License 2.0

12 stars 29 forks source link

'HEAD' verb case handled #40

Closed nitish2018 closed 2 years ago

nitish2018 commented 2 years ago

Need not go to process_response(body, headers, event) in case of 'HEAD' verb

Release notes

What does this PR do?

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

Checklist

[ ] My code follows the style guidelines of this project
[ ] 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

Author's Checklist

[ ]

How to test this PR locally

Related issues

Use cases

Screenshots

Logs

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

💚 CLA has been signed

nitish2018 commented 2 years ago

@jsvd please review it

nitish2018 commented 2 years ago

@kares can you please review?

jsvd commented 2 years ago

@nitish2018 this change has been included in https://github.com/logstash-plugins/logstash-filter-http/pull/41, merged, and will be published as v1.4.1. Attribution to you has been preserved (https://github.com/logstash-plugins/logstash-filter-http/commit/8dfe8fb1a1c40782cacd98de8dc7feddaae5b313).

jsvd commented 2 years ago

@nitish2018 this change has been included in https://github.com/logstash-plugins/logstash-filter-http/pull/41, merged, and will be published as v1.4.1. Attribution to you has been preserved (https://github.com/logstash-plugins/logstash-filter-http/commit/8dfe8fb1a1c40782cacd98de8dc7feddaae5b313).