{"action":"created","issue":{"url":"https://api.github.com/repos/logstash-plugins/logstash-output-elasticsearch/issues/176","labels_url":"https://api.github.com/repos/logstash-plugins/logstash-output-elasticsearch/issues/176/labels{/name}","comments_url":"https://api.github.com/repos/logstash-plugins/logstash-output-elasticsearch/issues/176/comments","events_url":"https://api.github.com/repos/logstash-plugins/logstash-output-elasticsearch/issues/176/events","html_url":"https://github.com/logstash-plugins/logstash-output-elasticsearch/pull/176","id":88801480,"number":176,"title":"Http Path support","user":{"login":"andrewvc","id":131427,"avatar_url":"https://avatars.githubusercontent.com/u/131427?v=3","gravatar_id":"","url":"https://api.github.com/users/andrewvc","html_url":"https://github.com/andrewvc","followers_url":"https://api.github.com/users/andrewvc/followers","following_url":"https://api.github.com/users/andrewvc/following{/other_user}","gists_url":"https://api.github.com/users/andrewvc/gists{/gist_id}","starred_url":"https://api.github.com/users/andrewvc/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/andrewvc/subscriptions","organizations_url":"https://api.github.com/users/andrewvc/orgs","repos_url":"https://api.github.com/users/andrewvc/repos","events_url":"https://api.github.com/users/andrewvc/events{/privacy}","received_events_url":"https://api.github.com/users/andrewvc/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-06-16T18:19:53Z","updated_at":"2015-06-18T22:39:40Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/logstash-plugins/logstash-output-elasticsearch/pulls/176","html_url":"https://github.com/logstash-plugins/logstash-output-elasticsearch/pull/176","diff_url":"https://github.com/logstash-plugins/logstash-output-elasticsearch/pull/176.diff","patch_url":"https://github.com/logstash-plugins/logstash-output-elasticsearch/pull/176.patch"},"body":"This fixes #168 , adding HTTP Path support. The unit tests here just check that the params are correct and are well formed(ish). I hand tested this against an Nginx proxy->elasticsearch, and it seems to work correctly. The manual test config I used was:\r\n\r\n```\r\ninput {\r\n  generator {\r\n    lines => [\r\n      \"test 1\",\r\n      \"test 2\"\r\n    ]\r\n    count => 1\r\n  }\r\n}\r\n\r\noutput {\r\n  elasticsearch {\r\n          host => \"127.0.0.1\"\r\n          port => 8080\r\n          path => \"elastic-proxy\"\r\n          protocol => \"http\"\r\n        }\r\n\r\n}\r\n\r\n```"},"comment":{"url":"https://api.github.com/repos/logstash-plugins/logstash-output-elasticsearch/issues/comments/113306906","html_url":"https://github.com/logstash-plugins/logstash-output-elasticsearch/pull/176#issuecomment-113306906","issue_url":"https://api.github.com/repos/logstash-plugins/logstash-output-elasticsearch/issues/176","id":113306906,"user":{"login":"jordansissel","id":131818,"avatar_url":"https://avatars.githubusercontent.com/u/131818?v=3","gravatar_id":"","url":"https://api.github.com/users/jordansissel","html_url":"https://github.com/jordansissel","followers_url":"https://api.github.com/users/jordansissel/followers","following_url":"https://api.github.com/users/jordansissel/following{/other_user}","gists_url":"https://api.github.com/users/jordansissel/gists{/gist_id}","starred_url":"https://api.github.com/users/jordansissel/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jordansissel/subscriptions","organizations_url":"https://api.github.com/users/jordansissel/orgs","repos_url":"https://api.github.com/users/jordansissel/repos","events_url":"https://api.github.com/users/jordansissel/events{/privacy}","received_events_url":"https://api.github.com/users/jordansissel/received_events","type":"User","site_admin":false},"created_at":"2015-06-18T22:39:40Z","updated_at":"2015-06-18T22:39:40Z","body":"Other than my last comment (https://github.com/logstash-plugins/logstash-output-elasticsearch/pull/176#discussion_r32785644), LGTM. Looks like you may need to rebase since we just merged @talevy's epic test restructuring patch."}}