{"action":"created","issue":{"url":"https://api.github.com/repos/influxdata/telegraf/issues/2612","repository_url":"https://api.github.com/repos/influxdata/telegraf","labels_url":"https://api.github.com/repos/influxdata/telegraf/issues/2612/labels{/name}","comments_url":"https://api.github.com/repos/influxdata/telegraf/issues/2612/comments","events_url":"https://api.github.com/repos/influxdata/telegraf/issues/2612/events","html_url":"https://github.com/influxdata/telegraf/pull/2612","id":218955417,"number":2612,"title":"Add input plugin for OpenLDAP","user":{"login":"cobaugh","id":113132,"avatar_url":"https://avatars0.githubusercontent.com/u/113132?v=3","gravatar_id":"","url":"https://api.github.com/users/cobaugh","html_url":"https://github.com/cobaugh","followers_url":"https://api.github.com/users/cobaugh/followers","following_url":"https://api.github.com/users/cobaugh/following{/other_user}","gists_url":"https://api.github.com/users/cobaugh/gists{/gist_id}","starred_url":"https://api.github.com/users/cobaugh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cobaugh/subscriptions","organizations_url":"https://api.github.com/users/cobaugh/orgs","repos_url":"https://api.github.com/users/cobaugh/repos","events_url":"https://api.github.com/users/cobaugh/events{/privacy}","received_events_url":"https://api.github.com/users/cobaugh/received_events","type":"User","site_admin":false},"labels":[{"id":526041350,"url":"https://api.github.com/repos/influxdata/telegraf/labels/pm/extensibility","name":"pm/extensibility","color":"c2e0c6","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":{"url":"https://api.github.com/repos/influxdata/telegraf/milestones/9","html_url":"https://github.com/influxdata/telegraf/milestone/9","labels_url":"https://api.github.com/repos/influxdata/telegraf/milestones/9/labels","id":2278695,"number":9,"title":"1.4.0","description":"","creator":{"login":"sparrc","id":7155926,"avatar_url":"https://avatars2.githubusercontent.com/u/7155926?v=3","gravatar_id":"","url":"https://api.github.com/users/sparrc","html_url":"https://github.com/sparrc","followers_url":"https://api.github.com/users/sparrc/followers","following_url":"https://api.github.com/users/sparrc/following{/other_user}","gists_url":"https://api.github.com/users/sparrc/gists{/gist_id}","starred_url":"https://api.github.com/users/sparrc/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sparrc/subscriptions","organizations_url":"https://api.github.com/users/sparrc/orgs","repos_url":"https://api.github.com/users/sparrc/repos","events_url":"https://api.github.com/users/sparrc/events{/privacy}","received_events_url":"https://api.github.com/users/sparrc/received_events","type":"User","site_admin":false},"open_issues":79,"closed_issues":16,"state":"open","created_at":"2017-01-25T23:19:35Z","updated_at":"2017-05-25T20:45:21Z","due_on":"2017-07-12T07:00:00Z","closed_at":null},"comments":12,"created_at":"2017-04-03T14:40:17Z","updated_at":"2017-05-25T22:52:58Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/influxdata/telegraf/pulls/2612","html_url":"https://github.com/influxdata/telegraf/pull/2612","diff_url":"https://github.com/influxdata/telegraf/pull/2612.diff","patch_url":"https://github.com/influxdata/telegraf/pull/2612.patch"},"body":"This is a plugin to pull metrics out of cn=Monitor from a local or remote OpenLDAP server.\r\n### Required for all PRs:\r\n\r\n- [ ] CHANGELOG.md updated (we recommend not updating this until the PR has been approved by a maintainer)\r\n- [x] Sign [CLA](https://influxdata.com/community/cla/) (if not already signed)\r\n- [x] README.md updated (if adding a new plugin)\r\n"},"comment":{"url":"https://api.github.com/repos/influxdata/telegraf/issues/comments/304145838","html_url":"https://github.com/influxdata/telegraf/pull/2612#issuecomment-304145838","issue_url":"https://api.github.com/repos/influxdata/telegraf/issues/2612","id":304145838,"user":{"login":"danielnelson","id":1048079,"avatar_url":"https://avatars0.githubusercontent.com/u/1048079?v=3","gravatar_id":"","url":"https://api.github.com/users/danielnelson","html_url":"https://github.com/danielnelson","followers_url":"https://api.github.com/users/danielnelson/followers","following_url":"https://api.github.com/users/danielnelson/following{/other_user}","gists_url":"https://api.github.com/users/danielnelson/gists{/gist_id}","starred_url":"https://api.github.com/users/danielnelson/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/danielnelson/subscriptions","organizations_url":"https://api.github.com/users/danielnelson/orgs","repos_url":"https://api.github.com/users/danielnelson/repos","events_url":"https://api.github.com/users/danielnelson/events{/privacy}","received_events_url":"https://api.github.com/users/danielnelson/received_events","type":"User","site_admin":false},"created_at":"2017-05-25T22:52:58Z","updated_at":"2017-05-25T22:52:58Z","body":"The STARTTLS vs ldaps bit did not really sink in for me before, but I think I got it now.\r\n\r\n> I also don't think StartTLS will \"just work\" if you pass in a tlsConfig like DialTLS does.\r\n\r\nIsn't the code currently passing a tls.Config into StartTLS?  Does it work now?\r\n\r\n> If we do want to support LDAPS in the future, we'll need some way of differentiating between starttls and ssl besides the ssl_ca config option being set.\r\n\r\nWhat if we use a origin style connection url?  By this I'm referring to a url of the form `scheme://host:port`.  This way you can control ldap vs ldaps via the scheme.  With a ldap url with certs it would use the starttls method."}}
{"actor":{"display_login":"danielnelson"}}