[GitHub] nifi issue #2760: NIFI-5266: Sanitize ES parameters in PutElasticsearchHttp ...

2018-06-18 Thread alopresto
Github user alopresto commented on the issue: https://github.com/apache/nifi/pull/2760 Thanks @mattyb149 . I ran through this using the template you provided. Ran `contrib-check` and all tests pass. I squashed the commits. +1, merging. ---

[GitHub] nifi issue #2760: NIFI-5266: Sanitize ES parameters in PutElasticsearchHttp ...

2018-06-05 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/nifi/pull/2760 +1 (non-binding), contrib-check build looks good, tests updated. ---

[GitHub] nifi issue #2760: NIFI-5266: Sanitize ES parameters in PutElasticsearchHttp ...

2018-06-04 Thread alopresto
Github user alopresto commented on the issue: https://github.com/apache/nifi/pull/2760 Reviewing... ---