Fix: Subscriptions from GNU Social partially don't work

This commit is contained in:
Michael 2018-05-18 12:18:12 +00:00
parent 13ae3436b6
commit feb3d76a5b
3 changed files with 34 additions and 49 deletions

View file

@ -41,7 +41,7 @@ define('FRIENDICA_PLATFORM', 'Friendica');
define('FRIENDICA_CODENAME', 'The Tazmans Flax-lily'); define('FRIENDICA_CODENAME', 'The Tazmans Flax-lily');
define('FRIENDICA_VERSION', '2018.05-rc'); define('FRIENDICA_VERSION', '2018.05-rc');
define('DFRN_PROTOCOL_VERSION', '2.23'); define('DFRN_PROTOCOL_VERSION', '2.23');
define('DB_UPDATE_VERSION', 1263); define('DB_UPDATE_VERSION', 1264);
define('NEW_UPDATE_ROUTINE_VERSION', 1170); define('NEW_UPDATE_ROUTINE_VERSION', 1170);
/** /**

View file

@ -42,12 +42,11 @@ function pubsubhubbub_init(App $a) {
} elseif ($hub_mode === 'unsubscribe') { } elseif ($hub_mode === 'unsubscribe') {
$subscribe = 0; $subscribe = 0;
} else { } else {
logger("pubsubhubbub: invalid hub_mode=$hub_mode, ignoring."); logger("Invalid hub_mode=$hub_mode, ignoring.");
System::httpExit(404); System::httpExit(404);
} }
logger("pubsubhubbub: $hub_mode request from " . logger("$hub_mode request from " . $_SERVER['REMOTE_ADDR']);
$_SERVER['REMOTE_ADDR']);
// get the nick name from the topic, a bit hacky but needed as a fallback // get the nick name from the topic, a bit hacky but needed as a fallback
$nick = substr(strrchr($hub_topic, "/"), 1); $nick = substr(strrchr($hub_topic, "/"), 1);
@ -58,44 +57,37 @@ function pubsubhubbub_init(App $a) {
} }
if (!$nick) { if (!$nick) {
logger('pubsubhubbub: bad hub_topic=$hub_topic, ignoring.'); logger('Bad hub_topic=$hub_topic, ignoring.');
System::httpExit(404); System::httpExit(404);
} }
// fetch user from database given the nickname // fetch user from database given the nickname
$r = q("SELECT * FROM `user` WHERE `nickname` = '%s'" . $condition = ['nickname' => $nick, 'account_expired' => false, 'account_removed' => false];
" AND `account_expired` = 0 AND `account_removed` = 0 LIMIT 1", $owner = dba::selectFirst('user', ['uid', 'hidewall'], $condition);
dbesc($nick)); if (!DBM::is_result($owner)) {
logger('Local account not found: ' . $nick . ' - topic: ' . $hub_topic . ' - callback: ' . $hub_callback);
if (!DBM::is_result($r)) {
logger('pubsubhubbub: local account not found: ' . $nick);
System::httpExit(404); System::httpExit(404);
} }
$owner = $r[0];
// abort if user's wall is supposed to be private // abort if user's wall is supposed to be private
if ($r[0]['hidewall']) { if ($owner['hidewall']) {
logger('pubsubhubbub: local user ' . $nick . logger('Local user ' . $nick . 'has chosen to hide wall, ignoring.');
'has chosen to hide wall, ignoring.');
System::httpExit(403); System::httpExit(403);
} }
// get corresponding row from contact table // get corresponding row from contact table
$r = q("SELECT * FROM `contact` WHERE `uid` = %d AND NOT `blocked`". $condition = ['uid' => $owner['uid'], 'blocked' => false,
" AND NOT `pending` AND `self` LIMIT 1", 'pending' => false, 'self' => true];
intval($owner['uid'])); $contact = dba::selectFirst('contact', ['poll'], $condition);
if (!DBM::is_result($r)) { if (!DBM::is_result($contact)) {
logger('pubsubhubbub: contact not found.'); logger('Self contact for user ' . $owner['uid'] . ' not found.');
System::httpExit(404); System::httpExit(404);
} }
$contact = $r[0];
// sanity check that topic URLs are the same // sanity check that topic URLs are the same
if (!link_compare($hub_topic, $contact['poll'])) { $hub_topic2 = str_replace('/feed/', '/dfrn_poll/', $hub_topic);
logger('pubsubhubbub: hub topic ' . $hub_topic . ' != ' . if (!link_compare($hub_topic, $contact['poll']) && !link_compare($hub_topic2, $contact['poll'])) {
$contact['poll']); logger('Hub topic ' . $hub_topic . ' != ' . $contact['poll']);
System::httpExit(404); System::httpExit(404);
} }
@ -117,22 +109,19 @@ function pubsubhubbub_init(App $a) {
// give up if the HTTP return code wasn't a success (2xx) // give up if the HTTP return code wasn't a success (2xx)
if ($ret < 200 || $ret > 299) { if ($ret < 200 || $ret > 299) {
logger("pubsubhubbub: subscriber verification at $hub_callback ". logger("Subscriber verification for $hub_topic at $hub_callback returned $ret, ignoring.");
"returned $ret, ignoring.");
System::httpExit(404); System::httpExit(404);
} }
// check that the correct hub_challenge code was echoed back // check that the correct hub_challenge code was echoed back
if (trim($body) !== $hub_challenge) { if (trim($body) !== $hub_challenge) {
logger("pubsubhubbub: subscriber did not echo back ". logger("Subscriber did not echo back hub.challenge, ignoring.");
"hub.challenge, ignoring.");
logger("\"$hub_challenge\" != \"".trim($body)."\""); logger("\"$hub_challenge\" != \"".trim($body)."\"");
System::httpExit(404); System::httpExit(404);
} }
// fetch the old subscription if it exists // fetch the old subscription if it exists
$r = q("SELECT * FROM `push_subscriber` WHERE `callback_url` = '%s'", $subscriber = dba::selectFirst('push_subscriber', ['last_update', 'push'], ['callback_url' => $hub_callback]);
dbesc($hub_callback));
// delete old subscription if it exists // delete old subscription if it exists
dba::delete('push_subscriber', ['callback_url' => $hub_callback]); dba::delete('push_subscriber', ['callback_url' => $hub_callback]);
@ -142,30 +131,25 @@ function pubsubhubbub_init(App $a) {
$push_flag = 0; $push_flag = 0;
// if we are just updating an old subscription, keep the // if we are just updating an old subscription, keep the
// old values for push and last_update // old values for last_update but reset the push
if (DBM::is_result($r)) { if (DBM::is_result($subscriber)) {
$last_update = $r[0]['last_update']; $last_update = $subscriber['last_update'];
$push_flag = $r[0]['push']; $push_flag = min($subscriber['push'], 1);
} }
// subscribe means adding the row to the table // subscribe means adding the row to the table
q("INSERT INTO `push_subscriber` (`uid`, `callback_url`, " . $fields = ['uid' => $owner['uid'], 'callback_url' => $hub_callback,
"`topic`, `nickname`, `push`, `last_update`, `secret`) values " . 'topic' => $hub_topic, 'nickname' => $nick, 'push' => $push_flag,
"(%d, '%s', '%s', '%s', %d, '%s', '%s')", 'last_update' => $last_update, 'renewed' => DateTimeFormat::utcNow(),
intval($owner['uid']), 'secret' => $hub_secret];
dbesc($hub_callback), dba::insert('push_subscriber', $fields);
dbesc($hub_topic),
dbesc($nick), logger("Successfully subscribed [$hub_callback].");
intval($push_flag),
dbesc($last_update),
dbesc($hub_secret));
logger("pubsubhubbub: successfully subscribed [$hub_callback].");
} else { } else {
logger("pubsubhubbub: successfully unsubscribed [$hub_callback]."); logger("Successfully unsubscribed [$hub_callback].");
// we do nothing here, since the row was already deleted // we do nothing here, since the row was already deleted
} }
System::httpExit(202); System::httpExit(202);
} }
killme(); killme();
} }

View file

@ -1556,6 +1556,7 @@ class DBStructure
"push" => ["type" => "tinyint unsigned", "not null" => "1", "default" => "0", "comment" => "Retrial counter"], "push" => ["type" => "tinyint unsigned", "not null" => "1", "default" => "0", "comment" => "Retrial counter"],
"last_update" => ["type" => "datetime", "not null" => "1", "default" => NULL_DATE, "comment" => "Date of last successful trial"], "last_update" => ["type" => "datetime", "not null" => "1", "default" => NULL_DATE, "comment" => "Date of last successful trial"],
"next_try" => ["type" => "datetime", "not null" => "1", "default" => NULL_DATE, "comment" => "Next retrial date"], "next_try" => ["type" => "datetime", "not null" => "1", "default" => NULL_DATE, "comment" => "Next retrial date"],
"renewed" => ["type" => "datetime", "not null" => "1", "default" => NULL_DATE, "comment" => "Date of last subscription renewal"],
"secret" => ["type" => "varchar(255)", "not null" => "1", "default" => "", "comment" => ""], "secret" => ["type" => "varchar(255)", "not null" => "1", "default" => "", "comment" => ""],
], ],
"indexes" => [ "indexes" => [