{"action":"created","issue":{"url":"https://api.github.com/repos/yutakahirano/fetch-with-streams/issues/66","repository_url":"https://api.github.com/repos/yutakahirano/fetch-with-streams","labels_url":"https://api.github.com/repos/yutakahirano/fetch-with-streams/issues/66/labels{/name}","comments_url":"https://api.github.com/repos/yutakahirano/fetch-with-streams/issues/66/comments","events_url":"https://api.github.com/repos/yutakahirano/fetch-with-streams/issues/66/events","html_url":"https://github.com/yutakahirano/fetch-with-streams/issues/66","id":168812751,"number":66,"title":"Uploading: how to handle redirects","user":{"login":"domenic","id":617481,"avatar_url":"https://avatars.githubusercontent.com/u/617481?v=3","gravatar_id":"","url":"https://api.github.com/users/domenic","html_url":"https://github.com/domenic","followers_url":"https://api.github.com/users/domenic/followers","following_url":"https://api.github.com/users/domenic/following{/other_user}","gists_url":"https://api.github.com/users/domenic/gists{/gist_id}","starred_url":"https://api.github.com/users/domenic/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/domenic/subscriptions","organizations_url":"https://api.github.com/users/domenic/orgs","repos_url":"https://api.github.com/users/domenic/repos","events_url":"https://api.github.com/users/domenic/events{/privacy}","received_events_url":"https://api.github.com/users/domenic/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":9,"created_at":"2016-08-02T06:56:51Z","updated_at":"2016-08-19T16:21:27Z","closed_at":null,"body":"I see two options:\r\n\r\n- Do as the spec does now, and tee the stream so we can handle redirects. A bit complicated...\r\n\r\n- Only allow uploads if the redirect mode is \"error\".\r\n  - Automatically set it to \"error\" if they supply a stream, or require them to opt-in?\r\n  - Should we allow \"manual\" also?\r\n\r\nIn practice I think disallowing upload redirects is not that bad. Redirects for uploads seems pretty weird to me... We could also do that as an initial version and then if there is developer need, loosen the restriction later?"},"comment":{"url":"https://api.github.com/repos/yutakahirano/fetch-with-streams/issues/comments/241064206","html_url":"https://github.com/yutakahirano/fetch-with-streams/issues/66#issuecomment-241064206","issue_url":"https://api.github.com/repos/yutakahirano/fetch-with-streams/issues/66","id":241064206,"user":{"login":"domenic","id":617481,"avatar_url":"https://avatars.githubusercontent.com/u/617481?v=3","gravatar_id":"","url":"https://api.github.com/users/domenic","html_url":"https://github.com/domenic","followers_url":"https://api.github.com/users/domenic/followers","following_url":"https://api.github.com/users/domenic/following{/other_user}","gists_url":"https://api.github.com/users/domenic/gists{/gist_id}","starred_url":"https://api.github.com/users/domenic/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/domenic/subscriptions","organizations_url":"https://api.github.com/users/domenic/orgs","repos_url":"https://api.github.com/users/domenic/repos","events_url":"https://api.github.com/users/domenic/events{/privacy}","received_events_url":"https://api.github.com/users/domenic/received_events","type":"User","site_admin":false},"created_at":"2016-08-19T16:21:27Z","updated_at":"2016-08-19T16:21:27Z","body":"I think I would prefer to tee all the things, or disallow authentication. Causing a roundtrip tax on all from-JavaScript uploads just because a small minority of cases might need authentication seems like a bad tradeoff."}}
{"actor":{"display_login":"domenic"}}