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

feat: add rfc6598 reserved prefix #103

Merged
merged 1 commit into from
Mar 11, 2021
Merged

Conversation

DuckSoft
Copy link

@DuckSoft DuckSoft commented Mar 10, 2021

Just have been using Outline Shadowsocks Server for a great long while, and it worked pretty fine though,
utill few days ago an abuse alert Email from Hetnzer was received, warning that my server is "sending malicious UDP attack traffic".

图片

And here's their "evidence" -- UDP access log from their side:
图片

To them it seemed that my server is constantly attacking that 100.64.blah.blah address via UDP flooding.

A quick investigation revealed that this IP address is actually an IP address that is used and allocated by one of my users' ISP, and somehow the traffic is not correctly discarded. Looking at the code of Outline Shadowsocks Server, there is no 100.64.0.0/10 CGNAT prefix in private addresses.

So I've made this Pull Request. Hopefully this will prevent more users under CGNAT receiving some sorts of "Abuse Alert" from any VPS Providers.

@google-cla
Copy link

google-cla bot commented Mar 10, 2021

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

ℹ️ Googlers: Go here for more info.

@DuckSoft
Copy link
Author

@googlebot I signed it!

Copy link

@alalamav alalamav left a comment

Choose a reason for hiding this comment

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

LGTM, thank you for the patch.

@alalamav alalamav merged commit 77b265e into Jigsaw-Code:master Mar 11, 2021
@fortuna
Copy link

fortuna commented Mar 12, 2021

This is now released as outline-ss-server v1.3.5: https://github.com/Jigsaw-Code/outline-ss-server/releases/tag/v1.3.5
We still need to release the change to outline-server.

@fortuna fortuna changed the title add rfc6598 reserved prefix feat: add rfc6598 reserved prefix Nov 30, 2022
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