Trucker\Requests\Auth\BasicAuthenticator

Complexity

Total Complexity

2

Size/Duplication

Total Lines	31
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	authenticateRequest()	0	5	1
A	__construct()	0	3	1

<?php

namespace Trucker\Requests\Auth;

use Guzzle\Http\Message\Request;
use Illuminate\Container\Container;
use Trucker\Facades\Config;

class BasicAuthenticator implements AuthenticationInterface
{
    /**
     * The IoC Container.
     *
     * @var Container
     */
    protected $app;

    /**
     * Constructor, likely never called in implementation
     * but rather through the Factory.
     *
     * @param Container $app
     */
    public function __construct(Container $app)
    {
        $this->app = $app;
    }

    /**
     * Function to add the necessary authentication
     * to the request.
     *
     * @param Request $request Request passed by reference
     */
    public function authenticateRequest($request)
    {
        $username = Config::get('auth.basic.username');

The method get() does not exist on Trucker\Facades\Config. Since you implemented __callStatic, consider adding a @method annotation.

        $password = Config::get('auth.basic.password');
        $request->setAuth($username, $password);
    }
}