Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cnameAliasList set to * seems to block some content on websites #788

Closed
troysjanda opened this issue Nov 20, 2019 · 13 comments
Closed

cnameAliasList set to * seems to block some content on websites #788

troysjanda opened this issue Nov 20, 2019 · 13 comments
Labels
invalid not a uBlock issue

Comments

@troysjanda
Copy link

troysjanda commented Nov 20, 2019

Prerequisites

  • [x ] I verified that this is not a filter issue
  • [x ] This is not a support issue or a question
  • [ x] I performed a cursory search of the issue tracker to avoid opening a duplicate issue
    • Your issue may already be reported.
  • I tried to reproduce the issue when...
    • [x ] uBlock Origin is the only extension
    • [x ] uBlock Origin with default lists/settings
    • [x ] using a new, unmodified browser profile
  • [x ] I am running the latest version of uBlock Origin
  • [x ] I checked the documentation to understand that the issue I report is not a normal behavior

Description

Seem with the advance setting cnameAliasList set to * blocks some content on Sports.yahoo.com

Screen shots:
With cnameAliasList set to unset (default)
alt text
With cnameAliasList set to *
alt text

A specific URL where the issue occurs

sports.yahoo.com, news.yahoo.com only tested these

Steps to Reproduce

  1. set advance setting cnameAliasList to *
  2. Visit sports.yahoo.com
  3. [and so on...]

Expected behavior:

web page should display corrently

Actual behavior:

partial loading, navbar gone and most articles not showing and some images not displaying

Your environment

  • uBlock Origin version: latest dev
  • Browser Name and version: FF 70.0.1
  • Operating System and version: Windows 10 ENT, 1909 18363.476
@troysjanda troysjanda changed the title New Feature seems to block some element cnameAliasList set to * seems to block some content on websites Nov 20, 2019
@uBlock-user uBlock-user added the unable to reproduce cannot reproduce the issue label Nov 20, 2019
@troysjanda
Copy link
Author

I have this issue on 2 different computers not sure why its unable to reproduce

@uBlock-user
Copy link
Contributor

Open the logger, put s.yimg.com in the filter box and refresh sports.yahoo.com does it appear blocked by any filterlist or by you ?

That breakage only happens if s.yimg.com is blocked.

@troysjanda
Copy link
Author

troysjanda commented Nov 20, 2019

Its not block or listed as blocked except this single time

https://s.yimg.com/dh/ap/ad/advertisement/sponsor/promo/ad.png

@troysjanda
Copy link
Author

Fixed, when I set the setting to * there was a space before the * did not realize it. I removed the space and reloaded sports.yahoo.com and it loads correctly.

@uBlock-user uBlock-user added the invalid not a uBlock issue label Nov 20, 2019
@gorhill gorhill added filterlist a filter list issue and removed unable to reproduce cannot reproduce the issue labels Nov 20, 2019
@gorhill
Copy link
Member

gorhill commented Nov 20, 2019

Do not open filter issue here please, especially filter issues arising as a result of you altering uBO's filtering behavior by modifying advanced user settings. Complaining that uBO's filtering behavior is modified by setting cnameAliasList to * is equivalent to complaining that uBO's filtering behavior is modified by adding a custom dynamic filtering rule: you are entirely responsible for the result.

@gorhill gorhill removed the filterlist a filter list issue label Nov 20, 2019
@gwarser
Copy link
Member

gwarser commented Nov 20, 2019

+63 -share-buttons.$script -- sports.yahoo.com 3 script https://edge.gycpi.b.yahoodns.net/xe/tdv2-sports-site/js/react-share-buttons.e081619b08a957ecf8ee.min.js cnameOf=s.yimg.com
+63 @@||yimg.com^*/react-share-buttons.$script,domain=yahoo.com ++ sports.yahoo.com 3 script https://s.yimg.com/xe/tdv2-sports-site/js/react-share-buttons.e081619b08a957ecf8ee.min.js

-share-buttons is whitelisted when comming from yimg.com domain, but with CNAME it comes from yahoodns.net

