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

Ability to block specific commenters #941

Open
3 tasks done
fluffy-critter opened this issue Dec 26, 2022 · 4 comments
Open
3 tasks done

Ability to block specific commenters #941

fluffy-critter opened this issue Dec 26, 2022 · 4 comments
Labels
feature needs-contributor Someone needs to implement this. Help wanted! needs-decision Architectural/Behavioral decision by maintainers needed
Milestone

Comments

@fluffy-critter
Copy link
Contributor

Checklist

  • I am running the latest version. Installing Isso from GitHub from the master branch does not fix my issue
  • I have checked the troubleshooting guide
  • I have searched the open issues, but my issue has not already been reported

What is not working?

I have a couple of extremely persistent spammers who have not yet realized that I moderate all my comments. It would be nice if I could make comments from specific email addresses or matching specific URL patterns just go into a black hole so that I don't have to see them ever again.

How can one reproduce this issue?

  1. Have good enough SEO
@ix5
Copy link
Member

ix5 commented Jan 8, 2023

That certainly is an annoying issue to deal with.

At the heart of the issue (IMO) is the fact that for Isso, commenters are mostly ephemeral. As of right now, Isso doesn't have much in the way of user management - #261 has been open for quite some time without a resolution.

Hardcoding blocked or allowed users into the config seems kind of iffy to me as well.

@fluffy-critter do you have any ideas which would be elegant but still fit within the project's current paradigms?

@ix5 ix5 added needs-contributor Someone needs to implement this. Help wanted! feature needs-decision Architectural/Behavioral decision by maintainers needed labels Jan 8, 2023
@ix5 ix5 added this to the 0.14 milestone Jan 8, 2023
@fluffy-critter
Copy link
Contributor Author

fluffy-critter commented Jan 8, 2023

I feel like a good solution in general would be to have a plugin architecture of some sort where all incoming comments can call into a function which takes all of the message data and returns a disposition, such as "delete," "post," or "hold for moderation," which could then also be a good spot to hook in a spam filter. I seem to have mentioned the nucleus of idea on #11 (without going into any actual details of what I had in mind, oops) and I still feel like it's a fundamentally good idea, and maybe now that I have some free time I can bring myself to actually work on it.

@ix5
Copy link
Member

ix5 commented Jan 8, 2023

That sounds lovely!

Make sure to include the other maintainers and regular contributors into your thought process - I'm already onboard with your plugin idea, and we do have a signals architecture in place that might prove useful for it.

@jelmer
Copy link
Member

jelmer commented Feb 1, 2024

An alternative to a plugin system might be to have say a good Python package that uses the API to automatically moderate? That might be simpler to set up and easier to extend in the future. It also allows isso itself to stay fairly lean.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature needs-contributor Someone needs to implement this. Help wanted! needs-decision Architectural/Behavioral decision by maintainers needed
Projects
None yet
Development

No branches or pull requests

3 participants