Check that the user is in the context of a salmon slap

This commit is contained in:
Mikael Nordfeldth 2016-02-23 23:42:41 +01:00
parent d672547112
commit 0eb5122817
2 changed files with 35 additions and 24 deletions

View File

@ -71,30 +71,6 @@ class UsersalmonAction extends SalmonAction
throw new ClientException(_m('Cannot handle that kind of post.')); throw new ClientException(_m('Cannot handle that kind of post.'));
} }
// Notice must either be a) in reply to a notice by this user
// or b) in reply to a notice to the attention of this user
// or c) to the attention of this user
$context = $this->activity->context;
$notice = false;
if (!empty($context->replyToID)) {
$notice = Notice::getKV('uri', $context->replyToID);
}
if ($notice instanceof Notice &&
($notice->profile_id == $this->target->id ||
array_key_exists($this->target->id, $notice->getReplies())))
{
// In reply to a notice either from or mentioning this user.
} elseif (!empty($context->attention) &&
array_key_exists($this->target->getUri(), $context->attention)) {
// To the attention of this user.
} else {
// TRANS: Client exception.
throw new ClientException(_m('Not to anyone in reply to anything.'));
}
try { try {
$this->saveNotice(); $this->saveNotice();
} catch (AlreadyFulfilledException $e) { } catch (AlreadyFulfilledException $e) {

View File

@ -98,6 +98,41 @@ class SalmonAction extends Action
assert($this->target instanceof Profile); assert($this->target instanceof Profile);
common_log(LOG_DEBUG, "Got a " . $this->activity->verb); common_log(LOG_DEBUG, "Got a " . $this->activity->verb);
// Notice must either be a) in reply to a notice by this user
// or b) in reply to a notice to the attention of this user
// or c) to the attention of this user
// or d) reference the user as an activity:object
$notice = null;
if (!empty($this->activity->context->replyToID)) {
try {
$notice = Notice::getKV('uri', $this->activity->context->replyToID);
} catch (NoResultException $e) {
$notice = false;
}
}
if ($notice instanceof Notice &&
($this->target->sameAs($notice->getProfile())
|| array_key_exists($this->target->getID(), $notice->getAttentionProfileIDs())
)) {
// In reply to a notice either from or mentioning this user.
common_debug('User is the owner or was in the attention list of thr:in-reply-to activity.');
} elseif (!empty($this->activity->context->attention) &&
array_key_exists($this->target->getUri(), $this->activity->context->attention)) {
// To the attention of this user.
common_debug('User was in attention list of salmon slap.');
} elseif (!empty($this->activity->objects) && $this->activity->objects[0]->id === $this->target->getUri()) {
// The user is the object of this slap (unfollow for example)
common_debug('User URI was the id of the salmon slap object.');
} else {
common_debug('User was NOT found in salmon slap context.');
// TRANS: Client exception.
throw new ClientException(_m('The owner of this salmon endpoint was not in the context of the carried slap.'));
}
try { try {
$options = [ 'source' => 'ostatus' ]; $options = [ 'source' => 'ostatus' ];
common_debug('Save salmon slap directly with Notice::saveActivity for actor=='.$this->actor->getID()); common_debug('Save salmon slap directly with Notice::saveActivity for actor=='.$this->actor->getID());