]> git.mxchange.org Git - quix0rs-gnu-social.git/commitdiff
getAliases for Profile and Notice
authorMikael Nordfeldth <mmn@hethane.se>
Tue, 23 Feb 2016 13:00:59 +0000 (14:00 +0100)
committerMikael Nordfeldth <mmn@hethane.se>
Tue, 23 Feb 2016 13:00:59 +0000 (14:00 +0100)
Also move fancyurlfix into site-wide $config['fix']['fancyurls']

TODO: getByUri should make use of this directly I guess?

classes/Managed_DataObject.php
lib/default.php
plugins/WebFinger/WebFingerPlugin.php
plugins/WebFinger/lib/webfingerresource.php

index 31ae6614fbd971872a3da0f27e1261c1cd076ac9..cab7edd5c71750a014cd9463dc0110dd09ca33ab 100644 (file)
@@ -412,6 +412,55 @@ abstract class Managed_DataObject extends Memcached_DataObject
         return intval($this->id);
     }
 
+    /**
+     * WARNING: Only use this on Profile and Notice. We should probably do
+     * this with traits/"implements" or whatever, but that's over the top
+     * right now, I'm just throwing this in here to avoid code duplication
+     * in Profile and Notice classes.
+     */
+    public function getAliases()
+    {
+        $aliases = array();
+        $aliases[$this->getUri()] = $this->getID();
+
+        try {
+            $aliases[$this->getUrl()] = $this->getID();
+        } catch (InvalidUrlException $e) {
+            // getUrl failed because no valid URL could be returned, just ignore it
+        }
+
+        if (common_config('fix', 'fancyurls')) {
+            /**
+             * Here we add some hacky hotfixes for remote lookups that have been taught the
+             * (at least now) wrong URI but it's still obviously the same user. Such as:
+             * - https://site.example/user/1 even if the client requests https://site.example/index.php/user/1
+             * - https://site.example/user/1 even if the client requests https://site.example//index.php/user/1
+             * - https://site.example/index.php/user/1 even if the client requests https://site.example/user/1
+             * - https://site.example/index.php/user/1 even if the client requests https://site.example///index.php/user/1
+             */
+            foreach ($aliases as $alias=>$id) {
+                try {
+                    // get a "fancy url" version of the alias, even without index.php/
+                    $alt_url = common_fake_local_fancy_url($alias);
+                    // store this as well so remote sites can be sure we really are the same profile
+                    $aliases[$alt_url] = $id;
+                } catch (Exception $e) {
+                    // Apparently we couldn't rewrite that, the $alias was as the function wanted it to be
+                }
+
+                try {
+                    // get a non-"fancy url" version of the alias, i.e. add index.php/
+                    $alt_url = common_fake_local_nonfancy_url($alias);
+                    // store this as well so remote sites can be sure we really are the same profile
+                    $aliases[$alt_url] = $id;
+                } catch (Exception $e) {
+                    // Apparently we couldn't rewrite that, the $alias was as the function wanted it to be
+                }
+            }
+        }
+        return $aliases;
+    }
+
     // 'update' won't write key columns, so we have to do it ourselves.
     // This also automatically calls "update" _before_ it sets the keys.
     // FIXME: This only works with single-column primary keys so far! Beware!
index 1b420684b664ba87d1b20faea71606332514e32f..f8ce3bd4fe8880effeab8dfa207bfa77439ec1ef 100644 (file)
@@ -81,6 +81,9 @@ $default =
               'log_queries' => false, // true to log all DB queries
               'log_slow_queries' => 0, // if set, log queries taking over N seconds
               'mysql_foreign_keys' => false), // if set, enables experimental foreign key support on MySQL
