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

Blacklist facebook comments plugin #921

Merged
merged 2 commits into from
Jan 2, 2025
Merged

Blacklist facebook comments plugin #921

merged 2 commits into from
Jan 2, 2025

Conversation

Copy link
Contributor

@MikeZeDev MikeZeDev left a comment

Choose a reason for hiding this comment

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

you might want to blacklist non www domains too, and perhaps fbcdn ?

@Sheepux
Copy link
Contributor Author

Sheepux commented Dec 11, 2024

you might want to blacklist non www domains too,

Check the MR ;)

    '*://*.facebook.com/*',
    '*://*.facebook.net/*',

and perhaps fbcdn ?

I don't know the impact, that'll be a topic for another time.

I just did this PR in case you or Ronny had issues in mind if banning such domains would break things (connectors that would stop working)

@MikeZeDev
Copy link
Contributor

MikeZeDev commented Dec 11, 2024

AFAIK, those pattern will filter

https://anything.facebook.com/anything

but not

https://facebook.com/anything

Like i've said you may want to filter it too ^^

@Sheepux
Copy link
Contributor Author

Sheepux commented Dec 11, 2024

Don't need to unless proven otherwise

@ronny1982
Copy link
Contributor

ronny1982 commented Dec 13, 2024

Will this prevent users from interactive login on manga websites via facebook?

@Sheepux
Copy link
Contributor Author

Sheepux commented Dec 14, 2024

Didn't think about that !
I don't have facebook/connector, can someone test it ?

@Sheepux
Copy link
Contributor Author

Sheepux commented Dec 14, 2024

Or should i limit the scope to
*://www.facebook.com/*/plugins/comments.php*

@MikeZeDev
Copy link
Contributor

I think limiting to the comment plugin may be a viable solution.

@ronny1982
Copy link
Contributor

Probably safer for now

@Sheepux Sheepux changed the title Blacklist facebook Blacklist facebook comments plugin Dec 14, 2024
@Sheepux Sheepux added the Engine label Jan 2, 2025
@Sheepux Sheepux merged commit f61067a into master Jan 2, 2025
7 checks passed
@MikeZeDev MikeZeDev deleted the facebook branch January 18, 2025 18:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants