{"action":"created","issue":{"url":"https://api.github.com/repos/ansible/ansible-modules-core/issues/1237","labels_url":"https://api.github.com/repos/ansible/ansible-modules-core/issues/1237/labels{/name}","comments_url":"https://api.github.com/repos/ansible/ansible-modules-core/issues/1237/comments","events_url":"https://api.github.com/repos/ansible/ansible-modules-core/issues/1237/events","html_url":"https://github.com/ansible/ansible-modules-core/pull/1237","id":72405083,"number":1237,"title":"rackspace: set headers when uploading files to CF.","user":{"login":"vmihailenco","id":290976,"avatar_url":"https://avatars.githubusercontent.com/u/290976?v=3","gravatar_id":"","url":"https://api.github.com/users/vmihailenco","html_url":"https://github.com/vmihailenco","followers_url":"https://api.github.com/users/vmihailenco/followers","following_url":"https://api.github.com/users/vmihailenco/following{/other_user}","gists_url":"https://api.github.com/users/vmihailenco/gists{/gist_id}","starred_url":"https://api.github.com/users/vmihailenco/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/vmihailenco/subscriptions","organizations_url":"https://api.github.com/users/vmihailenco/orgs","repos_url":"https://api.github.com/users/vmihailenco/repos","events_url":"https://api.github.com/users/vmihailenco/events{/privacy}","received_events_url":"https://api.github.com/users/vmihailenco/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/ansible/ansible-modules-core/labels/bugfix_pull_request","name":"bugfix_pull_request","color":"ded270"},{"url":"https://api.github.com/repos/ansible/ansible-modules-core/labels/cloud","name":"cloud","color":"c7def8"},{"url":"https://api.github.com/repos/ansible/ansible-modules-core/labels/community_review","name":"community_review","color":"fad8c7"},{"url":"https://api.github.com/repos/ansible/ansible-modules-core/labels/needs_rebase","name":"needs_rebase","color":"eb6420"},{"url":"https://api.github.com/repos/ansible/ansible-modules-core/labels/P3","name":"P3","color":"fef2c0"}],"state":"open","locked":false,"assignee":{"login":"sivel","id":39340,"avatar_url":"https://avatars.githubusercontent.com/u/39340?v=3","gravatar_id":"","url":"https://api.github.com/users/sivel","html_url":"https://github.com/sivel","followers_url":"https://api.github.com/users/sivel/followers","following_url":"https://api.github.com/users/sivel/following{/other_user}","gists_url":"https://api.github.com/users/sivel/gists{/gist_id}","starred_url":"https://api.github.com/users/sivel/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sivel/subscriptions","organizations_url":"https://api.github.com/users/sivel/orgs","repos_url":"https://api.github.com/users/sivel/repos","events_url":"https://api.github.com/users/sivel/events{/privacy}","received_events_url":"https://api.github.com/users/sivel/received_events","type":"User","site_admin":false},"milestone":null,"comments":15,"created_at":"2015-05-01T10:43:32Z","updated_at":"2015-12-08T19:43:56Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/ansible/ansible-modules-core/pulls/1237","html_url":"https://github.com/ansible/ansible-modules-core/pull/1237","diff_url":"https://github.com/ansible/ansible-modules-core/pull/1237.diff","patch_url":"https://github.com/ansible/ansible-modules-core/pull/1237.patch"},"body":"Otherwise CDN (Akamai) downloads file without the headers. The sequence\r\nis following:\r\n1. Ansible uploads file to CF.\r\n2. Akamai downloads the file and caches it in CDN.\r\n3. Ansible sets headers.\r\n\r\nAs a result Akamai serves file without headers.\r\n\r\nThis is backwards incompatible change, because headers keys are not\r\nprefixed with `x-object-meta-`. Which allows user to set headers like\r\n`Access-Control-Allow-Origin`."},"comment":{"url":"https://api.github.com/repos/ansible/ansible-modules-core/issues/comments/162993501","html_url":"https://github.com/ansible/ansible-modules-core/pull/1237#issuecomment-162993501","issue_url":"https://api.github.com/repos/ansible/ansible-modules-core/issues/1237","id":162993501,"user":{"login":"phumpal","id":654902,"avatar_url":"https://avatars.githubusercontent.com/u/654902?v=3","gravatar_id":"","url":"https://api.github.com/users/phumpal","html_url":"https://github.com/phumpal","followers_url":"https://api.github.com/users/phumpal/followers","following_url":"https://api.github.com/users/phumpal/following{/other_user}","gists_url":"https://api.github.com/users/phumpal/gists{/gist_id}","starred_url":"https://api.github.com/users/phumpal/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/phumpal/subscriptions","organizations_url":"https://api.github.com/users/phumpal/orgs","repos_url":"https://api.github.com/users/phumpal/repos","events_url":"https://api.github.com/users/phumpal/events{/privacy}","received_events_url":"https://api.github.com/users/phumpal/received_events","type":"User","site_admin":false},"created_at":"2015-12-08T19:43:56Z","updated_at":"2015-12-08T19:43:56Z","body":"@gregdek - as @vmihailenco has mentioned we migrated off CF to S3 so we don't have a reliable means of testing. \r\n\r\nHowever, I think it would be beneficial if this were patched for other users. \r\n\r\nWhat else is needed to complete the review process except the `\"ready_for_review\"` text?"}}