{"action":"created","issue":{"url":"https://api.github.com/repos/cyverse/DE/issues/69","repository_url":"https://api.github.com/repos/cyverse/DE","labels_url":"https://api.github.com/repos/cyverse/DE/issues/69/labels{/name}","comments_url":"https://api.github.com/repos/cyverse/DE/issues/69/comments","events_url":"https://api.github.com/repos/cyverse/DE/issues/69/events","html_url":"https://github.com/cyverse/DE/pull/69","id":131458933,"number":69,"title":"CORE-6215: migrate /secured/filesystem/anon-files","user":{"login":"ianmcorvidae","id":106004,"avatar_url":"https://avatars.githubusercontent.com/u/106004?v=3","gravatar_id":"","url":"https://api.github.com/users/ianmcorvidae","html_url":"https://github.com/ianmcorvidae","followers_url":"https://api.github.com/users/ianmcorvidae/followers","following_url":"https://api.github.com/users/ianmcorvidae/following{/other_user}","gists_url":"https://api.github.com/users/ianmcorvidae/gists{/gist_id}","starred_url":"https://api.github.com/users/ianmcorvidae/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ianmcorvidae/subscriptions","organizations_url":"https://api.github.com/users/ianmcorvidae/orgs","repos_url":"https://api.github.com/users/ianmcorvidae/repos","events_url":"https://api.github.com/users/ianmcorvidae/events{/privacy}","received_events_url":"https://api.github.com/users/ianmcorvidae/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2016-02-04T20:03:49Z","updated_at":"2016-02-05T01:07:38Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/cyverse/DE/pulls/69","html_url":"https://github.com/cyverse/DE/pull/69","diff_url":"https://github.com/cyverse/DE/pull/69.diff","patch_url":"https://github.com/cyverse/DE/pull/69.patch"},"body":"The first commit removes an AVU-based hack that we used to use for revoking permissions on unsharing, but don't any more. Since the code was also written to circumvent a jargon bug at the time, it also used the units field as the attribute field, so it added very unuseful AVUs, and only to user home directories, which usually the AVUs can't even be seen on.\r\n\r\nThis also migrates the start of some sharing stuff, but that will probably be refactored as I migrate the actual sharing endpoints, since I'd like to make those more file-centric. Currently the function is written to take only one permission, a list of users, and a list of files, and it assigns each user that permission on each file. With a future PR, I'll most likely change this to expect a single file (per invocation) and a list of user-permission pairs."},"comment":{"url":"https://api.github.com/repos/cyverse/DE/issues/comments/180133211","html_url":"https://github.com/cyverse/DE/pull/69#issuecomment-180133211","issue_url":"https://api.github.com/repos/cyverse/DE/issues/69","id":180133211,"user":{"login":"ianmcorvidae","id":106004,"avatar_url":"https://avatars.githubusercontent.com/u/106004?v=3","gravatar_id":"","url":"https://api.github.com/users/ianmcorvidae","html_url":"https://github.com/ianmcorvidae","followers_url":"https://api.github.com/users/ianmcorvidae/followers","following_url":"https://api.github.com/users/ianmcorvidae/following{/other_user}","gists_url":"https://api.github.com/users/ianmcorvidae/gists{/gist_id}","starred_url":"https://api.github.com/users/ianmcorvidae/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ianmcorvidae/subscriptions","organizations_url":"https://api.github.com/users/ianmcorvidae/orgs","repos_url":"https://api.github.com/users/ianmcorvidae/repos","events_url":"https://api.github.com/users/ianmcorvidae/events{/privacy}","received_events_url":"https://api.github.com/users/ianmcorvidae/received_events","type":"User","site_admin":false},"created_at":"2016-02-05T01:07:38Z","updated_at":"2016-02-05T01:07:38Z","body":"Ah, right. I'd initially started refactoring the share function to be user-focused as mentioned, and switched the arguments to reflect that. But, I pulled out most of those changes. I'll put them back the previous order tomorrow just to make the steps of the process more distinct.\n\nOn February 4, 2016 6:05:18 PM MST, Paul Sarando <notifications@github.com> wrote:\n>Is there a particular reason the `share-withs` and `fpaths` arguments\n>are swapped for the `share` and `share-paths` functions? Was this just\n>copied that way from terrain?\n>\n>Otherwise, I didn't catch any issues with this PR. :shipit: \n>\n>---\n>Reply to this email directly or view it on GitHub:\n>https://github.com/cyverse/DE/pull/69#issuecomment-180132887\n"}}