Sketchy502 / SDV-Summary

An application to display a summary of the player from a Stardew Valley save file.
206 stars 21 forks source link

'Not secure' warnings on homepage due to mixed content warnings #57

Closed NickColley closed 4 years ago

NickColley commented 4 years ago

The homepage says 'not secure' since there is an image loaded over HTTP.

Mixed Content: The page at 'https://upload.farm/' was loaded over HTTPS, but requested an insecure image 'http://upload.farm/static/uploader/uploader_v2_thm.png'. This content should also be served over HTTPS.

I'd be interested in fixing this if you're accepting pull requests, let me know.

Laukei commented 4 years ago

Hi Nick, apologies for the delayed reply. We're definitely open for pull requests!

NickColley commented 4 years ago

Hey @Laukei,

I've not been able to run the project locally but it looks to me that this content is within the database so would need admin access to the live site, perhaps you can solve it?

I would be interested in contributing other accessibility related fixes but unfortunately I've found it too difficult to run this project.

Laukei commented 4 years ago

Yeah, it's a bit of a nightmare to run it, we didn't design it well - we were but fledgling developers when we started the project. I have a feeling I can solve it with a few config tweaks, I'll have a look when I have some free time! Thanks for trying anyway!