GoCardlessPayment\Http\Controllers\WebhookController

Complexity

Total Complexity

5

Size/Duplication

Total Lines	37
Duplicated Lines	0 %

Test Coverage

Coverage

88.89%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	handleEvent()	0	7	2
A	handleWebhook()	0	23	3

<?php

namespace GoCardlessPayment\Http\Controllers;

use GoCardlessPayment\Events\GoCardlessWebhookEventReceived;
use GoCardlessPayment\GoCardlessPayment;
use GoCardlessPro\Resources\Event;
use Illuminate\Http\Request;
use Illuminate\Support\Facades\Log;
use Symfony\Component\HttpFoundation\Response;

class WebhookController
{
    /**
     * @see https://developer.gocardless.com/getting-started/staying-up-to-date-with-webhooks/#processing_events
     */
    public function handleWebhook(Request $request)
    {
        $requestSignature = $request->header('Webhook-Signature');

        try {
            /** @var Event[] $events */
            /** @psalm-suppress UndefinedDocblockClass */
            $events = \GoCardlessPro\Webhook::parse(
                $request->getContent(),
                $requestSignature,

It seems like $requestSignature can also be of type array; however, parameter $signature_header of GoCardlessPro\Webhook::parse() does only seem to accept string, maybe add an additional type check?

                config('gocardless-payment.web.webhook_endpoint_secret')
            );
        } catch (\Exception $e) {
            Log::error($e->getMessage());

            return new Response($e->getMessage(), 500);
        }

        foreach ($events as $event) {
            $this->handleEvent($event);
        }

        return new Response;
    }

    protected function handleEvent(Event $event): void
    {
        GoCardlessWebhookEventReceived::dispatch($event);

        $jobClass = GoCardlessPayment::getWebhookJob("{$event->resource_type}.{$event->action}");
        if ($jobClass) {
            $jobClass::dispatch($event)->onQueue(config('gocardless-payment.queue'));
        }
    }
}