Signature::setHashingAlgorithm() - Code Metrics - Inspection of "Changed Bitrix24 authorization method" - Lusitanian/PHPoAuthLib - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#483)

unknown

created 2016-06-14 05:39 UTC

Signature::setHashingAlgorithm() A

↳ Parent: Signature

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	4
rs	10
c	0
b	0
f	0
cc	1
eloc	2
nc	1
nop	1

<?php

namespace OAuth\OAuth1\Signature;

use OAuth\Common\Consumer\CredentialsInterface;
use OAuth\Common\Http\Uri\UriInterface;
use OAuth\OAuth1\Signature\Exception\UnsupportedHashAlgorithmException;

class Signature implements SignatureInterface
{
    /**
     * @var Credentials
     */
    protected $credentials;

    /**
     * @var string
     */
    protected $algorithm;

    /**
     * @var string
     */
    protected $tokenSecret = null;

    /**
     * @param CredentialsInterface $credentials
     */
    public function __construct(CredentialsInterface $credentials)
    {
        $this->credentials = $credentials;

    }

    /**
     * @param string $algorithm
     */
    public function setHashingAlgorithm($algorithm)
    {
        $this->algorithm = $algorithm;
    }

    /**
     * @param string $token
     */
    public function setTokenSecret($token)
    {
        $this->tokenSecret = $token;
    }

    /**
     * @param UriInterface $uri
     * @param array        $params
     * @param string       $method
     *
     * @return string
     */
    public function getSignature(UriInterface $uri, array $params, $method = 'POST')
    {
        parse_str($uri->getQuery(), $queryStringData);

        foreach (array_merge($queryStringData, $params) as $key => $value) {
            $signatureData[rawurlencode($key)] = rawurlencode($value);
foreach ($collection as $item) {
    $myArray['foo'] = $item->getFoo();

    if ($item->hasBar()) {
        $myArray['bar'] = $item->getBar();
    }

    // do something with $myArray
}
        }

        ksort($signatureData);

        // determine base uri
        $baseUri = $uri->getScheme() . '://' . $uri->getRawAuthority();

        if ('/' === $uri->getPath()) {
            $baseUri .= $uri->hasExplicitTrailingHostSlash() ? '/' : '';
        } else {
            $baseUri .= $uri->getPath();
        }

        $baseString = strtoupper($method) . '&';
        $baseString .= rawurlencode($baseUri) . '&';
        $baseString .= rawurlencode($this->buildSignatureDataString($signatureData));

        return base64_encode($this->hash($baseString));
    }

    /**
     * @param array $signatureData
     *
     * @return string
     */
    protected function buildSignatureDataString(array $signatureData)
    {
        $signatureString = '';
        $delimiter = '';
        foreach ($signatureData as $key => $value) {
            $signatureString .= $delimiter . $key . '=' . $value;

            $delimiter = '&';
        }

        return $signatureString;
    }

    /**
     * @return string
     */
    protected function getSigningKey()
    {
        $signingKey = rawurlencode($this->credentials->getConsumerSecret()) . '&';
        if ($this->tokenSecret !== null) {
            $signingKey .= rawurlencode($this->tokenSecret);
        }

        return $signingKey;
    }

    /**
     * @param string $data
     *
     * @return string
     *
     * @throws UnsupportedHashAlgorithmException
     */
    protected function hash($data)
    {
        switch (strtoupper($this->algorithm)) {
            case 'HMAC-SHA1':
                return hash_hmac('sha1', $data, $this->getSigningKey(), true);
            default:
                throw new UnsupportedHashAlgorithmException(
                    'Unsupported hashing algorithm (' . $this->algorithm . ') used.'
                );
        }
    }
}


