Apply suggestions from code review
Co-authored-by: Philipp <admin+Github@philipp.info>
This commit is contained in:
parent
c074da2b87
commit
1ef9768b98
|
@ -58,7 +58,7 @@ class PubSub extends \Friendica\BaseModule
|
||||||
|
|
||||||
$this->logger->info('Feed arrived.', ['from' => $this->request->getRemoteAddress(), 'for' => $this->args->getCommand(), 'user-agent' => $this->server['HTTP_USER_AGENT']]);
|
$this->logger->info('Feed arrived.', ['from' => $this->request->getRemoteAddress(), 'for' => $this->args->getCommand(), 'user-agent' => $this->server['HTTP_USER_AGENT']]);
|
||||||
$this->logger->debug('Data stream.', ['xml' => $xml]);
|
$this->logger->debug('Data stream.', ['xml' => $xml]);
|
||||||
$this->logger->debug('Gut request data.', ['request' => $request]);
|
$this->logger->debug('Got request data.', ['request' => $request]);
|
||||||
|
|
||||||
$nickname = $this->parameters['nickname'] ?? '';
|
$nickname = $this->parameters['nickname'] ?? '';
|
||||||
$contact_id = $this->parameters['cid'] ?? 0;
|
$contact_id = $this->parameters['cid'] ?? 0;
|
||||||
|
|
|
@ -71,7 +71,7 @@ class PubSubHubBub extends \Friendica\BaseModule
|
||||||
throw new HTTPException\ForbiddenException();
|
throw new HTTPException\ForbiddenException();
|
||||||
}
|
}
|
||||||
|
|
||||||
$this->logger->debug('Gut request data.', ['request' => $request]);
|
$this->logger->debug('Got request data.', ['request' => $request]);
|
||||||
|
|
||||||
// Subscription request from subscriber
|
// Subscription request from subscriber
|
||||||
// https://pubsubhubbub.github.io/PubSubHubbub/pubsubhubbub-core-0.4.html#rfc.section.5.1
|
// https://pubsubhubbub.github.io/PubSubHubbub/pubsubhubbub-core-0.4.html#rfc.section.5.1
|
||||||
|
|
|
@ -65,7 +65,7 @@ class Salmon extends \Friendica\BaseModule
|
||||||
protected function post(array $request = [])
|
protected function post(array $request = [])
|
||||||
{
|
{
|
||||||
$xml = Network::postdata();
|
$xml = Network::postdata();
|
||||||
$this->logger->debug('Gut request data.', ['request' => $request]);
|
$this->logger->debug('Got request data.', ['request' => $request]);
|
||||||
|
|
||||||
$nickname = $this->parameters['nickname'] ?? '';
|
$nickname = $this->parameters['nickname'] ?? '';
|
||||||
if (empty($nickname)) {
|
if (empty($nickname)) {
|
||||||
|
|
Loading…
Reference in New Issue
Block a user