X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=mod%2Fsalmon.php;h=2e7a5596e5192cbf7d245654660a862b27471342;hb=3972fe62fe8afb3791e9d6526e7665501a577b81;hp=bc4410434bbf362c14eee893fca773ade3bb0d70;hpb=390f6be42ea9f71be3e1abd7eeb4b32b0c674c37;p=friendica.git diff --git a/mod/salmon.php b/mod/salmon.php index bc4410434b..2e7a5596e5 100644 --- a/mod/salmon.php +++ b/mod/salmon.php @@ -1,6 +1,6 @@ argc > 1) ? Strings::escapeTags(trim($a->argv[1])) : ''); + $nick = ((DI::args()->getArgc() > 1) ? Strings::escapeTags(trim(DI::args()->getArgv()[1])) : ''); $importer = DBA::selectFirst('user', [], ['nickname' => $nick, 'account_expired' => false, 'account_removed' => false]); if (! DBA::isResult($importer)) { @@ -62,7 +64,7 @@ function salmon_post(App $a, $xml = '') { $base = $dom; if (empty($base)) { - Logger::log('unable to locate salmon data in xml '); + Logger::notice('unable to locate salmon data in xml'); throw new \Friendica\Network\HTTPException\BadRequestException(); } @@ -100,18 +102,18 @@ function salmon_post(App $a, $xml = '') { $author_link = $author["author-link"]; if(! $author_link) { - Logger::log('Could not retrieve author URI.'); + Logger::notice('Could not retrieve author URI.'); throw new \Friendica\Network\HTTPException\BadRequestException(); } // Once we have the author URI, go to the web and try to find their public key - Logger::log('Fetching key for ' . $author_link); + Logger::notice('Fetching key for ' . $author_link); $key = Salmon::getKey($author_link, $keyhash); if(! $key) { - Logger::log('Could not retrieve author key.'); + Logger::notice('Could not retrieve author key.'); throw new \Friendica\Network\HTTPException\BadRequestException(); } @@ -131,23 +133,23 @@ function salmon_post(App $a, $xml = '') { $mode = 1; if (! $verify) { - Logger::log('message did not verify using protocol. Trying compliant format.'); + Logger::notice('message did not verify using protocol. Trying compliant format.'); $verify = Crypto::rsaVerify($compliant_format, $signature, $pubkey); $mode = 2; } if (! $verify) { - Logger::log('message did not verify using padding. Trying old statusnet format.'); + Logger::notice('message did not verify using padding. Trying old statusnet format.'); $verify = Crypto::rsaVerify($stnet_signed_data, $signature, $pubkey); $mode = 3; } if (! $verify) { - Logger::log('Message did not verify. Discarding.'); + Logger::notice('Message did not verify. Discarding.'); throw new \Friendica\Network\HTTPException\BadRequestException(); } - Logger::log('Message verified with mode '.$mode); + Logger::notice('Message verified with mode '.$mode); /* @@ -156,50 +158,25 @@ function salmon_post(App $a, $xml = '') { * */ - $r = q("SELECT * FROM `contact` WHERE `network` IN ('%s', '%s') - AND (`nurl` = '%s' OR `alias` = '%s' OR `alias` = '%s') - AND `uid` = %d LIMIT 1", - DBA::escape(Protocol::OSTATUS), - DBA::escape(Protocol::DFRN), - DBA::escape(Strings::normaliseLink($author_link)), - DBA::escape($author_link), - DBA::escape(Strings::normaliseLink($author_link)), - intval($importer['uid']) - ); - - if (!DBA::isResult($r)) { - Logger::log('Author ' . $author_link . ' unknown to user ' . $importer['uid'] . '.'); - - if (DI::pConfig()->get($importer['uid'], 'system', 'ostatus_autofriend')) { - $result = Contact::createFromProbe($importer, $author_link); - - if ($result['success']) { - $r = q("SELECT * FROM `contact` WHERE `network` = '%s' AND ( `url` = '%s' OR `alias` = '%s') - AND `uid` = %d LIMIT 1", - DBA::escape(Protocol::OSTATUS), - DBA::escape($author_link), - DBA::escape($author_link), - intval($importer['uid']) - ); - } - } + $contact = DBA::selectFirst('contact', ["`network` IN (?, ?) AND (`nurl` = ? OR `alias` = ? OR `alias` = ?) AND `uid` = ?", + Protocol::OSTATUS, Protocol::DFRN, Strings::normaliseLink($author_link), $author_link, Strings::normaliseLink($author_link), $importer['uid']]); + + if (!empty($contact['gsid'])) { + GServer::setProtocol($contact['gsid'], Post\DeliveryData::OSTATUS); } // Have we ignored the person? // If so we can not accept this post. - //if((DBA::isResult($r)) && (($r[0]['readonly']) || ($r[0]['rel'] == Contact::FOLLOWER) || ($r[0]['blocked']))) { - if (DBA::isResult($r) && $r[0]['blocked']) { - Logger::log('Ignoring this author.'); + if (!empty($contact['blocked'])) { + Logger::notice('Ignoring this author.'); throw new \Friendica\Network\HTTPException\AcceptedException(); } // Placeholder for hub discovery. $hub = ''; - $contact_rec = ((DBA::isResult($r)) ? $r[0] : []); - - OStatus::import($data, $importer, $contact_rec, $hub); + OStatus::import($data, $importer, $contact ?: [], $hub); throw new \Friendica\Network\HTTPException\OKException(); }