Skip to content

add mass scanner exclusion as default#545

Merged
mlodic merged 6 commits intodevelopfrom
add_filters
Jul 17, 2025
Merged

add mass scanner exclusion as default#545
mlodic merged 6 commits intodevelopfrom
add_filters

Conversation

@mlodic
Copy link
Copy Markdown
Member

@mlodic mlodic commented Jul 17, 2025

only consider this commit f953887 @regulartim. In this way by default the users of the public instance can get valuable IP addresses without the majority of the noise.

I am merging and deploying even before approval to fasten things up (next week I am away). Still please review so that we can add changes later in case. :) Thanks

@mlodic mlodic changed the title add mass scanner exclusion from base query add mass scanner exclusion as default Jul 17, 2025
@mlodic mlodic linked an issue Jul 17, 2025 that may be closed by this pull request
@mlodic mlodic requested a review from regulartim July 17, 2025 17:29
@mlodic mlodic merged commit aadf228 into develop Jul 17, 2025
5 checks passed
Copy link
Copy Markdown
Collaborator

@regulartim regulartim left a comment

Choose a reason for hiding this comment

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

Nice! :)

self.feed_type_sorting = None

def exclude_mass_scanners(self):
self.exclude_reputation.append("mass scanner")
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Just a minor thing: I think I would move this line directly below self.exclude_reputation = query_params["exclude_reputation"].split(";") if "exclude_reputation" in query_params else [] to make it more obvious what this property actually contains.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add option to exclude mass scanners to the Feeds page

2 participants