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

Share link not visible because of ad blockers #10434

Closed
TecJon opened this issue Jul 28, 2018 · 27 comments · Fixed by #11844
Closed

Share link not visible because of ad blockers #10434

TecJon opened this issue Jul 28, 2018 · 27 comments · Fixed by #11844
Labels
1. to develop Accepted and waiting to be taken care of bug feature: sharing help wanted
Milestone

Comments

@TecJon
Copy link

TecJon commented Jul 28, 2018

Steps to reproduce

  1. Click on share symbol for file or folder
  2. Check out side panel for sharing
  3. Miss share link option

Expected behaviour

I expect an option to share a link.

Actual behaviour

Only input for user sharing is visible
capture

Server configuration

Nextcloud version: 14.0.0 Beta 1

Updated from an older Nextcloud/ownCloud or fresh install: fresh

Where did you install Nextcloud from:

Signing status:

Signing status
No errors have been found.

Are you using external storage, if yes which one: no

Are you using encryption: no

Are you using an external user-backend, if yes which one: no

Client configuration

Browser: Firefox 61.0.1 (64-bit)

Operating system: Win 10

Logs

Browser log

Browser log
Content Security Policy: Directive ‘child-src’ has been deprecated. Please use directive ‘worker-src’ to control workers, or directive ‘frame-src’ to control frames respectively.
JQMIGRATE: Migrate is installed, version 1.4.0 core.js:7:542
Content Security Policy: The page’s settings blocked the loading of a resource at https://cloud.example.com/index.php/apps/files/ (“script-src”).
<script> source URI is not allowed in this document: “https://cloud.example.com/apps/notifications/js/notifications.js?v=1f8ad6d7-0”. files:51
unreachable code after return statement[Learn More] merged.js:1494:4
window.controllers/Controllers is deprecated. Do not use it for UA detection. merged.js:2171
New search handler registered search.js:66:4
querying Whats New data was successful: nocontent merged-template-prepend.js:4455:4
undefined  merged-template-prepend.js:4456:4
@rullzer
Copy link
Member

rullzer commented Jul 30, 2018

@nextcloud/javascript seems like some glitch...

@skjnldsv
Copy link
Member

Wut!

@skjnldsv skjnldsv self-assigned this Jul 30, 2018
@skjnldsv
Copy link
Member

I cannot reproduce:
capture d ecran_2018-07-30_12-57-56

Browser: Firefox 61.0.1 (64-bit)

@TecJon can you clear your caches?

@TecJon
Copy link
Author

TecJon commented Jul 30, 2018

@skjnldsv uBlock Origin was evildoer. When enabled it will not show up (even though reportedly 0% of the page is blocked). Might still be worth looking into?

@skjnldsv
Copy link
Member

@TecJon What is the filter blocking this? A custom one?
Because I also have ublock origin (default settings) and it still shows :)

@TecJon
Copy link
Author

TecJon commented Jul 30, 2018

It seems like the option "Parse and enforce cosmetic filters" is the reason. I'm guessing this filter is not enabled in your environment, @skjnldsv ?

This option enables the parsing and enforcing of Adblock Plus-compatible “element hiding” filters. These filters are essentially cosmetic, they serve to hide elements in a web page which are deemed to be a visual nuisance, and which can't be blocked by the net request-based filtering engine.

I guess that is why it says 0% is blocked. It's not a net request-based filtering.

@skjnldsv
Copy link
Member

I do not have this option available, where do you find this?

@TecJon
Copy link
Author

TecJon commented Jul 30, 2018

I find it in the filter lists tab of the ublock origin settings.
See this screenshot from their github page. Mine looks slightly different and the tabs name is "Filter lists" instead of "3rd-party filters" but the location is the same.
I know I added some filters which were not selected by default. I do not use any custom filters.

filter

@skjnldsv
Copy link
Member

Nope, I can't reproduce. All of my settings are already enabled and the parse and enforce cosmetic filters too. :/

@nextcloud/designers can one of you reproduce?

@blizzz
Copy link
Member

blizzz commented Jul 31, 2018

Cannot reproduce either with uBock origin enabled.

@skjnldsv
Copy link
Member

Sorry @TecJon
Thanks a lot for reporting, let's close this one and see if people complains as well (you're the only one so far)
Let's hope this is something unrelated :)

@TecJon
Copy link
Author

TecJon commented Jul 31, 2018

Will try to find out which filter it was. Probably some odd combination.

@tplusm
Copy link

tplusm commented Oct 1, 2018

Running into the same issue with Adguard. When enabled, the 3 dots (...) to copy public link do not show. Turn Adguard off, 3 dots appear, clicking on it shows an option to copy public link.
Using basic filters on Adgaurd.

@TecJon TecJon reopened this Oct 1, 2018
@skjnldsv
Copy link
Member

skjnldsv commented Oct 1, 2018

Dammit 😁
Can someone help us and narrow the exact filter/rule that make this happen? So we can change our ui to work around this :)