+        'fix' =>
+        array('fancyurls' => true,   // makes sure aliases in WebFinger etc. are not f'd by index.php/ URLs
+              ),
         'syslog' =>
         array('appname' => 'statusnet', # for syslog
               'priority' => 'debug', # XXX: currently ignored
index fd25482c7a232e1dc1e89179e45c21a29c701768..d902947d934da0c59ffc441c387cf6d8e1db06de 100644 (file)
@@ -36,12 +36,10 @@ class WebFingerPlugin extends Plugin
     const OAUTH_AUTHORIZE_REL       = 'http://apinamespace.org/oauth/authorize';
 
     public $http_alias = false;
-    public $fancyurlfix = true; // adds + interprets some extra aliases related to 'index.php/' URLs
 
     public function initialize()
     {
         common_config_set('webfinger', 'http_alias', $this->http_alias);
-        common_config_set('webfinger', 'fancyurlfix', $this->fancyurlfix);
     }
 
     public function onRouterInitialized($m)
@@ -106,7 +104,7 @@ class WebFingerPlugin extends Plugin
                 $user = User::getByUri($resource);
                 $profile = $user->getProfile();
             } catch (NoResultException $e) {
-                if (common_config('webfinger', 'fancyurlfix')) {
+                if (common_config('fix', 'fancyurls')) {
                     try {
                         try {   // if it's a /index.php/ url
                             // common_fake_local_fancy_url can throw an exception
index e04d3b407f2f11f8cd6a614f76deba1cfbe9d6f5..3afbd41713a004385f04460507f74cd04e52fa37 100644 (file)
@@ -31,49 +31,18 @@ abstract class WebFingerResource
 
     public function getAliases()
     {
-        $aliases = array();
-
-        // Add the URI as an identity, this is _not_ necessarily an HTTP url
-        $uri = $this->object->getUri();
-        $aliases[$uri] = true;
-        if (common_config('webfinger', 'http_alias')
-                && strtolower(parse_url($uri, PHP_URL_SCHEME)) === 'https') {
-            $aliases[preg_replace('/^https:/', 'http:', $uri, 1)] = true;
-        }
-
-        try {
-            $aliases[$this->object->getUrl()] = true;
-        } catch (InvalidUrlException $e) {
-            // getUrl failed because no valid URL could be returned, just ignore it
-        }
-
-        if (common_config('webfinger', 'fancyurlfix')) {
-            /**
-             * Here we add some hacky hotfixes for remote lookups that have been taught the
-             * (at least now) wrong URI but it's still obviously the same user. Such as:
-             * - https://site.example/user/1 even if the client requests https://site.example/index.php/user/1
-             * - https://site.example/user/1 even if the client requests https://site.example//index.php/user/1
-             * - https://site.example/index.php/user/1 even if the client requests https://site.example/user/1
-             * - https://site.example/index.php/user/1 even if the client requests https://site.example///index.php/user/1
-             */
-            foreach(array_keys($aliases) as $alias) {
-                try {
-                    // get a "fancy url" version of the alias, even without index.php/
-                    $alt_url = common_fake_local_fancy_url($alias);
-                    // store this as well so remote sites can be sure we really are the same profile
-                    $aliases[$alt_url] = true;
-                } catch (Exception $e) {
-                    // Apparently we couldn't rewrite that, the $alias was as the function wanted it to be
-                }
-    
-                try {
-                    // get a non-"fancy url" version of the alias, i.e. add index.php/
-                    $alt_url = common_fake_local_nonfancy_url($alias);
-                    // store this as well so remote sites can be sure we really are the same profile
-                    $aliases[$alt_url] = true;
-                } catch (Exception $e) {
-                    // Apparently we couldn't rewrite that, the $alias was as the function wanted it to be
+        $aliases = $this->object->getAliases();
+
+        // Some sites have changed from http to https and still want
+        // (because remote sites look for it) verify that they are still
+        // the same identity as they were on HTTP. Should NOT be used if
+        // you've run HTTPS all the time!
+        if (common_config('webfinger', 'http_alias')) {
+            foreach ($aliases as $alias=>$id) {
+                if (!strtolower(parse_url($alias, PHP_URL_SCHEME)) === 'https') {
+                    continue;
                 }
+                $aliases[preg_replace('/^https:/', 'http:', $alias, 1)] = $id;
             }
         }