mushti / request

src/Components/Authorization.php

<?php

namespace Raptor\Request\Components;

class Authorization
{
    /**
     * Authorization type.
     *
     * @var string
     */
    protected $type;

    /**
     * Authorization username.
     *
     * @var string
     */
    protected $username;

    /**
     * Authorization password.
     *
     * @var string
     */
    protected $password;

    /**
     * Authorization token.
     *
     * @var string
     */
    protected $token;

    /**
     * Authorization header-field.
     *
     * @var string
     */
    protected $field;

    /**
     * Get authorization type.
     *
     * @return string
     */
    public function type()
    {
        if ($this->type !== null) {
            return $this->type;
        }
        if ($this->field() === false || count($string = explode(' ', $this->field)) < 2) {
            return $this->type = false;

The property $type was declared of type string, but false is of type false. Maybe add a type cast?

The expression return $this->type = false returns the type false which is incompatible with the documented return type string.

        }
        return $this->type = $string[0];
    }

    /**
     * Get authorization username.
     *
     * @return string
     */
    public function username()
    {
        if ($this->username !== null) {
            return $this->username;
        }

        if (isset($_SERVER['PHP_AUTH_USER'])) {
            return $this->username = $_SERVER['PHP_AUTH_USER'];
        }

        if (strtolower($this->type()) == 'basic') {
            // Decode AUTHORIZATION header into PHP_AUTH_USER when authorization header is basic
            $token = explode(':', base64_decode($this->token()), 1);
            if (count($token) == 1) {
                return $this->username = $_SERVER['PHP_AUTH_USER'] = $token[0];
            }
        }

        return $this->username = false;

The property $username was declared of type string, but false is of type false. Maybe add a type cast?

The expression return $this->username = false returns the type false which is incompatible with the documented return type string.

    }

    /**
     * Get authorization password.
     *
     * @return string
     */
    public function password()
    {
        if ($this->password !== null) {
            $this->password;
        }
        
        if (isset($_SERVER['PHP_AUTH_USER'])) {
            return $this->password = isset($_SERVER['PHP_AUTH_PW']) ? $_SERVER['PHP_AUTH_PW'] : '';
        }

        if (strtolower($this->type()) == 'basic') {
            // Decode AUTHORIZATION header into PHP_AUTH_PW when authorization header is basic
            $token = explode(':', base64_decode($this->token()), 2);
            if (count($token) == 2) {
                return $this->password = $_SERVER['PHP_AUTH_PW'] = $token[1];
            }
        }
     
        return $this->password = false;

The property $password was declared of type string, but false is of type false. Maybe add a type cast?

The expression return $this->password = false returns the type false which is incompatible with the documented return type string.

    }

    /**
     * Get authorization token.
     *
     * @return string
     */
    public function token()
    {
        if ($this->token !== null) {
            return $this->token;
        }
        if ($this->field() === false || count($string = explode(' ', $this->field)) != 2) {
            return $this->token = false;

The expression return $this->token = false returns the type false which is incompatible with the documented return type string.

The property $token was declared of type string, but false is of type false. Maybe add a type cast?

        }
        return $this->token = $string[1];
    }

    /**
     * Get authorization header-field.
     *
     * @return  string
     */
    public function field()
    {
        if ($this->field !== null) {
            return $this->field;
        }

        // HTTP_AUTHORIZATION
        if (isset($_SERVER['HTTP_AUTHORIZATION'])) {
            $this->field = $_SERVER['HTTP_AUTHORIZATION'];
        }
        // REDIRECT_HTTP_AUTHORIZATION
        elseif (isset($_SERVER['REDIRECT_HTTP_AUTHORIZATION'])) {
            $this->field = $_SERVER['REDIRECT_HTTP_AUTHORIZATION'];
        }
        if ($this->field) {
            if (empty($_SERVER['PHP_AUTH_DIGEST']) && (0 === stripos($this->field, 'digest '))) {
                // In some circumstances PHP_AUTH_DIGEST needs to be set.
                return $_SERVER['PHP_AUTH_DIGEST'] = $this->field;
            }
        }
        // PHP_AUTH_USER
        if (isset($_SERVER['PHP_AUTH_USER'])) {
            return $this->field = 'Basic '.base64_encode($_SERVER['PHP_AUTH_USER'].':'.$_SERVER['PHP_AUTH_PW']);
        }
        // PHP_AUTH_DIGEST
        if (isset($_SERVER['PHP_AUTH_DIGEST'])) {
            return $this->field = $_SERVER['PHP_AUTH_DIGEST'];
        }

        return $this->field = false;

The expression return $this->field = false returns the type false which is incompatible with the documented return type string.

The property $field was declared of type string, but false is of type false. Maybe add a type cast?

    }
}