sunrise-php / http-message

src/Header/RefreshHeader.php

Indentation +61 added lines, -61 removed lines

@@ -31,72 +31,72 @@
 class RefreshHeader extends Header
 {
 
-    /**
-     * @var int
-     */
-    private int $delay;
+	/**
+	 * @var int
+	 */
+	private int $delay;
 
-    /**
-     * @var UriInterface
-     */
-    private UriInterface $uri;
+	/**
+	 * @var UriInterface
+	 */
+	private UriInterface $uri;
 
-    /**
-     * Constructor of the class
-     *
-     * @param int $delay
-     * @param mixed $uri
-     *
-     * @throws InvalidUriException
-     *         If the URI isn't valid.
-     *
-     * @throws InvalidHeaderValueException
-     *         If the delay isn't valid.
-     */
-    public function __construct(int $delay, $uri)
-    {
-        $this->validateDelay($delay);
+	/**
+	 * Constructor of the class
+	 *
+	 * @param int $delay
+	 * @param mixed $uri
+	 *
+	 * @throws InvalidUriException
+	 *         If the URI isn't valid.
+	 *
+	 * @throws InvalidHeaderValueException
+	 *         If the delay isn't valid.
+	 */
+	public function __construct(int $delay, $uri)
+	{
+		$this->validateDelay($delay);
 
-        $uri = Uri::create($uri);
+		$uri = Uri::create($uri);
 
-        $this->delay = $delay;
-        $this->uri = $uri;
-    }
+		$this->delay = $delay;
+		$this->uri = $uri;
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    public function getFieldName(): string
-    {
-        return 'Refresh';
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function getFieldName(): string
+	{
+		return 'Refresh';
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    public function getFieldValue(): string
-    {
-        return sprintf('%d; url=%s', $this->delay, $this->uri->__toString());
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function getFieldValue(): string
+	{
+		return sprintf('%d; url=%s', $this->delay, $this->uri->__toString());
+	}
 
-    /**
-     * Validates the redirection delay
-     *
-     * @param int $delay
-     *
-     * @return void
-     *
-     * @throws InvalidHeaderValueException
-     *         If the delay isn't valid.
-     */
-    private function validateDelay(int $delay): void
-    {
-        if (! ($delay >= 0)) {
-            throw new InvalidHeaderValueException(sprintf(
-                'The delay "%2$d" for the header "%1$s" is not valid',
-                $this->getFieldName(),
-                $delay
-            ));
-        }
-    }
+	/**
+	 * Validates the redirection delay
+	 *
+	 * @param int $delay
+	 *
+	 * @return void
+	 *
+	 * @throws InvalidHeaderValueException
+	 *         If the delay isn't valid.
+	 */
+	private function validateDelay(int $delay): void
+	{
+		if (! ($delay >= 0)) {
+			throw new InvalidHeaderValueException(sprintf(
+				'The delay "%2$d" for the header "%1$s" is not valid',
+				$this->getFieldName(),
+				$delay
+			));
+		}
+	}
 }

Spacing +1 added lines, -1 removed lines

@@ -91,7 +91,7 @@
      */
     private function validateDelay(int $delay): void
     {
-        if (! ($delay >= 0)) {
+        if (!($delay >= 0)) {
             throw new InvalidHeaderValueException(sprintf(
                 'The delay "%2$d" for the header "%1$s" is not valid',
                 $this->getFieldName(),

src/Header/AccessControlAllowHeadersHeader.php

Indentation +32 added lines, -32 removed lines

@@ -28,41 +28,41 @@
 class AccessControlAllowHeadersHeader extends Header
 {
 
-    /**
-     * @var list<string>
-     */
-    private array $headers;
+	/**
+	 * @var list<string>
+	 */
+	private array $headers;
 
-    /**
-     * Constructor of the class
-     *
-     * @param string ...$headers
-     *
-     * @throws InvalidHeaderValueException
-     *         If one of the header names isn't valid.
-     */
-    public function __construct(string ...$headers)
-    {
-        /** @var list<string> $headers */
+	/**
+	 * Constructor of the class
+	 *
+	 * @param string ...$headers
+	 *
+	 * @throws InvalidHeaderValueException
+	 *         If one of the header names isn't valid.
+	 */
+	public function __construct(string ...$headers)
+	{
+		/** @var list<string> $headers */
 
-        $this->validateToken(...$headers);
+		$this->validateToken(...$headers);
 
-        $this->headers = $headers;
-    }
+		$this->headers = $headers;
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    public function getFieldName(): string
-    {
-        return 'Access-Control-Allow-Headers';
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function getFieldName(): string
+	{
+		return 'Access-Control-Allow-Headers';
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    public function getFieldValue(): string
-    {
-        return implode(', ', $this->headers);
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function getFieldValue(): string
+	{
+		return implode(', ', $this->headers);
+	}
 }

src/Header/ContentEncodingHeader.php

Indentation +41 added lines, -41 removed lines

@@ -28,51 +28,51 @@
 class ContentEncodingHeader extends Header
 {
 
-    /**
-     * Directives
-     *
-     * @var string
-     */
-    public const GZIP = 'gzip';
-    public const COMPRESS = 'compress';
-    public const DEFLATE = 'deflate';
-    public const BR = 'br';
+	/**
+	 * Directives
+	 *
+	 * @var string
+	 */
+	public const GZIP = 'gzip';
+	public const COMPRESS = 'compress';
+	public const DEFLATE = 'deflate';
+	public const BR = 'br';
 
-    /**
-     * @var list<string>
-     */
-    private array $directives;
+	/**
+	 * @var list<string>
+	 */
+	private array $directives;
 
-    /**
-     * Constructor of the class
-     *
-     * @param string ...$directives
-     *
-     * @throws InvalidHeaderValueException
-     *         If one of the directives isn't valid.
-     */
-    public function __construct(string ...$directives)
-    {
-        /** @var list<string> $directives */
+	/**
+	 * Constructor of the class
+	 *
+	 * @param string ...$directives
+	 *
+	 * @throws InvalidHeaderValueException
+	 *         If one of the directives isn't valid.
+	 */
+	public function __construct(string ...$directives)
+	{
+		/** @var list<string> $directives */
 
-        $this->validateToken(...$directives);
+		$this->validateToken(...$directives);
 
-        $this->directives = $directives;
-    }
+		$this->directives = $directives;
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    public function getFieldName(): string
-    {
-        return 'Content-Encoding';
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function getFieldName(): string
+	{
+		return 'Content-Encoding';
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    public function getFieldValue(): string
-    {
-        return implode(', ', $this->directives);
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function getFieldValue(): string
+	{
+		return implode(', ', $this->directives);
+	}
 }

src/Header/CacheControlHeader.php

Indentation +41 added lines, -41 removed lines

@@ -29,52 +29,52 @@
 class CacheControlHeader extends Header
 {
 
-    /**
-     * @var array<string, string>
-     */
-    private array $parameters;
+	/**
+	 * @var array<string, string>
+	 */
+	private array $parameters;
 
-    /**
-     * Constructor of the class
-     *
-     * @param array<array-key, mixed> $parameters
-     *
-     * @throws InvalidHeaderValueParameterException
-     *         If the parameters aren't valid.
-     */
-    public function __construct(array $parameters)
-    {
-        $parameters = $this->validateParameters($parameters);
+	/**
+	 * Constructor of the class
+	 *
+	 * @param array<array-key, mixed> $parameters
+	 *
+	 * @throws InvalidHeaderValueParameterException
+	 *         If the parameters aren't valid.
+	 */
+	public function __construct(array $parameters)
+	{
+		$parameters = $this->validateParameters($parameters);
 
-        $this->parameters = $parameters;
-    }
+		$this->parameters = $parameters;
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    public function getFieldName(): string
-    {
-        return 'Cache-Control';
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function getFieldName(): string
+	{
+		return 'Cache-Control';
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    public function getFieldValue(): string
-    {
-        $segments = [];
-        foreach ($this->parameters as $name => $value) {
-            // the construction <foo=> isn't valid...
-            if ($value === '') {
-                $segments[] = $name;
-                continue;
-            }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function getFieldValue(): string
+	{
+		$segments = [];
+		foreach ($this->parameters as $name => $value) {
+			// the construction <foo=> isn't valid...
+			if ($value === '') {
+				$segments[] = $name;
+				continue;
+			}
 
-            $format = $this->isToken($value) ? '%s=%s' : '%s="%s"';
+			$format = $this->isToken($value) ? '%s=%s' : '%s="%s"';
 
-            $segments[] = sprintf($format, $name, $value);
-        }
+			$segments[] = sprintf($format, $name, $value);
+		}
 
-        return implode(', ', $segments);
-    }
+		return implode(', ', $segments);
+	}
 }

src/Header/SetCookieHeader.php

Indentation +239 added lines, -239 removed lines

@@ -35,243 +35,243 @@
 class SetCookieHeader extends Header
 {
 
-    /**
-     * Cookies are not sent on normal cross-site subrequests, but
-     * are sent when a user is navigating to the origin site.
-     *
-     * This is the default cookie value if SameSite has not been
-     * explicitly specified in recent browser versions..
-     *
-     * @link https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Set-Cookie/SameSite#lax
-     *
-     * @var string
-     */
-    public const SAME_SITE_LAX = 'Lax';
-
-    /**
-     * Cookies will only be sent in a first-party context and not
-     * be sent along with requests initiated by third party websites.
-     *
-     * @link https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Set-Cookie/SameSite#strict
-     *
-     * @var string
-     */
-    public const SAME_SITE_STRICT = 'Strict';
-
-    /**
-     * Cookies will be sent in all contexts, i.e. in responses to
-     * both first-party and cross-site requests.
-     *
-     * If SameSite=None is set, the cookie Secure attribute must
-     * also be set (or the cookie will be blocked).
-     *
-     * @link https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Set-Cookie/SameSite#none
-     *
-     * @var string
-     */
-    public const SAME_SITE_NONE = 'None';
-
-    /**
-     * Cookie option keys
-     *
-     * @var string
-     */
-    public const OPTION_KEY_PATH = 'path';
-    public const OPTION_KEY_DOMAIN = 'domain';
-    public const OPTION_KEY_SECURE = 'secure';
-    public const OPTION_KEY_HTTP_ONLY = 'httpOnly';
-    public const OPTION_KEY_SAMESITE = 'sameSite';
-
-    /**
-     * Default cookie options
-     *
-     * @var array{
-     *        path?: ?string,
-     *        domain?: ?string,
-     *        secure?: ?bool,
-     *        httpOnly?: ?bool,
-     *        sameSite?: ?string
-     *      }
-     */
-    protected static array $defaultOptions = [
-        self::OPTION_KEY_PATH      => '/',
-        self::OPTION_KEY_DOMAIN    => null,
-        self::OPTION_KEY_SECURE    => null,
-        self::OPTION_KEY_HTTP_ONLY => true,
-        self::OPTION_KEY_SAMESITE  => self::SAME_SITE_LAX,
-    ];
-
-    /**
-     * The cookie name
-     *
-     * @var string
-     */
-    private string $name;
-
-    /**
-     * The cookie value
-     *
-     * @var string
-     */
-    private string $value;
-
-    /**
-     * The cookie expiration date
-     *
-     * @var DateTimeInterface|null
-     */
-    private ?DateTimeInterface $expires;
-
-    /**
-     * The cookie options
-     *
-     * @var array{
-     *        path?: ?string,
-     *        domain?: ?string,
-     *        secure?: ?bool,
-     *        httpOnly?: ?bool,
-     *        sameSite?: ?string
-     *      }
-     */
-    private array $options;
-
-    /**
-     * Constructor of the class
-     *
-     * @param string $name
-     * @param string $value
-     * @param DateTimeInterface|null $expires
-     * @param array{path?: ?string, domain?: ?string, secure?: ?bool, httpOnly?: ?bool, sameSite?: ?string} $options
-     *
-     * @throws InvalidHeaderValueException
-     *         If one of the parameters isn't valid.
-     */
-    public function __construct(string $name, string $value, ?DateTimeInterface $expires = null, array $options = [])
-    {
-        $this->validateCookieName($name);
-
-        if (isset($options[self::OPTION_KEY_PATH])) {
-            $this->validateCookieOption(self::OPTION_KEY_PATH, $options[self::OPTION_KEY_PATH]);
-        }
-
-        if (isset($options[self::OPTION_KEY_DOMAIN])) {
-            $this->validateCookieOption(self::OPTION_KEY_DOMAIN, $options[self::OPTION_KEY_DOMAIN]);
-        }
-
-        if (isset($options[self::OPTION_KEY_SAMESITE])) {
-            $this->validateCookieOption(self::OPTION_KEY_SAMESITE, $options[self::OPTION_KEY_SAMESITE]);
-        }
-
-        if ($value === '') {
-            $value = 'deleted';
-            $expires = new DateTimeImmutable('1 year ago');
-        }
-
-        $options += static::$defaultOptions;
-
-        $this->name = $name;
-        $this->value = $value;
-        $this->expires = $expires;
-        $this->options = $options;
-    }
-
-    /**
-     * {@inheritdoc}
-     */
-    public function getFieldName(): string
-    {
-        return 'Set-Cookie';
-    }
-
-    /**
-     * {@inheritdoc}
-     */
-    public function getFieldValue(): string
-    {
-        $name = rawurlencode($this->name);
-        $value = rawurlencode($this->value);
-        $result = sprintf('%s=%s', $name, $value);
-
-        if (isset($this->expires)) {
-            $result .= '; Expires=' . $this->formatDateTime($this->expires);
-            $result .= '; Max-Age=' . max($this->expires->getTimestamp() - time(), 0);
-        }
-
-        if (isset($this->options[self::OPTION_KEY_PATH])) {
-            $result .= '; Path=' . $this->options[self::OPTION_KEY_PATH];
-        }
-
-        if (isset($this->options[self::OPTION_KEY_DOMAIN])) {
-            $result .= '; Domain=' . $this->options[self::OPTION_KEY_DOMAIN];
-        }
-
-        if (isset($this->options[self::OPTION_KEY_SECURE]) && $this->options[self::OPTION_KEY_SECURE]) {
-            $result .= '; Secure';
-        }
-
-        if (isset($this->options[self::OPTION_KEY_HTTP_ONLY]) && $this->options[self::OPTION_KEY_HTTP_ONLY]) {
-            $result .= '; HttpOnly';
-        }
-
-        if (isset($this->options[self::OPTION_KEY_SAMESITE])) {
-            $result .= '; SameSite=' . $this->options[self::OPTION_KEY_SAMESITE];
-        }
-
-        return $result;
-    }
-
-    /**
-     * Validates the given cookie name
-     *
-     * @param string $name
-     *
-     * @return void
-     *
-     * @throws InvalidHeaderValueException
-     *         If the cookie name isn't valid.
-     */
-    private function validateCookieName(string $name): void
-    {
-        if ('' === $name) {
-            throw new InvalidHeaderValueException('Cookie name cannot be empty');
-        }
-
-        // https://github.com/php/php-src/blob/02a5335b710aa36cd0c3108bfb9c6f7a57d40000/ext/standard/head.c#L93
-        if (strpbrk($name, "=,; \t\r\n\013\014") !== false) {
-            throw new InvalidHeaderValueException(sprintf(
-                'The cookie name "%s" contains prohibited characters',
-                $name
-            ));
-        }
-    }
-
-    /**
-     * Validates the given cookie option
-     *
-     * @param string $validKey
-     * @param mixed $value
-     *
-     * @return void
-     *
-     * @throws InvalidHeaderValueException
-     *         If the cookie option isn't valid.
-     */
-    private function validateCookieOption(string $validKey, $value): void
-    {
-        if (!is_string($value)) {
-            throw new InvalidHeaderValueException(sprintf(
-                'The cookie option "%s" must be a string',
-                $validKey
-            ));
-        }
-
-        // https://github.com/php/php-src/blob/02a5335b710aa36cd0c3108bfb9c6f7a57d40000/ext/standard/head.c#L103
-        // https://github.com/php/php-src/blob/02a5335b710aa36cd0c3108bfb9c6f7a57d40000/ext/standard/head.c#L108
-        if (strpbrk($value, ",; \t\r\n\013\014") !== false) {
-            throw new InvalidHeaderValueException(sprintf(
-                'The cookie option "%s" contains prohibited characters',
-                $validKey
-            ));
-        }
-    }
+	/**
+	 * Cookies are not sent on normal cross-site subrequests, but
+	 * are sent when a user is navigating to the origin site.
+	 *
+	 * This is the default cookie value if SameSite has not been
+	 * explicitly specified in recent browser versions..
+	 *
+	 * @link https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Set-Cookie/SameSite#lax
+	 *
+	 * @var string
+	 */
+	public const SAME_SITE_LAX = 'Lax';
+
+	/**
+	 * Cookies will only be sent in a first-party context and not
+	 * be sent along with requests initiated by third party websites.
+	 *
+	 * @link https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Set-Cookie/SameSite#strict
+	 *
+	 * @var string
+	 */
+	public const SAME_SITE_STRICT = 'Strict';
+
+	/**
+	 * Cookies will be sent in all contexts, i.e. in responses to
+	 * both first-party and cross-site requests.
+	 *
+	 * If SameSite=None is set, the cookie Secure attribute must
+	 * also be set (or the cookie will be blocked).
+	 *
+	 * @link https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Set-Cookie/SameSite#none
+	 *
+	 * @var string
+	 */
+	public const SAME_SITE_NONE = 'None';
+
+	/**
+	 * Cookie option keys
+	 *
+	 * @var string
+	 */
+	public const OPTION_KEY_PATH = 'path';
+	public const OPTION_KEY_DOMAIN = 'domain';
+	public const OPTION_KEY_SECURE = 'secure';
+	public const OPTION_KEY_HTTP_ONLY = 'httpOnly';
+	public const OPTION_KEY_SAMESITE = 'sameSite';
+
+	/**
+	 * Default cookie options
+	 *
+	 * @var array{
+	 *        path?: ?string,
+	 *        domain?: ?string,
+	 *        secure?: ?bool,
+	 *        httpOnly?: ?bool,
+	 *        sameSite?: ?string
+	 *      }
+	 */
+	protected static array $defaultOptions = [
+		self::OPTION_KEY_PATH      => '/',
+		self::OPTION_KEY_DOMAIN    => null,
+		self::OPTION_KEY_SECURE    => null,
+		self::OPTION_KEY_HTTP_ONLY => true,
+		self::OPTION_KEY_SAMESITE  => self::SAME_SITE_LAX,
+	];
+
+	/**
+	 * The cookie name
+	 *
+	 * @var string
+	 */
+	private string $name;
+
+	/**
+	 * The cookie value
+	 *
+	 * @var string
+	 */
+	private string $value;
+
+	/**
+	 * The cookie expiration date
+	 *
+	 * @var DateTimeInterface|null
+	 */
+	private ?DateTimeInterface $expires;
+
+	/**
+	 * The cookie options
+	 *
+	 * @var array{
+	 *        path?: ?string,
+	 *        domain?: ?string,
+	 *        secure?: ?bool,
+	 *        httpOnly?: ?bool,
+	 *        sameSite?: ?string
+	 *      }
+	 */
+	private array $options;
+
+	/**
+	 * Constructor of the class
+	 *
+	 * @param string $name
+	 * @param string $value
+	 * @param DateTimeInterface|null $expires
+	 * @param array{path?: ?string, domain?: ?string, secure?: ?bool, httpOnly?: ?bool, sameSite?: ?string} $options
+	 *
+	 * @throws InvalidHeaderValueException
+	 *         If one of the parameters isn't valid.
+	 */
+	public function __construct(string $name, string $value, ?DateTimeInterface $expires = null, array $options = [])
+	{
+		$this->validateCookieName($name);
+
+		if (isset($options[self::OPTION_KEY_PATH])) {
+			$this->validateCookieOption(self::OPTION_KEY_PATH, $options[self::OPTION_KEY_PATH]);
+		}
+
+		if (isset($options[self::OPTION_KEY_DOMAIN])) {
+			$this->validateCookieOption(self::OPTION_KEY_DOMAIN, $options[self::OPTION_KEY_DOMAIN]);
+		}
+
+		if (isset($options[self::OPTION_KEY_SAMESITE])) {
+			$this->validateCookieOption(self::OPTION_KEY_SAMESITE, $options[self::OPTION_KEY_SAMESITE]);
+		}
+
+		if ($value === '') {
+			$value = 'deleted';
+			$expires = new DateTimeImmutable('1 year ago');
+		}
+
+		$options += static::$defaultOptions;
+
+		$this->name = $name;
+		$this->value = $value;
+		$this->expires = $expires;
+		$this->options = $options;
+	}
+
+	/**
+	 * {@inheritdoc}
+	 */
+	public function getFieldName(): string
+	{
+		return 'Set-Cookie';
+	}
+
+	/**
+	 * {@inheritdoc}
+	 */
+	public function getFieldValue(): string
+	{
+		$name = rawurlencode($this->name);
+		$value = rawurlencode($this->value);
+		$result = sprintf('%s=%s', $name, $value);
+
+		if (isset($this->expires)) {
+			$result .= '; Expires=' . $this->formatDateTime($this->expires);
+			$result .= '; Max-Age=' . max($this->expires->getTimestamp() - time(), 0);
+		}
+
+		if (isset($this->options[self::OPTION_KEY_PATH])) {
+			$result .= '; Path=' . $this->options[self::OPTION_KEY_PATH];
+		}
+
+		if (isset($this->options[self::OPTION_KEY_DOMAIN])) {
+			$result .= '; Domain=' . $this->options[self::OPTION_KEY_DOMAIN];
+		}
+
+		if (isset($this->options[self::OPTION_KEY_SECURE]) && $this->options[self::OPTION_KEY_SECURE]) {
+			$result .= '; Secure';
+		}
+
+		if (isset($this->options[self::OPTION_KEY_HTTP_ONLY]) && $this->options[self::OPTION_KEY_HTTP_ONLY]) {
+			$result .= '; HttpOnly';
+		}
+
+		if (isset($this->options[self::OPTION_KEY_SAMESITE])) {
+			$result .= '; SameSite=' . $this->options[self::OPTION_KEY_SAMESITE];
+		}
+
+		return $result;
+	}
+
+	/**
+	 * Validates the given cookie name
+	 *
+	 * @param string $name
+	 *
+	 * @return void
+	 *
+	 * @throws InvalidHeaderValueException
+	 *         If the cookie name isn't valid.
+	 */
+	private function validateCookieName(string $name): void
+	{
+		if ('' === $name) {
+			throw new InvalidHeaderValueException('Cookie name cannot be empty');
+		}
+
+		// https://github.com/php/php-src/blob/02a5335b710aa36cd0c3108bfb9c6f7a57d40000/ext/standard/head.c#L93
+		if (strpbrk($name, "=,; \t\r\n\013\014") !== false) {
+			throw new InvalidHeaderValueException(sprintf(
+				'The cookie name "%s" contains prohibited characters',
+				$name
+			));
+		}
+	}
+
+	/**
+	 * Validates the given cookie option
+	 *
+	 * @param string $validKey
+	 * @param mixed $value
+	 *
+	 * @return void
+	 *
+	 * @throws InvalidHeaderValueException
+	 *         If the cookie option isn't valid.
+	 */
+	private function validateCookieOption(string $validKey, $value): void
+	{
+		if (!is_string($value)) {
+			throw new InvalidHeaderValueException(sprintf(
+				'The cookie option "%s" must be a string',
+				$validKey
+			));
+		}
+
+		// https://github.com/php/php-src/blob/02a5335b710aa36cd0c3108bfb9c6f7a57d40000/ext/standard/head.c#L103
+		// https://github.com/php/php-src/blob/02a5335b710aa36cd0c3108bfb9c6f7a57d40000/ext/standard/head.c#L108
+		if (strpbrk($value, ",; \t\r\n\013\014") !== false) {
+			throw new InvalidHeaderValueException(sprintf(
+				'The cookie option "%s" contains prohibited characters',
+				$validKey
+			));
+		}
+	}
 }

src/Header/ClearSiteDataHeader.php

Indentation +36 added lines, -36 removed lines

@@ -29,46 +29,46 @@
 class ClearSiteDataHeader extends Header
 {
 
-    /**
-     * @var list<string>
-     */
-    private array $directives;
+	/**
+	 * @var list<string>
+	 */
+	private array $directives;
 
-    /**
-     * Constructor of the class
-     *
-     * @param string ...$directives
-     *
-     * @throws InvalidHeaderValueException
-     *         If one of the directives isn't valid.
-     */
-    public function __construct(string ...$directives)
-    {
-        /** @var list<string> $directives */
+	/**
+	 * Constructor of the class
+	 *
+	 * @param string ...$directives
+	 *
+	 * @throws InvalidHeaderValueException
+	 *         If one of the directives isn't valid.
+	 */
+	public function __construct(string ...$directives)
+	{
+		/** @var list<string> $directives */
 
-        $this->validateQuotedString(...$directives);
+		$this->validateQuotedString(...$directives);
 
-        $this->directives = $directives;
-    }
+		$this->directives = $directives;
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    public function getFieldName(): string
-    {
-        return 'Clear-Site-Data';
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function getFieldName(): string
+	{
+		return 'Clear-Site-Data';
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    public function getFieldValue(): string
-    {
-        $segments = [];
-        foreach ($this->directives as $directive) {
-            $segments[] = sprintf('"%s"', $directive);
-        }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function getFieldValue(): string
+	{
+		$segments = [];
+		foreach ($this->directives as $directive) {
+			$segments[] = sprintf('"%s"', $directive);
+		}
 
-        return implode(', ', $segments);
-    }
+		return implode(', ', $segments);
+	}
 }

src/Header/KeepAliveHeader.php

Indentation +41 added lines, -41 removed lines

@@ -29,52 +29,52 @@
 class KeepAliveHeader extends Header
 {
 
-    /**
-     * @var array<string, string>
-     */
-    private array $parameters;
+	/**
+	 * @var array<string, string>
+	 */
+	private array $parameters;
 
-    /**
-     * Constructor of the class
-     *
-     * @param array<array-key, mixed> $parameters
-     *
-     * @throws InvalidHeaderValueParameterException
-     *         If the parameters aren't valid.
-     */
-    public function __construct(array $parameters = [])
-    {
-        $parameters = $this->validateParameters($parameters);
+	/**
+	 * Constructor of the class
+	 *
+	 * @param array<array-key, mixed> $parameters
+	 *
+	 * @throws InvalidHeaderValueParameterException
+	 *         If the parameters aren't valid.
+	 */
+	public function __construct(array $parameters = [])
+	{
+		$parameters = $this->validateParameters($parameters);
 
-        $this->parameters = $parameters;
-    }
+		$this->parameters = $parameters;
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    public function getFieldName(): string
-    {
-        return 'Keep-Alive';
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function getFieldName(): string
+	{
+		return 'Keep-Alive';
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    public function getFieldValue(): string
-    {
-        $segments = [];
-        foreach ($this->parameters as $name => $value) {
-            // the construction <foo=> isn't valid...
-            if ($value === '') {
-                $segments[] = $name;
-                continue;
-            }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function getFieldValue(): string
+	{
+		$segments = [];
+		foreach ($this->parameters as $name => $value) {
+			// the construction <foo=> isn't valid...
+			if ($value === '') {
+				$segments[] = $name;
+				continue;
+			}
 
-            $format = $this->isToken($value) ? '%s=%s' : '%s="%s"';
+			$format = $this->isToken($value) ? '%s=%s' : '%s="%s"';
 
-            $segments[] = sprintf($format, $name, $value);
-        }
+			$segments[] = sprintf($format, $name, $value);
+		}
 
-        return implode(', ', $segments);
-    }
+		return implode(', ', $segments);
+	}
 }

src/Header/EtagHeader.php

Indentation +31 added lines, -31 removed lines

@@ -28,39 +28,39 @@
 class EtagHeader extends Header
 {
 
-    /**
-     * @var string
-     */
-    private string $value;
+	/**
+	 * @var string
+	 */
+	private string $value;
 
-    /**
-     * Constructor of the class
-     *
-     * @param string $value
-     *
-     * @throws InvalidHeaderValueException
-     *         If the value isn't valid.
-     */
-    public function __construct(string $value)
-    {
-        $this->validateQuotedString($value);
+	/**
+	 * Constructor of the class
+	 *
+	 * @param string $value
+	 *
+	 * @throws InvalidHeaderValueException
+	 *         If the value isn't valid.
+	 */
+	public function __construct(string $value)
+	{
+		$this->validateQuotedString($value);
 
-        $this->value = $value;
-    }
+		$this->value = $value;
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    public function getFieldName(): string
-    {
-        return 'ETag';
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function getFieldName(): string
+	{
+		return 'ETag';
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    public function getFieldValue(): string
-    {
-        return sprintf('"%s"', $this->value);
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function getFieldValue(): string
+	{
+		return sprintf('"%s"', $this->value);
+	}
 }

src/Header/AllowHeader.php

Indentation +35 added lines, -35 removed lines

@@ -29,44 +29,44 @@
 class AllowHeader extends Header
 {
 
-    /**
-     * @var list<string>
-     */
-    private array $methods = [];
+	/**
+	 * @var list<string>
+	 */
+	private array $methods = [];
 
-    /**
-     * Constructor of the class
-     *
-     * @param string ...$methods
-     *
-     * @throws InvalidHeaderValueException
-     *         If one of the methods isn't valid.
-     */
-    public function __construct(string ...$methods)
-    {
-        /** @var list<string> $methods */
+	/**
+	 * Constructor of the class
+	 *
+	 * @param string ...$methods
+	 *
+	 * @throws InvalidHeaderValueException
+	 *         If one of the methods isn't valid.
+	 */
+	public function __construct(string ...$methods)
+	{
+		/** @var list<string> $methods */
 
-        $this->validateToken(...$methods);
+		$this->validateToken(...$methods);
 
-        // normalize the list of methods...
-        foreach ($methods as $method) {
-            $this->methods[] = strtoupper($method);
-        }
-    }
+		// normalize the list of methods...
+		foreach ($methods as $method) {
+			$this->methods[] = strtoupper($method);
+		}
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    public function getFieldName(): string
-    {
-        return 'Allow';
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function getFieldName(): string
+	{
+		return 'Allow';
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    public function getFieldValue(): string
-    {
-        return implode(', ', $this->methods);
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function getFieldValue(): string
+	{
+		return implode(', ', $this->methods);
+	}
 }