{"action":"created","issue":{"url":"https://api.github.com/repos/tusharmath/Multi-threaded-downloader/issues/70","repository_url":"https://api.github.com/repos/tusharmath/Multi-threaded-downloader","labels_url":"https://api.github.com/repos/tusharmath/Multi-threaded-downloader/issues/70/labels{/name}","comments_url":"https://api.github.com/repos/tusharmath/Multi-threaded-downloader/issues/70/comments","events_url":"https://api.github.com/repos/tusharmath/Multi-threaded-downloader/issues/70/events","html_url":"https://github.com/tusharmath/Multi-threaded-downloader/pull/70","id":142817811,"number":70,"title":"fix(observables): get content-length from GET not HEAD","user":{"login":"continuata","id":1593397,"avatar_url":"https://avatars.githubusercontent.com/u/1593397?v=3","gravatar_id":"","url":"https://api.github.com/users/continuata","html_url":"https://github.com/continuata","followers_url":"https://api.github.com/users/continuata/followers","following_url":"https://api.github.com/users/continuata/following{/other_user}","gists_url":"https://api.github.com/users/continuata/gists{/gist_id}","starred_url":"https://api.github.com/users/continuata/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/continuata/subscriptions","organizations_url":"https://api.github.com/users/continuata/orgs","repos_url":"https://api.github.com/users/continuata/repos","events_url":"https://api.github.com/users/continuata/events{/privacy}","received_events_url":"https://api.github.com/users/continuata/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2016-03-23T00:16:29Z","updated_at":"2016-03-23T00:18:05Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/tusharmath/Multi-threaded-downloader/pulls/70","html_url":"https://github.com/tusharmath/Multi-threaded-downloader/pull/70","diff_url":"https://github.com/tusharmath/Multi-threaded-downloader/pull/70.diff","patch_url":"https://github.com/tusharmath/Multi-threaded-downloader/pull/70.patch"},"body":"- signed URLs from AWS S3 don’t give the content-length in a HEAD request\r\n- this fix uses a standard GET to get the content-length and destroys the request before proceeding to download data, which is practically equivalent to calling HEAD.\r\n\r\nI know you weren't keen on doing it this way ... but it does seem reasonably sensible, considering that an aborted GET is effectively the same as a HEAD request ... plus ... it works ;-)\r\n\r\nI was able to download a signed AWS URL in multi-parts with this fix"},"comment":{"url":"https://api.github.com/repos/tusharmath/Multi-threaded-downloader/issues/comments/200090826","html_url":"https://github.com/tusharmath/Multi-threaded-downloader/pull/70#issuecomment-200090826","issue_url":"https://api.github.com/repos/tusharmath/Multi-threaded-downloader/issues/70","id":200090826,"user":{"login":"continuata","id":1593397,"avatar_url":"https://avatars.githubusercontent.com/u/1593397?v=3","gravatar_id":"","url":"https://api.github.com/users/continuata","html_url":"https://github.com/continuata","followers_url":"https://api.github.com/users/continuata/followers","following_url":"https://api.github.com/users/continuata/following{/other_user}","gists_url":"https://api.github.com/users/continuata/gists{/gist_id}","starred_url":"https://api.github.com/users/continuata/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/continuata/subscriptions","organizations_url":"https://api.github.com/users/continuata/orgs","repos_url":"https://api.github.com/users/continuata/repos","events_url":"https://api.github.com/users/continuata/events{/privacy}","received_events_url":"https://api.github.com/users/continuata/received_events","type":"User","site_admin":false},"created_at":"2016-03-23T00:18:05Z","updated_at":"2016-03-23T00:18:05Z","body":"from the looks of things, I may have to mock out the destroy function in the integration tests.\r\nI'll take a look at this tomorrow."}}