Skip to content
This repository has been archived by the owner on Oct 5, 2022. It is now read-only.

Mixed Content Warnings #3

Open
chilts opened this issue May 22, 2017 · 0 comments
Open

Mixed Content Warnings #3

chilts opened this issue May 22, 2017 · 0 comments

Comments

@chilts
Copy link

chilts commented May 22, 2017

The front page is showing the icons from the example feeds, however some of them are http and therefore make the browser not like your site.

I don't think this is a major, however I just thought I'd point it out in case you want to try something different. Perhaps a https proxy somewhere (much like GitHub do with using camo) but perhaps that's too much for what you want to do.

Or perhaps add a /image proxy passing the url of the feed's icon and then you can serve that from https://json-feed-viewer.herokuapp.com/ too - but perhaps that's a bad idea without proper consideration being taken for content type and content itself, which camo (above) already does I think. I haven't dug into that project enough apart from know it's there.

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

No branches or pull requests

2 participants