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

Update hosts.txt #615

Merged
merged 1 commit into from
Oct 25, 2023
Merged

Update hosts.txt #615

merged 1 commit into from
Oct 25, 2023

Conversation

stoyan
Copy link
Contributor

@stoyan stoyan commented May 5, 2023

@stoyan stoyan requested review from jefflembeck and pmeenan May 5, 2023 22:06
Copy link
Contributor

@pmeenan pmeenan left a comment

Choose a reason for hiding this comment

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

LGTM

@mjkozicki
Copy link
Contributor

@pmeenan This is an older PR, is it still valid?

@pmeenan
Copy link
Contributor

pmeenan commented Sep 25, 2023

Yep, looking at the source, that's still the latest update. Miners abusing the system haven't been a problem recently so it's less urgent, particularly since the login requirements and rate-limiting but it's a good idea to keep ahead of them.

@stoyan
Copy link
Contributor Author

stoyan commented Sep 25, 2023

I never merged this because I have not tested it in a real set up. Looks safe but I was extra paranoid.

@mjkozicki mjkozicki merged commit 4ac1576 into master Oct 25, 2023
@mjkozicki mjkozicki deleted the stoyan-patch-2 branch October 25, 2023 15:59
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.

3 participants