Completed
Push — master ( 465294...59d00e )
by Temitope
02:22
created

Middleware   A

Complexity

Total Complexity 4

Size/Duplication

Total Lines 21
Duplicated Lines 0 %

Coupling/Cohesion

Components 0
Dependencies 3

Importance

Changes 3
Bugs 2 Features 0
Metric Value
wmc 4
c 3
b 2
f 0
lcom 0
cbo 3
dl 0
loc 21
rs 10

1 Method

Rating   Name   Duplication   Size   Complexity  
A __invoke() 0 18 4
1
<?php
2
/**
3
 * @author   Temitope Olotin <[email protected]>
4
 * @license  <https://opensource.org/license/MIT> MIT
5
 */
6
7
namespace Laztopaz\EmojiRestfulAPI;
8
9
use Firebase\JWT\JWT;
10
use Psr\Http\Message\ResponseInterface as Response;
11
use Psr\Http\Message\ServerRequestInterface as Request;
12
13
class Middleware
14
{
15
    public function __invoke(Request $request, Response $response, $next)
16
    {
17
        $loadEnv = DatabaseConnection::loadEnv();
0 ignored issues
show
Bug introduced by
Are you sure the assignment to $loadEnv is correct as \Laztopaz\EmojiRestfulAP...seConnection::loadEnv() (which targets Laztopaz\EmojiRestfulAPI...seConnection::loadEnv()) seems to always return null.

This check looks for function or method calls that always return null and whose return value is assigned to a variable.

class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
$object = $a->getObject();

The method getObject() can return nothing but null, so it makes no sense to assign that value to a variable.

The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes.

Loading history...
Unused Code introduced by
$loadEnv is not used, you could remove the assignment.

This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently.

$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}

Both the $myVar assignment in line 1 and the $higher assignment in line 2 are dead. The first because $myVar is never used and the second because $higher is always overwritten for every possible time line.

Loading history...
18
        $authHeader = $request->getHeader('token');
19
        try {
20
            if (is_array($authHeader) && !empty($authHeader)) {
21
                $jwtoken = $authHeader[0];
22
                $secretKey = base64_decode(getenv('secret'));
23
                $jwt = json_decode($jwtoken, true);
24
                //decode the JWT using the key from config
25
                JWT::decode($jwt['jwt'], $secretKey, ['HS512']);
26
                return $next($request, $response);
27
            }
28
        } catch (\Exception $e) {
29
            return $response->withJson(['status' => $e->getMessage()], 401);
30
        }
31
        return $response->withJson(['status'], 401);
32
    }
33
}
34