MbinOrg / mbin

Mbin: a federated content aggregator, voting, discussion and microblogging platform
https://joinmbin.org
GNU Affero General Public License v3.0
241 stars 17 forks source link

Further Improve Handling of huge remote images #511

Open BentiGorlich opened 9 months ago

BentiGorlich commented 9 months ago

Follow-up issue for #501

Main problem after #501: if there is an entry without a local image (because it was too large) we do not have a thumb_image of that.

Solution: compress the image and safe it afterwards. We add a new column is_compressed, so we can display the original full res image from the source url when it is needed and we know that we don't have the original full res image at hand

melroy89 commented 9 months ago

Main problem after #501: If we display a remote image (from another domain), because there is no local image, this can cause issues with the server CORS and CSP HTTP Headers. Thus now allowing to load content from other domains (due to security concerns). It depends on the server settings tho.