BitWasp\Trezor\Device\Command\PingService

Complexity

Total Complexity

14

Size/Duplication

Total Lines	46
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
C	call()	0	44	14

<?php

declare(strict_types=1);

namespace BitWasp\Trezor\Device\Command;

use BitWasp\Trezor\Bridge\Session;
use BitWasp\Trezor\Device\Exception\IncorrectNonceException;
use BitWasp\Trezor\Device\Message;
use BitWasp\Trezor\Device\UserInput\CurrentPassphraseInputInterface;
use BitWasp\Trezor\Device\UserInput\CurrentPinInputInterface;
use BitWasp\TrezorProto\ButtonRequest;
use BitWasp\TrezorProto\ButtonRequestType;
use BitWasp\TrezorProto\PassphraseRequest;
use BitWasp\TrezorProto\Ping;
use BitWasp\TrezorProto\PinMatrixRequest;
use BitWasp\TrezorProto\Success;

class PingService extends DeviceService
{
    public function call(
        Session $session,
        Ping $ping,
        CurrentPinInputInterface $pinInput = null,
        CurrentPassphraseInputInterface $passphraseInput = null
    ): Success {
        if ($ping->hasPinProtection() && $ping->getPinProtection() && $pinInput === null) {
            throw new \InvalidArgumentException("Missing pin input");
        }

        if ($ping->hasPassphraseProtection() && $ping->getPassphraseProtection() && $passphraseInput === null) {
            throw new \InvalidArgumentException("Missing passphrase input");
        }

        $proto = $session->sendMessage(Message::ping($ping));
        if ($proto instanceof ButtonRequest) {
            // allow user to accept with the button
            $proto = $session->sendMessage($this->confirmWithButton($proto, ButtonRequestType::ButtonRequest_ProtectCall_VALUE));
        }

        if ($ping->hasPinProtection()) {
            // allow user to accept with their pin
            if ($proto instanceof PinMatrixRequest) {
                $proto = $session->sendMessage($this->provideCurrentPin($proto, $pinInput));

It seems like $pinInput can also be of type null; however, parameter $currentPinInput of BitWasp\Trezor\Device\Command\DeviceService::provideCurrentPin() does only seem to accept BitWasp\Trezor\Device\UserInput\CurrentPinInputInterface, maybe add an additional type check?

            }
        }

        if ($ping->hasPassphraseProtection()) {
            // allow user to accept with their passphrase
            if ($proto instanceof PassphraseRequest) {
                $proto = $session->sendMessage($this->provideCurrentPassphrase($passphraseInput));

It seems like $passphraseInput can also be of type null; however, parameter $passphraseInput of BitWasp\Trezor\Device\Command\DeviceService::provideCurrentPassphrase() does only seem to accept BitWasp\Trezor\Device\UserInput\CurrentPassphraseInputInterface, maybe add an additional type check?

            }
        }

        if (!($proto instanceof Success)) {
            throw new \RuntimeException("Unexpected response, expecting Success, got " . get_class($proto));
        }

        /** @var Success $proto */
        if (!hash_equals($proto->getMessage(), $ping->getMessage())) {
            throw new IncorrectNonceException("Nonce returned by device was incorrect");
        }

        return $proto;
    }
}