Make sure rules are consistent and catch more errors.

This commit is contained in:
James Cole 2023-01-06 06:12:39 +01:00
parent 1fee2092d6
commit 6d4e45b89f
No known key found for this signature in database
GPG Key ID: B49A324B7EAD6D80
2 changed files with 3 additions and 2 deletions

View File

@ -94,7 +94,7 @@ class UpdateRequest extends FormRequest
'trigger' => sprintf('in:%s', $triggers),
'response' => sprintf('in:%s', $responses),
'delivery' => sprintf('in:%s', $deliveries),
'url' => ['url', 'starts_with:https://', sprintf('uniqueExistingWebhook:%d', $webhook->id)],
'url' => ['url', sprintf('uniqueExistingWebhook:%d', $webhook->id)],
];
}
}

View File

@ -28,6 +28,7 @@ use FireflyIII\Helpers\Webhook\SignatureGeneratorInterface;
use FireflyIII\Models\WebhookAttempt;
use FireflyIII\Models\WebhookMessage;
use GuzzleHttp\Client;
use GuzzleHttp\Exception\ConnectException;
use GuzzleHttp\Exception\RequestException;
use JsonException;
use Log;
@ -109,7 +110,7 @@ class StandardWebhookSender implements WebhookSenderInterface
$client = new Client();
try {
$res = $client->request('POST', $this->message->webhook->url, $options);
} catch (RequestException $e) {
} catch (RequestException|ConnectException $e) {
Log::error($e->getMessage());
Log::error($e->getTraceAsString());