{"action":"created","issue":{"url":"https://api.github.com/repos/CartoDB/cartodb/issues/6905","repository_url":"https://api.github.com/repos/CartoDB/cartodb","labels_url":"https://api.github.com/repos/CartoDB/cartodb/issues/6905/labels{/name}","comments_url":"https://api.github.com/repos/CartoDB/cartodb/issues/6905/comments","events_url":"https://api.github.com/repos/CartoDB/cartodb/issues/6905/events","html_url":"https://github.com/CartoDB/cartodb/pull/6905","id":141452892,"number":6905,"title":"Regex change /^http:/ to 'https:' in user avatar url","user":{"login":"whokilledtheelectricmonk","id":1571667,"avatar_url":"https://avatars.githubusercontent.com/u/1571667?v=3","gravatar_id":"","url":"https://api.github.com/users/whokilledtheelectricmonk","html_url":"https://github.com/whokilledtheelectricmonk","followers_url":"https://api.github.com/users/whokilledtheelectricmonk/followers","following_url":"https://api.github.com/users/whokilledtheelectricmonk/following{/other_user}","gists_url":"https://api.github.com/users/whokilledtheelectricmonk/gists{/gist_id}","starred_url":"https://api.github.com/users/whokilledtheelectricmonk/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/whokilledtheelectricmonk/subscriptions","organizations_url":"https://api.github.com/users/whokilledtheelectricmonk/orgs","repos_url":"https://api.github.com/users/whokilledtheelectricmonk/repos","events_url":"https://api.github.com/users/whokilledtheelectricmonk/events{/privacy}","received_events_url":"https://api.github.com/users/whokilledtheelectricmonk/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"javitonino","id":7569673,"avatar_url":"https://avatars.githubusercontent.com/u/7569673?v=3","gravatar_id":"","url":"https://api.github.com/users/javitonino","html_url":"https://github.com/javitonino","followers_url":"https://api.github.com/users/javitonino/followers","following_url":"https://api.github.com/users/javitonino/following{/other_user}","gists_url":"https://api.github.com/users/javitonino/gists{/gist_id}","starred_url":"https://api.github.com/users/javitonino/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/javitonino/subscriptions","organizations_url":"https://api.github.com/users/javitonino/orgs","repos_url":"https://api.github.com/users/javitonino/repos","events_url":"https://api.github.com/users/javitonino/events{/privacy}","received_events_url":"https://api.github.com/users/javitonino/received_events","type":"User","site_admin":false},"milestone":null,"comments":5,"created_at":"2016-03-17T01:46:25Z","updated_at":"2016-04-05T16:43:30Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/CartoDB/cartodb/pulls/6905","html_url":"https://github.com/CartoDB/cartodb/pull/6905","diff_url":"https://github.com/CartoDB/cartodb/pull/6905.diff","patch_url":"https://github.com/CartoDB/cartodb/pull/6905.patch"},"body":"- User avatars are being served over HTTP even if the map embed is HTTPS, causing mixed content warnings on sites with SSL (#6904)\r\n- Assets on S3 should always have a HTTPS equivalent, so this should not break anything"},"comment":{"url":"https://api.github.com/repos/CartoDB/cartodb/issues/comments/205888586","html_url":"https://github.com/CartoDB/cartodb/pull/6905#issuecomment-205888586","issue_url":"https://api.github.com/repos/CartoDB/cartodb/issues/6905","id":205888586,"user":{"login":"whokilledtheelectricmonk","id":1571667,"avatar_url":"https://avatars.githubusercontent.com/u/1571667?v=3","gravatar_id":"","url":"https://api.github.com/users/whokilledtheelectricmonk","html_url":"https://github.com/whokilledtheelectricmonk","followers_url":"https://api.github.com/users/whokilledtheelectricmonk/followers","following_url":"https://api.github.com/users/whokilledtheelectricmonk/following{/other_user}","gists_url":"https://api.github.com/users/whokilledtheelectricmonk/gists{/gist_id}","starred_url":"https://api.github.com/users/whokilledtheelectricmonk/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/whokilledtheelectricmonk/subscriptions","organizations_url":"https://api.github.com/users/whokilledtheelectricmonk/orgs","repos_url":"https://api.github.com/users/whokilledtheelectricmonk/repos","events_url":"https://api.github.com/users/whokilledtheelectricmonk/events{/privacy}","received_events_url":"https://api.github.com/users/whokilledtheelectricmonk/received_events","type":"User","site_admin":false},"created_at":"2016-04-05T16:43:29Z","updated_at":"2016-04-05T16:43:29Z","body":"So some avatars are hosted off-site? What about importing them to your S3\nbucket?\n\nOn Tue, Apr 5, 2016, 8:29 AM Javier Torres <notifications@github.com> wrote:\n\n> Hi @whokilledtheelectricmonk <https://github.com/whokilledtheelectricmonk>\n> and thanks for the patch.\n>\n> Unfortunately, there is a problem with unconditionally replacing http\n> with https. Some of our older users have custom URLs for their avatars,\n> and changing the URL for them may break things for them.\n>\n> I think it would be better to solve this at creation time, so new avatars\n> will always use https, and users could simply fix this for any maps they\n> own by reuploading their avatars (or we could run a task to update the\n> database).\n>\n> —\n> You are receiving this because you were mentioned.\n> Reply to this email directly or view it on GitHub\n> <https://github.com/CartoDB/cartodb/pull/6905#issuecomment-205858635>\n>\n"}}