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

Get rid of the blacklisted_syslog_ranges feature #176

Open
Benjamintf1 opened this issue Jan 4, 2023 · 2 comments
Open

Get rid of the blacklisted_syslog_ranges feature #176

Benjamintf1 opened this issue Jan 4, 2023 · 2 comments

Comments

@Benjamintf1
Copy link
Member

I don't think anyone actually uses it, and even if they tried, I think it would be hard to make that feature actually useful. We should remove this functionality and the code associated with it.

@mkocher
Copy link
Member

mkocher commented Jan 9, 2023

This feature dates back to publicly accessible CF instances to prevent users from ddos'ing random IPs. I'm not sure if we want to say that's not a valid use case for CF?

@Benjamintf1
Copy link
Member Author

CF users always have had and continue to be able to ddos random ips if they wanted to.

The explaination I heard was to prevent exfiltration of logs/metrics to undesired ip addresses(which seems to me like being able to remove addresses rather then ip ranges would be easier to do in many cases?, or perhaps removing all external ip addresses or so on would be a much more functional usecase)

The biggest thing is I think that nobody is utilizing this feature at all.

@ctlong ctlong moved this from Done to Review in Progress in DEPRECATED App Platform - Logging and Metrics Jan 12, 2023
@ctlong ctlong moved this from Review in Progress to Issue - Triage complete. Needs fix. in DEPRECATED App Platform - Logging and Metrics Apr 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Waiting for Changes | Open for Contribution
DEPRECATED App Platform - Logging and...
Issue - Triage complete. Needs fix.
Development

No branches or pull requests

2 participants