Bug 4424 - Fix disabling remote content loading
Summary: Fix disabling remote content loading
Status: RESOLVED FIXED
Alias: None
Product: Claws Mail
Classification: Unclassified
Component: Plugins/Fancy (show other bugs)
Version: 4.0.0
Hardware: PC Linux
: P3 normal
Assignee: users
URL:
Depends on:
Blocks:
 
Reported: 2020-12-17 02:50 UTC by Orivej Desh
Modified: 2020-12-23 13:32 UTC (History)
1 user (show)

See Also:


Attachments
Disable remote content by using a void proxy (2.36 KB, patch)
2020-12-17 02:50 UTC, Orivej Desh
Details | Diff
Disable remote content by using a void proxy (2.90 KB, patch)
2020-12-17 03:55 UTC, Orivej Desh
Details | Diff
screendump of preference (48.39 KB, image/png)
2020-12-18 01:43 UTC, Michael Rasmussen
Details

Description Orivej Desh 2020-12-17 02:50:31 UTC
Created attachment 2157 [details]
Disable remote content by using a void proxy

# Patch description

Disable remote content by using a void proxy

enable-dns-prefetching=0 has stopped blocking remote content after the
switch to webkit2gtk. Disabling it with a false proxy works.

Setting proxy to "" works well with libsoup-backed webkit2gtk since it
prevents any connection attempts. If libcurl-backed webkit2gtk won't
like it, it may be replaced by "http://127.0.0.9:9" (with 9 being
reserved for the discard service, and 127.0.0.9 being more greppable
than 127.0.0.1).

# Patch license

https://claws-mail.org/COPYING
Comment 1 Michael Rasmussen 2020-12-17 03:22:55 UTC
The patch does not apply using git am. Have you made the patch using git format-patch?
Comment 2 Michael Rasmussen 2020-12-17 03:36:40 UTC
Using git apply works. git format-patch preferred since then commit message is added automatically.
Comment 4 Orivej Desh 2020-12-17 03:55:55 UTC
Created attachment 2159 [details]
Disable remote content by using a void proxy

Could you update the patch guidelines at https://claws-mail.org/devel.php ?
I had to edit the output of git format-patch to make it apply with -p0.
Comment 5 Orivej Desh 2020-12-17 03:56:54 UTC
Yes, please forward it to me.
Comment 6 Orivej Desh 2020-12-17 04:04:47 UTC
Could you check if changing the first argument of webkit_network_proxy_settings_new from "" to "http://127.0.0.9:9" helps?
Comment 7 Orivej Desh 2020-12-17 04:16:03 UTC
Another possibility is that the images were cached locally. Please disable remote content by default (if you've enabled it before), close claws-mail, delete ~/.cache/claws-mail/WebKitCache, then open the email with images.
Comment 8 Michael Rasmussen 2020-12-18 01:28:06 UTC
I have forwarded the email in question to you as an attachment.
Comment 9 Michael Rasmussen 2020-12-18 01:42:47 UTC
(In reply to Orivej Desh from comment #7)
I deleted ~/.cache/claws-mail/WebKitCache and of course had disabled remote contents. I have also tried changing the proxy url as suggested, including disabling embedded javascript and images but to no avail. I will add a screendump of my settings.

Cache content:
ll ~/.cache/claws-mail/WebKitCache/Version\ 16/Blobs/
total 0
Comment 10 Michael Rasmussen 2020-12-18 01:43:38 UTC
Created attachment 2161 [details]
screendump of preference
Comment 11 Paul 2020-12-23 13:13:42 UTC
it works for me, albeit with some warnings thrown.
Comment 12 Paul 2020-12-23 13:32:06 UTC
pushed to git. thanks!

Note You need to log in before you can comment on or make changes to this bug.