OpenCrypt::__construct() - Code Metrics - Inspection of "Merge pull request #3 from martinusso/opencrypt" - martinusso/opencrypt - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 2340f1...c1e1ba )

by Breno

created 2017-10-24 10:38 UTC

OpenCrypt::__construct() A

↳ Parent: OpenCrypt

Complexity

Conditions	1
Paths	1

Size

Total Lines	9
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	4
CRAP Score	1

Importance

Changes

Metric	Value
dl	0
loc	9
ccs	4
cts	4
cp	1
rs	9.6666
c	0
b	0
f	0
cc	1
eloc	5
nc	1
nop	2
crap	1

<?php

namespace OpenCrypt;

class OpenCrypt
{
    private $encryptMethod = "AES-256-CBC";
    private $secretKey;
    private $secretIV;

    public function __construct(
        string $secretKey,
        string $secretIV
    ) {
        // hash
        $this->secretKey = hash('sha256', $secretKey);
        // iv - encrypt method AES-256-CBC expects 16 bytes - else you will get a warning
        $this->secretIV = substr(hash('sha256', $secretIV), 0, 16);
    }

    function encrypt($value) {

        $output = openssl_encrypt(
            $value,
            $this->encryptMethod,
            $this->secretKey,
            0,
            $this->secretIV
        );
        return base64_encode($output);
    }

    function decrypt($value) {

        return openssl_decrypt(
            base64_decode($value),
            $this->encryptMethod,
            $this->secretKey,
            0,
            $this->secretIV
        );
    }
}


1		<?php
2
3		namespace OpenCrypt;
4
5		class OpenCrypt
6		{
7		private $encryptMethod = "AES-256-CBC";
8		private $secretKey;
9		private $secretIV;
10
11	1	public function __construct(
12		string $secretKey,
13		string $secretIV
14		) {
15		// hash
16	1	$this->secretKey = hash('sha256', $secretKey);
17		// iv - encrypt method AES-256-CBC expects 16 bytes - else you will get a warning
18	1	$this->secretIV = substr(hash('sha256', $secretIV), 0, 16);
19	1	}
20
21	1	function encrypt($value) {
		0 ignored issues – show Best Practice introduced 2017-10-23 18:52 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
22	1	$output = openssl_encrypt(
23	1	$value,
24	1	$this->encryptMethod,
25	1	$this->secretKey,
26	1	0,
27	1	$this->secretIV
28		);
29	1	return base64_encode($output);
30		}
31
32	1	function decrypt($value) {
		0 ignored issues – show Best Practice introduced 2017-10-23 18:52 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
33	1	return openssl_decrypt(
34	1	base64_decode($value),
35	1	$this->encryptMethod,
36	1	$this->secretKey,
37	1	0,
38	1	$this->secretIV
39		);
40		}
41		}
42

martinusso / opencrypt

Push — master ( 2340f1...c1e1ba )

OpenCrypt::__construct() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like