App\Commands\Ngrok\Open::handle()

Complexity

Conditions	8
Paths	21

Size

Total Lines	48
Code Lines	30

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	72

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function handle(): void
    {
        $site = Site::resolveFromPathOrCurrentWorkingDirectory($this->argument('site'));

It seems like $this->argument('site') can also be of type array; however, parameter $path of App\Models\Site::resolveFromPathOrCurrentWorkingDirectory() does only seem to accept null|string, maybe add an additional type check?

        $wasSecure = false;

        if (! $site) {
            $this->error('No site at this location, and no site path provided.');
            exit();

Using exit here is not recommended.

        }

        // We're now checking that Porter is using the dns:set-host IP. If we don't,
        // ngrok requests will only resolve to 127.0.0.1 which is internal
        // to the ngrok container, and results in a useless 502 error.
        try {
            if (app(Mechanic::class)->isUsingDefaultHostAddress()) {
                $this->info('You need to use an alternative loopback address.');
                $this->info('Please run porter dns:set-host and review the documentation here: https://github.com/konsulting/porter#dns');
                exit();

Using exit here is not recommended.

            }
        } catch (UnableToRetrieveIP $e) {
            $this->info('Please run porter dns:flush and try again. You may need to give it a little while.');
            exit();

Using exit here is not recommended.

        }

        if ($site->secure) {
            $this->info('Removing SSL for site (required for free ngrok version)');
            $site->unsecure();
            $wasSecure = true;
        }

        app(Porter::class)->stop('ngrok');

        $tls = ' -bind-tls='.($wasSecure ? 'true' : 'false');
        $region = ' -region='. $this->option('region');
        $inspect = ' -inspect='.($this->option('no-inspection') ? 'false' : 'true');

        $this->dockerCompose
            ->runContainer('ngrok')
            ->append("ngrok http -host-header=rewrite{$region}{$tls}{$inspect} {$site->url}:80")
            ->interactive()
            ->perform();

        if ($wasSecure) {
            $this->info('Restoring SSL for site');
            $site->secure();
        }

        app(Porter::class)->stop('ngrok');
    }