1			<?php
2
3			namespace OAuth\OAuth1\Signature;
4
5			use OAuth\Common\Consumer\CredentialsInterface;
6			use OAuth\Common\Http\Uri\UriInterface;
7			use OAuth\OAuth1\Signature\Exception\UnsupportedHashAlgorithmException;
8
9			class Signature implements SignatureInterface
10			{
11			/**
12			* @var Credentials
13			*/
14			protected $credentials;
15
16			/**
17			* @var string
18			*/
19			protected $algorithm;
20
21			/**
22			* @var string
23			*/
24			protected $tokenSecret = null;
25
26			/**
27			* @param CredentialsInterface $credentials
28			*/
29			public function __construct(CredentialsInterface $credentials)
30			{
31			$this->credentials = $credentials;
			0 ignored issues – show Documentation Bug introduced 2015-11-21 18:34 UTC by Report Bug Copy Issue Report It seems like `$credentials` of type `object<OAuth\Common\Cons...r\CredentialsInterface>` is incompatible with the declared type `object<OAuth\OAuth1\Signature\Credentials>` of property `$credentials`. Our type inference engine has found an assignment to a property that is incompatible with the declared type of that property. Either this assignment is in error or the assigned type should be added to the documentation/type hint for that property.. Loading history...
32			}
33
34			/**
35			* @param string $algorithm
36			*/
37			public function setHashingAlgorithm($algorithm)
38			{
39			$this->algorithm = $algorithm;
40			}
41
42			/**
43			* @param string $token
44			*/
45			public function setTokenSecret($token)
46			{
47			$this->tokenSecret = $token;
48			}
49
50			/**
51			* @param UriInterface $uri
52			* @param array $params
53			* @param string $method
54			*
55			* @return string
56			*/
57			public function getSignature(UriInterface $uri, array $params, $method = 'POST')
58			{
59			parse_str($uri->getQuery(), $queryStringData);
60
61			foreach (array_merge($queryStringData, $params) as $key => $value) {
62			$signatureData[rawurlencode($key)] = rawurlencode($value);
			0 ignored issues – show Coding Style Comprehensibility introduced 2015-11-21 18:34 UTC by Report Bug Copy Issue Report `$signatureData` was never initialized. Although not strictly required by PHP, it is generally a good practice to add `$signatureData = array();` before regardless. Adding an explicit array definition is generally preferable to implicit array definition as it guarantees a stable state of the code. Let’s take a look at an example: foreach ($collection as $item) { $myArray['foo'] = $item->getFoo(); if ($item->hasBar()) { $myArray['bar'] = $item->getBar(); } // do something with $myArray } As you can see in this example, the array `$myArray` is initialized the first time when the foreach loop is entered. You can also see that the value of the `bar` key is only written conditionally; thus, its value might result from a previous iteration. This might or might not be intended. To make your intention clear, your code more readible and to avoid accidental bugs, we recommend to add an explicit initialization `$myArray = array()` either outside or inside the foreach loop. Loading history...
63			}
64
65			ksort($signatureData);
66
67			// determine base uri
68			$baseUri = $uri->getScheme() . '://' . $uri->getRawAuthority();
69
70			if ('/' === $uri->getPath()) {
71			$baseUri .= $uri->hasExplicitTrailingHostSlash() ? '/' : '';
72			} else {
73			$baseUri .= $uri->getPath();
74			}
75
76			$baseString = strtoupper($method) . '&';
77			$baseString .= rawurlencode($baseUri) . '&';
78			$baseString .= rawurlencode($this->buildSignatureDataString($signatureData));
79
80			return base64_encode($this->hash($baseString));
81			}
82
83			/**
84			* @param array $signatureData
85			*
86			* @return string
87			*/
88			protected function buildSignatureDataString(array $signatureData)
89			{
90			$signatureString = '';
91			$delimiter = '';
92			foreach ($signatureData as $key => $value) {
93			$signatureString .= $delimiter . $key . '=' . $value;
94
95			$delimiter = '&';
96			}
97
98			return $signatureString;
99			}
100
101			/**
102			* @return string
103			*/
104			protected function getSigningKey()
105			{
106			$signingKey = rawurlencode($this->credentials->getConsumerSecret()) . '&';
107			if ($this->tokenSecret !== null) {
108			$signingKey .= rawurlencode($this->tokenSecret);
109			}
110
111			return $signingKey;
112			}
113
114			/**
115			* @param string $data
116			*
117			* @return string
118			*
119			* @throws UnsupportedHashAlgorithmException
120			*/
121			protected function hash($data)
122			{
123			switch (strtoupper($this->algorithm)) {
124			case 'HMAC-SHA1':
125			return hash_hmac('sha1', $data, $this->getSigningKey(), true);
126			default:
127			throw new UnsupportedHashAlgorithmException(
128			'Unsupported hashing algorithm (' . $this->algorithm . ') used.'
129			);
130			}
131			}
132			}
133

Lusitanian / PHPoAuthLib

Pull Request — master (#483)

Signature::setHashingAlgorithm() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like