diff options
author | Nick Thomas <nick@gitlab.com> | 2018-08-13 12:07:20 +0000 |
---|---|---|
committer | Jose Vargas <jvargas@gitlab.com> | 2018-08-24 14:48:31 -0500 |
commit | 7bf971eb9636db98741686088d1011081760ebcb (patch) | |
tree | 91ae729985d43d0bf1023b3a6c9141db87ebfa07 /lib | |
parent | a70266767b1f565a98eece14e51d309f1b09081c (diff) | |
download | gitlab-ce-7bf971eb9636db98741686088d1011081760ebcb.tar.gz |
Merge branch 'sh-block-link-local-master-11-1-port' into 'security-11-1'
Block link-local addresses in URLBlocker (11.1 port)
See merge request gitlab/gitlabhq!2461
Diffstat (limited to 'lib')
-rw-r--r-- | lib/gitlab/url_blocker.rb | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/lib/gitlab/url_blocker.rb b/lib/gitlab/url_blocker.rb index 38be75b7482..3b483f27e70 100644 --- a/lib/gitlab/url_blocker.rb +++ b/lib/gitlab/url_blocker.rb @@ -31,6 +31,7 @@ module Gitlab validate_localhost!(addrs_info) unless allow_localhost validate_local_network!(addrs_info) unless allow_local_network + validate_link_local!(addrs_info) unless allow_local_network true end @@ -89,6 +90,13 @@ module Gitlab raise BlockedUrlError, "Requests to the local network are not allowed" end + def validate_link_local!(addrs_info) + netmask = IPAddr.new('169.254.0.0/16') + return unless addrs_info.any? { |addr| addr.ipv6_linklocal? || netmask.include?(addr.ip_address) } + + raise BlockedUrlError, "Requests to the link local network are not allowed" + end + def internal?(uri) internal_web?(uri) || internal_shell?(uri) end |