{"action":"created","issue":{"url":"https://api.github.com/repos/golang/go/issues/27871","repository_url":"https://api.github.com/repos/golang/go","labels_url":"https://api.github.com/repos/golang/go/issues/27871/labels{/name}","comments_url":"https://api.github.com/repos/golang/go/issues/27871/comments","events_url":"https://api.github.com/repos/golang/go/issues/27871/events","html_url":"https://github.com/golang/go/pull/27871","id":363919941,"node_id":"MDExOlB1bGxSZXF1ZXN0MjE4MjM0Mzgw","number":27871,"title":"os: add support for long path names on unix RemoveAll","user":{"login":"ostenbom","id":7828615,"node_id":"MDQ6VXNlcjc4Mjg2MTU=","avatar_url":"https://avatars0.githubusercontent.com/u/7828615?v=4","gravatar_id":"","url":"https://api.github.com/users/ostenbom","html_url":"https://github.com/ostenbom","followers_url":"https://api.github.com/users/ostenbom/followers","following_url":"https://api.github.com/users/ostenbom/following{/other_user}","gists_url":"https://api.github.com/users/ostenbom/gists{/gist_id}","starred_url":"https://api.github.com/users/ostenbom/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ostenbom/subscriptions","organizations_url":"https://api.github.com/users/ostenbom/orgs","repos_url":"https://api.github.com/users/ostenbom/repos","events_url":"https://api.github.com/users/ostenbom/events{/privacy}","received_events_url":"https://api.github.com/users/ostenbom/received_events","type":"User","site_admin":false},"labels":[{"id":831707000,"node_id":"MDU6TGFiZWw4MzE3MDcwMDA=","url":"https://api.github.com/repos/golang/go/labels/cla:%20yes","name":"cla: yes","color":"0e8a16","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":173,"created_at":"2018-09-26T08:49:00Z","updated_at":"2018-10-30T15:14:45Z","closed_at":null,"author_association":"NONE","pull_request":{"url":"https://api.github.com/repos/golang/go/pulls/27871","html_url":"https://github.com/golang/go/pull/27871","diff_url":"https://github.com/golang/go/pull/27871.diff","patch_url":"https://github.com/golang/go/pull/27871.patch"},"body":"On unix systems, long enough path names will fail when performing syscalls\r\nlike `Lstat`. The current RemoveAll uses several of these syscalls, and so\r\nwill fail for long paths. This can be risky, as it can let users \"hide\"\r\nfiles from the system or otherwise make long enough paths for programs\r\nto fail. By using `Unlinkat` and `Openat` syscalls instead, RemoveAll is\r\nsafer on unix systems. Initially implemented for linux, darwin, and several bsds.\r\n\r\nFixes #27029\r\n\r\nCo-authored-by: Giuseppe Capizzi <gcapizzi@pivotal.io>\r\nCo-authored-by: Julia Nedialkova <yulia.nedyalkova@sap.com>"},"comment":{"url":"https://api.github.com/repos/golang/go/issues/comments/434341653","html_url":"https://github.com/golang/go/pull/27871#issuecomment-434341653","issue_url":"https://api.github.com/repos/golang/go/issues/27871","id":434341653,"node_id":"MDEyOklzc3VlQ29tbWVudDQzNDM0MTY1Mw==","user":{"login":"gopherbot","id":8566911,"node_id":"MDQ6VXNlcjg1NjY5MTE=","avatar_url":"https://avatars1.githubusercontent.com/u/8566911?v=4","gravatar_id":"","url":"https://api.github.com/users/gopherbot","html_url":"https://github.com/gopherbot","followers_url":"https://api.github.com/users/gopherbot/followers","following_url":"https://api.github.com/users/gopherbot/following{/other_user}","gists_url":"https://api.github.com/users/gopherbot/gists{/gist_id}","starred_url":"https://api.github.com/users/gopherbot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gopherbot/subscriptions","organizations_url":"https://api.github.com/users/gopherbot/orgs","repos_url":"https://api.github.com/users/gopherbot/repos","events_url":"https://api.github.com/users/gopherbot/events{/privacy}","received_events_url":"https://api.github.com/users/gopherbot/received_events","type":"User","site_admin":false},"created_at":"2018-10-30T15:14:45Z","updated_at":"2018-10-30T15:14:45Z","author_association":"NONE","body":"Message from Gerrit User 5976:\n\nPatch Set 22:\n\nBuild is still in progress...\nThis change failed on freebsd-amd64-11_1:\nSee https://storage.googleapis.com/go-build-log/8853b41f/freebsd-amd64-11_1_4f1cf3bb.log\n\nConsult https://build.golang.org/ to see whether it's a new failure. Other builds still in progress; subsequent failure notices suppressed until final report.\n\n---\nPlease don’t reply on this GitHub thread. Visit [golang.org/cl/137442](https://go-review.googlesource.com/c/go/+/137442#message-4611ca4a8a32194c1235d04d426293cc1e339d6a).\nAfter addressing review feedback, remember to [publish your drafts](https://github.com/golang/go/wiki/GerritBot#i-left-a-reply-to-a-comment-in-gerrit-but-no-one-but-me-can-see-it)!"}}
{"actor":{"display_login":"gopherbot"}}