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

Rename login disclaimer #4268

Merged

Conversation

dann-merlin
Copy link
Contributor

The HTML element disclaimerContainer on the login page is confused by adblockers using the "I don't care about cookies" filters with cookie disclaimers. This results in the disclaimer being blocked when using popular adblocker extensions like uBlock Origin.

Changes
This commit renames
disclaimerContainer -> loginDisclaimerContainer
disclaimer -> loginDisclaimer
to be more verbose and circumvent that filter.

Issues
This was most likely the reason for #3737

@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 8, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@dmitrylyzo
Copy link
Contributor

dmitrylyzo commented Jan 8, 2023

https://demo.jellyfin.org/stable works with uBlock Origin 1.46.0 (Firefox 108 and Chrome 108)

What kind of filter affects this?

UPD:
It seems to be EasyList Cookie.

@dann-merlin
Copy link
Contributor Author

I thought it was the i-dont-care-about-cookies.txt. Specifically I believe this line to be the culprit (found it in my uBlock Origin filters list):

##.disclaimerContainer:not(body):not(html):not(.grid__unit)

I might be wrong though. The demo site does not work for me (UngoogledChromium 108 with uBlock Origin 1.40.8 and Librewolf 108 with uBlock Origin 1.46.0).

I tried it with normal Firefox aswell now and there it works (Firefox 108 with uBlock Origin 1.46.0).

I don't know why this is the case tbh.

The list you mentioned also introduces a matching filter:

##.disclaimerContainer

but I could not find this one in my uBlock Origin filters list.

@thornbill
Copy link
Member

This seems like it could become almost a game of whack-a-mole with these plugins randomly hiding stuff. Maybe we should go with something like loginMessage to avoid the "disclaimer" bit entirely.

@thornbill thornbill added the enhancement Improve existing functionality or small fixes label Jan 12, 2023
disclaimerContainer is confused by adblockers using the
"I don't care about cookies" filters with cookie disclaimers.
This results in the disclaimer being blocked when using popular
AdBlocker Extensions.
This commit renames
disclaimerContainer -> loginDisclaimerContainer
disclaimer -> loginDisclaimer
to be more verbose and circumvent that filter
@thornbill thornbill force-pushed the dann-merlin/rename_login_disclaimer branch from 4bc312f to 9f8e98b Compare February 7, 2023 16:17
@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 7, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

Copy link
Member

@thornbill thornbill left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I changed my mind... we refer to this as a login disclaimer in the API/server so this change makes sense regardless of what dumb things browser plugins are doing. 😄

@thornbill thornbill merged commit 633b932 into jellyfin:master Feb 8, 2023
@dann-merlin dann-merlin deleted the dann-merlin/rename_login_disclaimer branch February 9, 2023 01:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Improve existing functionality or small fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants