logstash-plugins / logstash-filter-dns

Apache License 2.0
7 stars 28 forks source link

Update dns.rb #14

Closed briner closed 8 years ago

briner commented 9 years ago

type is depreceted and not anymore on the fields describe in the documentation

purbon commented 9 years ago

@briner thanks a lot for your contribution, before moving forward it will be necessary that you sign the CLA, you can see more info at https://www.elastic.co/contributor-agreement . Keep in mind to use the same email as the one used in your PR, this helps our do the necessary checks.

briner commented 9 years ago

the CLA is signed.

elasticsearch-release commented 9 years ago

Jenkins standing by to test this. If you aren't a maintainer, you can ignore this comment. Someone with commit access, please review this and clear it for Jenkins to run; then say 'jenkins, test it'.

guyboertje commented 8 years ago

LGTM