image proxying/caching only seems to work for thumbnails?

[Die4Ever]

Requirements

• This is a feature request and not a bug report. Otherwise, please create a new bug report instead.
• Please check to see if this request (or a similar one) already exists.
• It's a single feature. Please don't request multiple features in one issue.

Describe the feature you'd like

I thought this option would do it for the full images too, maybe that's not the intention so I'm filing this as a feature request.

example: https://lemmy.ml/post/27523111

I realized this because I was trying to help a frustrated user: https://old.reddit.com/r/Lemmy/comments/1jhqk58/is_there_a_way_to_browse_lemmy_without_requesting/

[dessalines]

@Nutomic I looked through crates/apub/src/objects/post.rs, but I don't see any calls to proxy_image_link in the from_json, like we do for local image creation.

[Nutomic]

Right this seems to be missing.

[Die4Ever]

I also see unproxied images in PMs

images in PMs should maybe be hidden by default for security? proxying might not be good enough security in the case of PMs

https://retrolemmy.com/comment/11825421

[dessalines]

That's a separate, UI related issue.

[Nutomic]

@Die4Ever I checked the code and it definitely does proxying for those. Are you sure the setting is enabled on your instance?