Knp\FriendlyContexts\Http\Security\HttpExtension

Complexity

Total Complexity

5

Size/Duplication

Total Lines	35
Duplicated Lines	34.29 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	secureClient()	0	3	1
A	secureRequest()	12	12	3

<?php

namespace Knp\FriendlyContexts\Http\Security;

use Guzzle\Http\Client;
use Guzzle\Http\Message\Request;
use Knp\FriendlyContexts\Builder\RequestBuilder;

class HttpExtension extends HttpBasicExtension implements SecurityExtensionInterface

The class Knp\FriendlyContexts\Http\Security\HttpBasicExtension has been deprecated with message: use HttpExtension. Will be removed in v1.0.0

{
    protected $schema;

    /**
     * @param string $scheme (allowed values: basic, digest, ntlm, any)
     */
    public function __construct($scheme = 'basic')
    {
        $this->scheme = $scheme;

The property scheme does not exist. Did you maybe forget to declare it?

    }

    /**
     * {@inheritdoc}
     */
    public function secureClient(Client $client, RequestBuilder $builder)
    {
    }

    /**
     * {@inheritdoc}
     */
    public function secureRequest(Request $request, RequestBuilder $builder)

This method seems to be duplicated in your project.

    {
        $credentials = $builder->getCredentials();

        if (!isset($credentials['username']) || !isset($credentials['password'])) {
            throw new \RuntimeException(
                'You must specified a "username" and a "password" for the http basic authentication.'
            );
        }

        $request->setAuth($credentials['username'], $credentials['password'], $this->scheme);
    }
}