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

Deny access to local subnets as well (not only 127.0.0.0/8) #13

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Deny access to local subnets as well (not only 127.0.0.0/8) #13

wants to merge 1 commit into from

Conversation

justabaka
Copy link

That would be great in terms of security if you're also hosting other VMs or projects your proxy users shouldn't be able to access directly.

@KostyaEsmukov
Copy link

Denying RFC1918 blocks is certainly a good start, but these are definitely not the only ones which are private.

I'd suggest to also deny the following ones:

  • RFC5737 (192.0.2.0/24, 198.51.100.0/24, 203.0.113.0/24)
  • RFC3171 Multicast (224.0.0.0/4)
  • Broadcast (255.255.255.255/32)

RFC5735 (which is updated by RFC6598) has a more complete list of reserved address blocks at section 4.

@KostyaEsmukov
Copy link

BTW for IPv6 the FC00::/7 block is also considered local (see RFC4193).

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.

2 participants