@troysjanda
Copy link
Author

Do not open filter issue here please, especially filter issues arising as a result of you altering uBO's filtering behavior by modifying advanced user settings. Complaining that uBO's filtering behavior is modified by setting cnameAliasList to * is equivalent to complaining that uBO's filtering behavior is modified by adding a custom dynamic filtering rule: you are entirely responsible for the result.

Mr Gorhill,

I was careful in my initial testing and had disabled all filters except the Ubo list section and the result was the same with that setting set to * some of the content would not appear. This is the reason I filed the issue. I later saw my error, when I set the advance setting from unset to * there was a space before the * when I removed that space the issue went away. later when ublock-user tagged with invalid he had me test a filter in the logger--s.yimg.com-- and the results were it was not blocked as this time I had already restore all my filterlists. s.yimg.com was not blocked but I had a single filter that was similar s.yimg.com/dh/ap/ad/advertisement/sponsor/promo/ad.png. I hope that clear it up a bit.

Thanks

@troysjanda
Copy link
Author

when I set the advance setting from unset to * there was a space before the * when I removed that space the issue went away

Completely unrelated, uBO trims spaces before processing the value, here is the code that does this.

thank you this was my experience with this issue I can't explain other than what I have in the future ill not open issues. please have a nice day and god bless.

@gorhill
Copy link
Member

gorhill commented Nov 20, 2019

in the future ill not open issues

I am merely asking for due diligence when opening issues, especially when it involves advanced user features, and especially when the primary purpose of the advanced user feature being modified is to modify the filtering behavior of uBO -- it is to be expected that as a result more or less stuff may end up being blocked. The "new issue" template says:

See what the logger https://github.com/gorhill/uBlock/wiki/The-logger reports when you reproduce the issue, this will help you determine whether this is a filter issue.

When an issue is about undue blocking, I do expect the first step to be investigation using the logger, and this was nowhere to be found in your report to make the case uBO was blocking stuff it shouldn't have blocked.

As for cnameAliasList, a space before * is not going to make a difference. However in investigating this specific claim, I found that a space after may affect the result, but this would be the equivalent of using unset, i.e. to not enable CNAME lookup, which is currently the default behavior.

@uBlock-user
Copy link
Contributor

That setting cannot block anything, all it does is pull the CNAME data for all the hostnames and only shows those ones who don't match; in the popup panel and the logger, thats it. There's no blocking involved in this functionality.

@gorhill
Copy link
Member

gorhill commented Nov 21, 2019

and only shows those ones who don't match

It does not only show, it replays them through the filtering engine as any other network requests. Those shown in the logger are those replayed through uBO's filtering engine.

There is blocking involved when a replayed URL matches a filter/rule, which is the case for eulerian.net (good), or as I have observed gstaticadssl.l.google.com (CNAME for fonts.gstatic.com) which is a false positive.

Vast majority of aliased hostnames through CNAME are for CDN purpose, hence why this can be enabled through an advanced settings only.

@uBlock-user
Copy link
Contributor

There is blocking involved when a replayed URL matches a filter/rule, which is the case for eulerian.net (good),

Yes but this setting is not directly responsible for blocking, thats my point.

@troysjanda
Copy link
Author

I do expect the first step to be investigation using the logger, and this was nowhere to be found in your report to make the case uBO was blocking stuff it shouldn't have blocked.

I did use the logger and reported that I disabled all filters except Ubo section and the issue persisted. The only thing that I did to restore the missing elements was reset the cnameAliasList to unset and refreshed the page and it was normal.

This would have made since that the trailing space in fact made cnameAliasList disabled (default) thus making sports.yahoo.com display correctly. I purposefully tested this with all lists disabled except Ubo section lists. This led me to believe that it was not a FILTER issue.

Maybe staring/testing code all day at work fried my brain. The conclusion that I made were, it was not a FILTER issue, since all filters were disabled. Seemed to be caused by enabling cnameAliasList, and restored once reset to default.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
invalid not a uBlock issue
Projects
None yet
Development

No branches or pull requests

4 participants