Eccube\EventListener\SameSiteCookieHotfixListener

Complexity

Total Complexity

5

Size/Duplication

Total Lines	45
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	onKernelRequest()	0	15	3
A	getSubscribedEvents()	0	6	1

<?php

/*
 * This file is part of EC-CUBE
 *
 * Copyright(c) EC-CUBE CO.,LTD. All Rights Reserved.
 *
 * http://www.ec-cube.co.jp/
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Eccube\EventListener;

use Eccube\Request\Context;
use Symfony\Component\EventDispatcher\EventSubscriberInterface;
use Symfony\Component\HttpFoundation\Response;
use Symfony\Component\HttpKernel\Event\GetResponseEvent;

/**
 * Safariの一部のバージョンでSameSite=Noneを正しく扱われないバグ対応.
 *
 * @see https://bugs.webkit.org/show_bug.cgi?id=198181
 */
class SameSiteCookieHotfixListener implements EventSubscriberInterface
{
    private static $TARGET_UA_PATTERNS = [
        '/^.*iPhone; CPU iPhone OS 1[0-2].*$/',
        '/^.*iPad; CPU OS 1[0-2].*$/',
        '/^.*iPod touch; CPU iPhone OS 1[0-2].*$/',
        '/^.*Macintosh; Intel Mac OS X.*Version\/1[0-2].*Safari.*$/',
    ];

    /**
     * @var Context
     */
    protected $requestContext;

    public function __construct(Context $requestContext)
    {
        $this->requestContext = $requestContext;
    }

    public function onKernelRequest(GetResponseEvent $event)
    {
        if (!$event->isMasterRequest()) {
            return;
        }

        $ua = $event->getRequest()->headers->get('User-Agent');
        $isUnsupported = array_filter(self::$TARGET_UA_PATTERNS, function ($pattern) use ($ua) {
            return preg_match($pattern, $ua);
        });

        if ($isUnsupported) {

The expression $isUnsupported of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

            $event->setResponse(new Response('お使いのブラウザーではご利用いただけません。'));
        }
    }

    /**
     * {@inheritdoc}
     */
    public static function getSubscribedEvents()
    {
        return [
            'kernel.request' => ['onKernelRequest', 256],
        ];
    }
}