Skip to content
This repository has been archived by the owner on Nov 6, 2023. It is now read-only.

Update Cloudfront.xml #6905

Closed
wants to merge 2 commits into from
Closed

Update Cloudfront.xml #6905

wants to merge 2 commits into from

Conversation

ivysrono
Copy link
Contributor

No description provided.

@jeremyn
Copy link
Contributor

jeremyn commented Sep 18, 2016

It looks like http://aww.bugsplatsoftware.com/Blog/ makes calls to http://d22iv91mzdv9l4.cloudfront.net , which doesn't exist in DNS. Is the point of this pull request to fix their page for them by rewriting http://d22iv91mzdv9l4.cloudfront.net to https://www.bugsplat.com ? If so, that is not something we should be doing in HTTPS Everywhere.

@ivysrono
Copy link
Contributor Author

ivysrono commented Sep 19, 2016

I think it is http to https...
Why not?
Does #6889 same?

@jeremyn
Copy link
Contributor

jeremyn commented Sep 19, 2016

CDN rewrites are risky, see the discussion in issue #2709 . That issue is about non-CDN > CDN but the reverse, like in this pull request, is a problem too for a similar reason.

I don't completely understand #6889 but it looks like it is rewriting http://img02.iwgc.cn to https://mmbiz.qpic.cn , so if iwgc.cn is a CDN, then yes that has the same problem as this pull request does.

@ivysrono
Copy link
Contributor Author

No, iwgc.cn is not CDN.

@jeremyn
Copy link
Contributor

jeremyn commented Sep 19, 2016

In that case, what we're discussing here may not apply to pull request #6889.

@ivysrono
Copy link
Contributor Author

ivysrono commented Sep 20, 2016

All the *.bugsplatsoftware.com are showing:
Error establishing a database connection
Thank you, @jeremyn

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants