]> git.mxchange.org Git - friendica.git/blobdiff - mod/dfrn_poll.php
Rewrite Proxy module
[friendica.git] / mod / dfrn_poll.php
index 4e7521b0edcc81651980bdb8cb03ad1e46a4cd63..6b2016886db462274ad9da07712f11de6505c43b 100644 (file)
@@ -7,16 +7,16 @@
 use Friendica\App;
 use Friendica\Core\Config;
 use Friendica\Core\L10n;
+use Friendica\Core\Logger;
 use Friendica\Core\System;
 use Friendica\Database\DBA;
 use Friendica\Module\Login;
 use Friendica\Protocol\DFRN;
 use Friendica\Protocol\OStatus;
 use Friendica\Util\Network;
+use Friendica\Util\Strings;
 use Friendica\Util\XML;
 
-require_once 'include/items.php';
-
 function dfrn_poll_init(App $a)
 {
        Login::sessionAuth();
@@ -29,7 +29,7 @@ function dfrn_poll_init(App $a)
        $sec             = defaults($_GET, 'sec'            , '');
        $dfrn_version    = (float) defaults($_GET, 'dfrn_version'   , 2.0);
        $perm            = defaults($_GET, 'perm'           , 'r');
-       $quiet                   = x($_GET, 'quiet');
+       $quiet                   = !empty($_GET['quiet']);
 
        // Possibly it is an OStatus compatible server that requests a user feed
        $user_agent = defaults($_SERVER, 'HTTP_USER_AGENT', '');
@@ -49,7 +49,7 @@ function dfrn_poll_init(App $a)
 
        $hidewall = false;
 
-       if (($dfrn_id === '') && (!x($_POST, 'dfrn_id'))) {
+       if (($dfrn_id === '') && empty($_POST['dfrn_id'])) {
                if (Config::get('system', 'block_public') && !local_user() && !remote_user()) {
                        System::httpExit(403);
                }
@@ -68,7 +68,7 @@ function dfrn_poll_init(App $a)
                        $user = $r[0]['nickname'];
                }
 
-               logger('dfrn_poll: public feed request from ' . $_SERVER['REMOTE_ADDR'] . ' for ' . $user);
+               Logger::log('dfrn_poll: public feed request from ' . $_SERVER['REMOTE_ADDR'] . ' for ' . $user);
                header("Content-type: application/atom+xml");
                echo DFRN::feed('', $user, $last_update, 0, $hidewall);
                killme();
@@ -104,14 +104,14 @@ function dfrn_poll_init(App $a)
                if (DBA::isResult($r)) {
                        $s = Network::fetchUrl($r[0]['poll'] . '?dfrn_id=' . $my_id . '&type=profile-check');
 
-                       logger("dfrn_poll: old profile returns " . $s, LOGGER_DATA);
+                       Logger::log("dfrn_poll: old profile returns " . $s, Logger::DATA);
 
                        if (strlen($s)) {
                                $xml = XML::parseString($s);
 
                                if ((int)$xml->status === 1) {
                                        $_SESSION['authenticated'] = 1;
-                                       if (!x($_SESSION, 'remote')) {
+                                       if (empty($_SESSION['remote'])) {
                                                $_SESSION['remote'] = [];
                                        }
 
@@ -191,7 +191,7 @@ function dfrn_poll_init(App $a)
                        }
 
                        if ($final_dfrn_id != $orig_id) {
-                               logger('profile_check: ' . $final_dfrn_id . ' != ' . $orig_id, LOGGER_DEBUG);
+                               Logger::log('profile_check: ' . $final_dfrn_id . ' != ' . $orig_id, Logger::DEBUG);
                                // did not decode properly - cannot trust this site
                                System::xmlExit(3, 'Bad decryption');
                        }
@@ -228,17 +228,17 @@ function dfrn_poll_init(App $a)
 
 function dfrn_poll_post(App $a)
 {
-       $dfrn_id      = x($_POST,'dfrn_id')      ? $_POST['dfrn_id']              : '';
-       $challenge    = x($_POST,'challenge')    ? $_POST['challenge']            : '';
-       $url          = x($_POST,'url')          ? $_POST['url']                  : '';
-       $sec          = x($_POST,'sec')          ? $_POST['sec']                  : '';
-       $ptype        = x($_POST,'type')         ? $_POST['type']                 : '';
-       $dfrn_version = x($_POST,'dfrn_version') ? (float) $_POST['dfrn_version'] : 2.0;
-       $perm         = x($_POST,'perm')         ? $_POST['perm']                 : 'r';
+       $dfrn_id      = defaults($_POST, 'dfrn_id'  , '');
+       $challenge    = defaults($_POST, 'challenge', '');
+       $url          = defaults($_POST, 'url'      , '');
+       $sec          = defaults($_POST, 'sec'      , '');
+       $ptype        = defaults($_POST, 'type'     , '');
+       $perm         = defaults($_POST, 'perm'     , 'r');
+       $dfrn_version = !empty($_POST['dfrn_version']) ? (float) $_POST['dfrn_version'] : 2.0;
 
        if ($ptype === 'profile-check') {
                if (strlen($challenge) && strlen($sec)) {
-                       logger('dfrn_poll: POST: profile-check');
+                       Logger::log('dfrn_poll: POST: profile-check');
 
                        DBA::delete('profile_check', ["`expire` < ?", time()]);
                        $r = q("SELECT * FROM `profile_check` WHERE `sec` = '%s' ORDER BY `expire` DESC LIMIT 1",
@@ -283,7 +283,7 @@ function dfrn_poll_post(App $a)
                        }
 
                        if ($final_dfrn_id != $orig_id) {
-                               logger('profile_check: ' . $final_dfrn_id . ' != ' . $orig_id, LOGGER_DEBUG);
+                               Logger::log('profile_check: ' . $final_dfrn_id . ' != ' . $orig_id, Logger::DEBUG);
                                // did not decode properly - cannot trust this site
                                System::xmlExit(3, 'Bad decryption');
                        }
@@ -372,7 +372,7 @@ function dfrn_poll_post(App $a)
                // NOTREACHED
        } else {
                // Update the writable flag if it changed
-               logger('dfrn_poll: post request feed: ' . print_r($_POST, true), LOGGER_DATA);
+               Logger::log('dfrn_poll: post request feed: ' . print_r($_POST, true), Logger::DATA);
                if ($dfrn_version >= 2.21) {
                        if ($perm === 'rw') {
                                $writable = 1;
@@ -397,14 +397,13 @@ function dfrn_poll_post(App $a)
 
 function dfrn_poll_content(App $a)
 {
-       $dfrn_id         = x($_GET,'dfrn_id')         ? $_GET['dfrn_id']              : '';
-       $type            = x($_GET,'type')            ? $_GET['type']                 : 'data';
-       $last_update     = x($_GET,'last_update')     ? $_GET['last_update']          : '';
-       $destination_url = x($_GET,'destination_url') ? $_GET['destination_url']      : '';
-       $sec             = x($_GET,'sec')             ? $_GET['sec']                  : '';
-       $dfrn_version    = x($_GET,'dfrn_version')    ? (float) $_GET['dfrn_version'] : 2.0;
-       $perm            = x($_GET,'perm')            ? $_GET['perm']                 : 'r';
-       $quiet           = x($_GET,'quiet')           ? true                          : false;
+       $dfrn_id         = defaults($_GET, 'dfrn_id'        , '');
+       $type            = defaults($_GET, 'type'           , 'data');
+       $last_update     = defaults($_GET, 'last_update'    , '');
+       $destination_url = defaults($_GET, 'destination_url', '');
+       $sec             = defaults($_GET, 'sec'            , '');
+       $dfrn_version    = !empty($_GET['dfrn_version'])    ? (float) $_GET['dfrn_version'] : 2.0;
+       $quiet           = !empty($_GET['quiet']);
 
        $direction = -1;
        if (strpos($dfrn_id, ':') == 1) {
@@ -414,7 +413,7 @@ function dfrn_poll_content(App $a)
 
        if ($dfrn_id != '') {
                // initial communication from external contact
-               $hash = random_string();
+               $hash = Strings::getRandomHex();
 
                $status = 0;
 
@@ -435,7 +434,7 @@ function dfrn_poll_content(App $a)
                switch ($direction) {
                        case -1:
                                if ($type === 'profile') {
-                                       $sql_extra = sprintf(" AND ( `dfrn-id` = '%s' OR `issued-id` = '%s' ) ", DBA::escape($dfrn_id), DBA::escape($dfrn_id));
+                                       $sql_extra = sprintf(" AND (`dfrn-id` = '%s' OR `issued-id` = '%s') ", DBA::escape($dfrn_id), DBA::escape($dfrn_id));
                                } else {
                                        $sql_extra = sprintf(" AND `issued-id` = '%s' ", DBA::escape($dfrn_id));
                                }
@@ -510,19 +509,19 @@ function dfrn_poll_content(App $a)
                                ])->getBody();
                        }
 
-                       logger("dfrn_poll: sec profile: " . $s, LOGGER_DATA);
+                       Logger::log("dfrn_poll: sec profile: " . $s, Logger::DATA);
 
                        if (strlen($s) && strstr($s, '<?xml')) {
                                $xml = XML::parseString($s);
 
-                               logger('dfrn_poll: profile: parsed xml: ' . print_r($xml, true), LOGGER_DATA);
+                               Logger::log('dfrn_poll: profile: parsed xml: ' . print_r($xml, true), Logger::DATA);
 
-                               logger('dfrn_poll: secure profile: challenge: ' . $xml->challenge . ' expecting ' . $hash);
-                               logger('dfrn_poll: secure profile: sec: ' . $xml->sec . ' expecting ' . $sec);
+                               Logger::log('dfrn_poll: secure profile: challenge: ' . $xml->challenge . ' expecting ' . $hash);
+                               Logger::log('dfrn_poll: secure profile: sec: ' . $xml->sec . ' expecting ' . $sec);
 
                                if (((int) $xml->status == 0) && ($xml->challenge == $hash) && ($xml->sec == $sec)) {
                                        $_SESSION['authenticated'] = 1;
-                                       if (!x($_SESSION, 'remote')) {
+                                       if (empty($_SESSION['remote'])) {
                                                $_SESSION['remote'] = [];
                                        }
 
@@ -549,18 +548,18 @@ function dfrn_poll_content(App $a)
 
                        switch ($destination_url) {
                                case 'profile':
-                                       $a->internalRedirect('profile/' . $profile . '?f=&tab=profile';
+                                       $a->internalRedirect('profile/' . $profile . '?f=&tab=profile');
                                        break;
                                case 'photos':
-                                       $a->internalRedirect('photos/' . $profile;
+                                       $a->internalRedirect('photos/' . $profile);
                                        break;
                                case 'status':
                                case '':
-                                       $a->internalRedirect('profile/' . $profile;
+                                       $a->internalRedirect('profile/' . $profile);
                                        break;
                                default:
                                        $appendix = (strstr($destination_url, '?') ? '&f=&redir=1' : '?f=&redir=1');
-                                       System::externalRedirect($destination_url . $appendix);
+                                       $a->redirect($destination_url . $appendix);
                                        break;
                        }
                        // NOTREACHED