summaryrefslogtreecommitdiff
path: root/lib/facebook-graph-sdk/src/Facebook/Url/FacebookUrlDetectionHandler.php
diff options
context:
space:
mode:
authoremkael <emkael@tlen.pl>2018-04-26 01:00:12 +0200
committeremkael <emkael@tlen.pl>2018-04-26 01:00:12 +0200
commit4b8a9a5189a625bf99fedec7fd31f6e146410a14 (patch)
treea29759d5f0f0c29f07c89c59f6b5cf38407d1470 /lib/facebook-graph-sdk/src/Facebook/Url/FacebookUrlDetectionHandler.php
parentab82beef33f4c196383a81213dcdff4ebe9447ae (diff)
Update FB API library
Diffstat (limited to 'lib/facebook-graph-sdk/src/Facebook/Url/FacebookUrlDetectionHandler.php')
-rw-r--r--lib/facebook-graph-sdk/src/Facebook/Url/FacebookUrlDetectionHandler.php25
1 files changed, 22 insertions, 3 deletions
diff --git a/lib/facebook-graph-sdk/src/Facebook/Url/FacebookUrlDetectionHandler.php b/lib/facebook-graph-sdk/src/Facebook/Url/FacebookUrlDetectionHandler.php
index 5fbb9ce..1d134dd 100644
--- a/lib/facebook-graph-sdk/src/Facebook/Url/FacebookUrlDetectionHandler.php
+++ b/lib/facebook-graph-sdk/src/Facebook/Url/FacebookUrlDetectionHandler.php
@@ -1,6 +1,6 @@
<?php
/**
- * Copyright 2014 Facebook, Inc.
+ * Copyright 2017 Facebook, Inc.
*
* You are hereby granted a non-exclusive, worldwide, royalty-free license to
* use, copy, modify, and distribute this software in source code or binary
@@ -95,8 +95,9 @@ class FacebookUrlDetectionHandler implements UrlDetectionInterface
protected function getHostName()
{
// Check for proxy first
- if ($host = $this->getHeader('X_FORWARDED_HOST')) {
- $elements = explode(',', $host);
+ $header = $this->getHeader('X_FORWARDED_HOST');
+ if ($header && $this->isValidForwardedHost($header)) {
+ $elements = explode(',', $header);
$host = $elements[count($elements) - 1];
} elseif (!$host = $this->getHeader('HOST')) {
if (!$host = $this->getServerVar('SERVER_NAME')) {
@@ -160,4 +161,22 @@ class FacebookUrlDetectionHandler implements UrlDetectionInterface
{
return $this->getServerVar('HTTP_' . $key);
}
+
+ /**
+ * Checks if the value in X_FORWARDED_HOST is a valid hostname
+ * Could prevent unintended redirections
+ *
+ * @param string $header
+ *
+ * @return boolean
+ */
+ protected function isValidForwardedHost($header)
+ {
+ $elements = explode(',', $header);
+ $host = $elements[count($elements) - 1];
+
+ return preg_match("/^([a-z\d](-*[a-z\d])*)(\.([a-z\d](-*[a-z\d])*))*$/i", $host) //valid chars check
+ && 0 < strlen($host) && strlen($host) < 254 //overall length check
+ && preg_match("/^[^\.]{1,63}(\.[^\.]{1,63})*$/", $host); //length of each label
+ }
}