fix(webhooks_listener): correctly set userId from event

Signed-off-by: Oleksander Piskun <oleksandr2088@icloud.com>
pull/52699/head
Oleksander Piskun 5 months ago
parent 04b1dfdf13
commit aa0135c223
  1. 3
      apps/webhook_listeners/lib/BackgroundJobs/WebhookCall.php
  2. 1
      apps/webhook_listeners/lib/Listener/WebhooksEventListener.php

@ -74,7 +74,8 @@ class WebhookCall extends QueuedJob {
} elseif (!$exApp['enabled']) {
throw new RuntimeException('ExApp ' . $exAppId . ' is disabled.');
}
$response = $appApiFunctions->exAppRequest($exAppId, $webhookUri, $webhookListener->getUserId(), $webhookListener->getHttpMethod(), [], $options);
$userId = ($data['user'] ?? [])['uid'] ?? null;
$response = $appApiFunctions->exAppRequest($exAppId, $webhookUri, $userId, $webhookListener->getHttpMethod(), [], $options);
if (is_array($response) && isset($response['error'])) {
throw new RuntimeException(sprintf('Error during request to ExApp(%s): %s', $exAppId, $response['error']));
}

@ -41,6 +41,7 @@ class WebhooksEventListener implements IEventListener {
// TODO add group membership to be able to filter on it
$data = [
'event' => $this->serializeEvent($event),
/* Do not remove 'user' from here, see BackgroundJobs/WebhookCall.php */
'user' => (is_null($user) ? null : JsonSerializer::serializeUser($user)),
'time' => time(),
];

Loading…
Cancel
Save