LE_ACME2\Response\Authorization\AbstractAuthorization

Complexity

Total Complexity

3

Size/Duplication

Total Lines	33
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	2
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	2	1
A	_isValid()	0	7	2

<?php

namespace LE_ACME2\Response\Authorization;

use LE_ACME2\Response\AbstractResponse;

use LE_ACME2\Connector\RawResponse;
use LE_ACME2\Exception;

class AbstractAuthorization extends AbstractResponse {

    // Status from RFC 8555 (7.1.4), version: March 2019

    const STATUS_PENDING = 'pending';
    const STATUS_VALID = 'valid';
    const STATUS_INVALID = 'invalid';
    const STATUS_DEACTIVATED = 'deactivated';
    const STATUS_EXPIRED = 'expired';
    const STATUS_REVOKED = 'revoked';

    /**
     * AbstractAuthorization constructor.
     * @param RawResponse $raw
     * @throws Exception\InvalidResponse
     * @throws Exception\RateLimitReached
     * @throws Exception\ExpiredAuthorization
     */
    public function __construct(RawResponse $raw) {
        parent::__construct($raw);
    }

    /**
     * @return bool
     * @throws Exception\ExpiredAuthorization
     */
    protected function _isValid() : bool {

        if($this->_preg_match_headerLine('/^HTTP\/.* 404/i') !== null) {

Are you sure the usage of $this->_preg_match_headerLine('/^HTTP\/.* 404/i') targeting LE_ACME2\Response\AbstractResponse::_preg_match_headerLine() seems to always return null.

The condition $this->_preg_match_headerLine('/^HTTP\/.* 404/i') !== null is always false.

            throw new Exception\ExpiredAuthorization();
        }

        return parent::_isValid();
    }
}