chillerlan\Traits\Crypto\SignedMessage

Complexity

Total Complexity

2

Size/Duplication

Total Lines	20
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	open()	0	8	1
A	create()	0	8	1

<?php
/**
 * Class SignedMessage
 *
 * @filesource   SignedMessage.php
 * @created      25.01.2018
 * @package      chillerlan\Traits\Crypto
 * @author       Smiley <smiley@chillerlan.net>
 * @copyright    2018 Smiley
 * @license      MIT
 */

namespace chillerlan\Traits\Crypto;

class SignedMessage extends CryptoBox{

	public function create(string $message):CryptoBoxInterface{
		$this->checkKeypair(SODIUM_CRYPTO_SIGN_SECRETKEYBYTES);

		$this->box = sodium_crypto_sign($this->checkMessage($message), $this->keypair->secret);

Accessing secret on the interface chillerlan\Traits\Crypto\CryptoKeyInterface suggest that you code against a concrete implementation. How about adding an instanceof check?

		sodium_memzero($message);

		return $this;
	}

	public function open(string $box_bin):CryptoBoxInterface{
		$this->checkKeypair(null, SODIUM_CRYPTO_SIGN_PUBLICKEYBYTES);

		$this->message = sodium_crypto_sign_open($box_bin, $this->keypair->public);

Accessing public on the interface chillerlan\Traits\Crypto\CryptoKeyInterface suggest that you code against a concrete implementation. How about adding an instanceof check?

		sodium_memzero($box_bin);

		return $this;
	}

}