{"action":"created","issue":{"url":"https://api.github.com/repos/owncloud/core/issues/17110","labels_url":"https://api.github.com/repos/owncloud/core/issues/17110/labels{/name}","comments_url":"https://api.github.com/repos/owncloud/core/issues/17110/comments","events_url":"https://api.github.com/repos/owncloud/core/issues/17110/events","html_url":"https://github.com/owncloud/core/pull/17110","id":90415946,"number":17110,"title":"[sharing] handle shares of users that aren't available anymore","user":{"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},"labels":[{"url":"https://api.github.com/repos/owncloud/core/labels/3%20-%20To%20Review","name":"3 - To Review","color":"ededed"},{"url":"https://api.github.com/repos/owncloud/core/labels/sev2-high","name":"sev2-high","color":"fc2929"},{"url":"https://api.github.com/repos/owncloud/core/labels/sharing","name":"sharing","color":"e102d8"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-06-23T15:16:03Z","updated_at":"2015-06-23T15:53:39Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/owncloud/core/pulls/17110","html_url":"https://github.com/owncloud/core/pull/17110","diff_url":"https://github.com/owncloud/core/pull/17110.diff","patch_url":"https://github.com/owncloud/core/pull/17110.patch"},"body":"* properly handle the case where an abandoned share is left and\r\n  simply skip it\r\n\r\n\r\n@karlitschek Should be backported to all stable branches.\r\n\r\nHow to test this:\r\n\r\n* have two LDAP users test1 and test2 (this is not only for LDAP, but for every external user backend, that allows user deletion without notifying ownCloud)\r\n* test1 shares \"abc\"  to test2\r\n* delete test1 inside of LDAP\r\n* login as test2\r\n\r\n### expected\r\n\r\n* the share is not available anymore\r\n\r\n### actual\r\n\r\n* HTTP 500 with a NoUserException in the logs\r\n\r\n@butonic @blizzz we discussed this\r\n\r\n@schiesbn Or should we delete the share right away?\r\n\r\nHow should we handle this case? Is this the way to go? Then simply files disappear from users, because of a user deletion. But this is also the case if an ownCloud internal user gets deleted.\r\n\r\ncc @nickvergessen @lukasreschke"},"comment":{"url":"https://api.github.com/repos/owncloud/core/issues/comments/114555224","html_url":"https://github.com/owncloud/core/pull/17110#issuecomment-114555224","issue_url":"https://api.github.com/repos/owncloud/core/issues/17110","id":114555224,"user":{"login":"scrutinizer-notifier","id":6398257,"avatar_url":"https://avatars.githubusercontent.com/u/6398257?v=3","gravatar_id":"","url":"https://api.github.com/users/scrutinizer-notifier","html_url":"https://github.com/scrutinizer-notifier","followers_url":"https://api.github.com/users/scrutinizer-notifier/followers","following_url":"https://api.github.com/users/scrutinizer-notifier/following{/other_user}","gists_url":"https://api.github.com/users/scrutinizer-notifier/gists{/gist_id}","starred_url":"https://api.github.com/users/scrutinizer-notifier/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/scrutinizer-notifier/subscriptions","organizations_url":"https://api.github.com/users/scrutinizer-notifier/orgs","repos_url":"https://api.github.com/users/scrutinizer-notifier/repos","events_url":"https://api.github.com/users/scrutinizer-notifier/events{/privacy}","received_events_url":"https://api.github.com/users/scrutinizer-notifier/received_events","type":"User","site_admin":false},"created_at":"2015-06-23T15:53:39Z","updated_at":"2015-06-23T15:53:39Z","body":"A [new inspection](https://scrutinizer-ci.com/g/owncloud/core/inspections/5c99bad7-421f-4c9b-98f1-d32df1f16145) was created."}}