Skip to content

Loading…

Removed images show broken image images links #174

Closed
njorogle opened this Issue · 10 comments

2 participants

@njorogle

Screenshot from a kickstarter email in gmail.

uBlock: ublock

ABP: abp

Nothing: nothing

@gorhill

Not sure how, best is if I could reproduce on my computer here so that I can investigate. It is rather unexpected, there is code in there to collapse blocked elements (if the setting is enabled -- which it is by default).

What filter lists do you have enabled in uBlock?
What filter lists do you have enabled in ABP?

@njorogle

I use the Fanboy's Social Blocking List on both. I can email you the broken HML if you like - I don't know how else to reproduce.

Edit: looks like ABP is reloading the page to fix the blocked element in a second pass, but it's briefly broken for ABP too.

@gorhill

Can you forward me the problematic email (if there is no privacy issue) in my throwaway Gmail account? Here: uKOeoIfHQDp8@gmail.com.

@njorogle

Done

@gorhill

Thanks, I can reproduce it. Now onto finding out why this happens.

@gorhill

Alright, found problem, fix a bit more complicated than thought, related to fragment part of the URL of net requests. Need to think of the best long term solution.

@njorogle

Many thanks. Nice plugin.

@njorogle njorogle closed this
@gorhill gorhill added a commit that referenced this issue
@gorhill gorhill this fixes #174 8be4e40
@gorhill

Fix available in 0.5.1.0 beta 2 for now: https://github.com/gorhill/uBlock/releases/tag/0.5.1.0-beta.2.

Will publish into stores when I am satisfied all work as expected.

@gorhill gorhill removed the need feedback label
@gorhill

So about the fix and for the record: after a couple of brain farts and a good night sleep, problem is simply that I was removing the URL fragment from the URL of the src property before checking if it was a blocked request. I can't remember why I thought it was a good idea.

@gorhill gorhill added a commit that referenced this issue
@gorhill gorhill this fixes #174 f7c6164
@gorhill

Oops, I meant to type "#374", not "#174".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.