@skjnldsv skjnldsv added help wanted 1. to develop Accepted and waiting to be taken care of bug labels Oct 1, 2018
@MorrisJobke
Copy link
Member

Dammit 😁
Can someone help us and narrow the exact filter/rule that make this happen? So we can change our ui to work around this :)

There you go: #11450 (comment)

@skjnldsv
Copy link
Member

skjnldsv commented Oct 1, 2018

Duplicate of #11450

Oups, it's the other way around 😁

@skjnldsv skjnldsv marked this as a duplicate of #11450 Oct 1, 2018
@skjnldsv skjnldsv closed this as completed Oct 1, 2018
@skjnldsv skjnldsv reopened this Oct 1, 2018
@skjnldsv skjnldsv marked this as not a duplicate of #11450 Oct 1, 2018
@jancborchardt
Copy link
Member

So we have 2 solutions here:

  1. We fix it at all the Adblockers → they will likely have more rules in the future and we need to adjust
  2. We fix it in our software → this will likely mean not using the word "share" anywhere … totally stupid rule from the adblockers. Also, they might even block the word we decide to use.

Any ideas?

@jancborchardt jancborchardt changed the title Share link option not visible in Firefox Share link not visible because of ad blockers Oct 3, 2018
@jancborchardt jancborchardt added this to the Nextcloud 15 milestone Oct 3, 2018
@blizzz
Copy link
Member

blizzz commented Oct 8, 2018

@jancborchardt #10434 (comment) sounds mostly like a config thing with ad blockers. So, configuring them it is imho. Default ublock origin does not do it, thus I think there's nothing we should/need do about it. Nor can sustainably. Perhaps a hint in FAQs or documentation.

@integras
Copy link

I have Ad Blocker Plus (ABP) and Ghostery - same problem.

But I agree with @blizzz : this is more a config issue to me, blockers can be disabled for such sites, at least if you use it as your own installation. I had looked in the manual, but not found any information:

https://docs.nextcloud.com/server/14/admin_manual/configuration_files/file_sharing_configuration.html

Just a small comment regarding this issue would have been sufficient to put me right :) So if someone can just add a sentence like:

"It is known that different ad-blockers block the share link from being displayed. If this happens to you, you have the option of disabling the ad-blocker for the site of the nextcloud installation. Most ad-blockers offer this possibility directly if you click on their logo in the browser window. Do not forget to reload the page."

Thanks!

@skjnldsv
Copy link
Member

Fixed in #11844

@SMichel72
Copy link

As I do not expect advertisments on my Nextcloud installation I've now deaktivated Adblock Plus. And it works fine again!

@dszego
Copy link

dszego commented Jan 4, 2019

I can confirm - the share link doesn't show up right with AdBlockPlus enabled. Set an exception rule for your NextCloud site.

@noisyle
Copy link

noisyle commented Feb 28, 2019

AdBlock inject "display: none !important;" to class "share-menu", which makes the share link not visible.
image

@MorrisJobke
Copy link
Member

AdBlock inject "display: none !important;" to class "share-menu", which makes the share link not visible.

Thinking about documentation in #14616

@tloubrieu
Copy link

My colleague has the same issue with the Adblocker for Chrome (NoAds) on macOS.
The nextcloud server version is 14.0.10,
Could you re-open this issue ?

@AxelTheGerman
Copy link

AxelTheGerman commented Jan 21, 2020

I was searching for the "share link" option for at least 5 minutes before decided to search for open issues.

Any chance we could just change the CSS class name to something nextcloud specific? e.g. nextcloud-share-menu

I agree that working around ad blockers sucks, but having to configure exceptions everywhere also sucks.

I doubt any ad blocker would specifically target nextcloud as long as we don't use the most obviously blocked class names

edit: not sure why I don't see the rule with that uBlock origin hides the element but it's still hidden.

Note: the clipboard icon is outside of the share menu so it's visible for existing shared links
Screen Shot 2020-01-21 at 11 44 14 AM

And as expected for non shared items, the "share link" doesn't let you share
Screen Shot 2020-01-21 at 11 47 01 AM

@skjnldsv skjnldsv removed their assignment Jan 22, 2020
@pooh22
Copy link

pooh22 commented Jun 20, 2020

I've had this issue for ages as well, I didn't know why and didn't look too hard to find out. I'm usually using firefox with ublock origin and privacy badger, canvas blocker, https everywhere plugins. My nextcloud version is 18.0.6, but this has been the case since at least nextcloud 14, maybe earlier.

The problem is that when you click on the share icon, there's no indication that something is missing, it's like the sharing capability doesn't exist. Also there's no alternative route to sharing a file, so you can't work around it, other then disabling ublock origin or using a different browser without the plugin.

This is a very basic and essential feature of nextcloud and it should just work, regardless of the plugins that on most sites protect you from trackers and ads.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. to develop Accepted and waiting to be taken care of bug feature: sharing help wanted
Projects
None yet
Development

Successfully merging a pull request may close this issue.