{"action":"created","issue":{"url":"https://api.github.com/repos/nextcloud/server/issues/600","repository_url":"https://api.github.com/repos/nextcloud/server","labels_url":"https://api.github.com/repos/nextcloud/server/issues/600/labels{/name}","comments_url":"https://api.github.com/repos/nextcloud/server/issues/600/comments","events_url":"https://api.github.com/repos/nextcloud/server/issues/600/events","html_url":"https://github.com/nextcloud/server/pull/600","id":167828956,"number":600,"title":"Enable changing passwords in user_ldap","user":{"login":"GitHubUser4234","id":16538178,"avatar_url":"https://avatars.githubusercontent.com/u/16538178?v=3","gravatar_id":"","url":"https://api.github.com/users/GitHubUser4234","html_url":"https://github.com/GitHubUser4234","followers_url":"https://api.github.com/users/GitHubUser4234/followers","following_url":"https://api.github.com/users/GitHubUser4234/following{/other_user}","gists_url":"https://api.github.com/users/GitHubUser4234/gists{/gist_id}","starred_url":"https://api.github.com/users/GitHubUser4234/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/GitHubUser4234/subscriptions","organizations_url":"https://api.github.com/users/GitHubUser4234/orgs","repos_url":"https://api.github.com/users/GitHubUser4234/repos","events_url":"https://api.github.com/users/GitHubUser4234/events{/privacy}","received_events_url":"https://api.github.com/users/GitHubUser4234/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/nextcloud/server/labels/enhancement","name":"enhancement","color":"0e8a16"},{"url":"https://api.github.com/repos/nextcloud/server/labels/feature:%20ldap","name":"feature: ldap","color":"c2e0c6"}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":{"url":"https://api.github.com/repos/nextcloud/server/milestones/6","html_url":"https://github.com/nextcloud/server/milestones/Nextcloud%2011.0","labels_url":"https://api.github.com/repos/nextcloud/server/milestones/6/labels","id":1896997,"number":6,"title":"Nextcloud 11.0","description":null,"creator":{"login":"MorrisJobke","id":245432,"avatar_url":"https://avatars.githubusercontent.com/u/245432?v=3","gravatar_id":"","url":"https://api.github.com/users/MorrisJobke","html_url":"https://github.com/MorrisJobke","followers_url":"https://api.github.com/users/MorrisJobke/followers","following_url":"https://api.github.com/users/MorrisJobke/following{/other_user}","gists_url":"https://api.github.com/users/MorrisJobke/gists{/gist_id}","starred_url":"https://api.github.com/users/MorrisJobke/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MorrisJobke/subscriptions","organizations_url":"https://api.github.com/users/MorrisJobke/orgs","repos_url":"https://api.github.com/users/MorrisJobke/repos","events_url":"https://api.github.com/users/MorrisJobke/events{/privacy}","received_events_url":"https://api.github.com/users/MorrisJobke/received_events","type":"User","site_admin":false},"open_issues":36,"closed_issues":119,"state":"open","created_at":"2016-07-21T15:58:16Z","updated_at":"2016-08-10T09:13:02Z","due_on":null,"closed_at":null},"comments":21,"created_at":"2016-07-27T11:10:05Z","updated_at":"2016-08-10T09:45:27Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/nextcloud/server/pulls/600","html_url":"https://github.com/nextcloud/server/pull/600","diff_url":"https://github.com/nextcloud/server/pull/600.diff","patch_url":"https://github.com/nextcloud/server/pull/600.patch"},"body":"This is based on #519, it has been discussed  at [#25114](https://github.com/owncloud/core/pull/25114) and covers the \"Ability to change LDAP password\" part that is described there. Password policy related errors should be able to be displayed for both LDAP and AD users. The committed source code is fully functional in this regard. Unit test will be added tomorrow."},"comment":{"url":"https://api.github.com/repos/nextcloud/server/issues/comments/238817781","html_url":"https://github.com/nextcloud/server/pull/600#issuecomment-238817781","issue_url":"https://api.github.com/repos/nextcloud/server/issues/600","id":238817781,"user":{"login":"blizzz","id":2184312,"avatar_url":"https://avatars.githubusercontent.com/u/2184312?v=3","gravatar_id":"","url":"https://api.github.com/users/blizzz","html_url":"https://github.com/blizzz","followers_url":"https://api.github.com/users/blizzz/followers","following_url":"https://api.github.com/users/blizzz/following{/other_user}","gists_url":"https://api.github.com/users/blizzz/gists{/gist_id}","starred_url":"https://api.github.com/users/blizzz/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/blizzz/subscriptions","organizations_url":"https://api.github.com/users/blizzz/orgs","repos_url":"https://api.github.com/users/blizzz/repos","events_url":"https://api.github.com/users/blizzz/events{/privacy}","received_events_url":"https://api.github.com/users/blizzz/received_events","type":"User","site_admin":false},"created_at":"2016-08-10T09:45:27Z","updated_at":"2016-08-10T09:45:27Z","body":"My late feedback:\r\n* we don't want to do fancy hashing on the client side\r\n* we need work around ldappasswd\r\n* going with ldapmodify for now is all we can do\r\n* [ ] next to the checkbox should be information that the password is sent in plaintext to the ldap server and transport encryption is a must. and also, that the LDAP server should be configured to deal with that.\r\n* [ ] a documentation section shall be added that explains this in details and points to ways to configure popular LDAP servers. And maybe a statement that if this is not feasible, the feature should just not be used.\r\n* or, alternatively, indeed we can offer some hashing options use a select box instead of a checkbox. "}}
{"actor":{"display_login":"blizzz"}}