Spatie\UptimeMonitor\Commands\CreateMonitor::handle()

Complexity

Conditions	5
Paths	6

Size

Total Lines

24

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function handle()
    {
        $url = Url::fromString($this->argument('url'));

It seems like $this->argument('url') targeting Illuminate\Console\Concerns\InteractsWithIO::argument() can also be of type array or null; however, Spatie\Url\Url::fromString() does only seem to accept string, maybe add an additional type check?

        if (! in_array($url->getScheme(), ['http', 'https'])) {
            if ($scheme = $this->choice("Which protocol needs to be used for checking `{$url}`?", [1 => 'https', 2 => 'http'], 1)) {
                $url = $url->withScheme($scheme);

It seems like $scheme defined by $this->choice("Which protocol needs to be used for checking `{$url}`?", array(1 => 'https', 2 => 'http'), 1) on line 19 can also be of type array; however, Spatie\Url\Url::withScheme() does only seem to accept string, maybe add an additional type check?

            }
        }

        if ($this->confirm('Should we look for a specific string on the response?')) {
            $lookForString = $this->ask('Which string?');
        }

        $monitor = Monitor::create([
            'url' => trim($url, '/'),
            'look_for_string' => $lookForString ?? '',
            'uptime_check_method' => isset($lookForString) ? 'get' : 'head',
            'certificate_check_enabled' => $url->getScheme() === 'https',
            'uptime_check_interval_in_minutes' => config('uptime-monitor.uptime_check.run_interval_in_minutes'),
        ]);

        $this->warn("{$monitor->url} will be monitored!");
    }
}