[3.x] Fix telescope.domain usage.

[finagin]

// config/telescope.php
return [
    'domain' => 'telescope.example.com',
    'path' => null,
];

If you use domain without path handlingApprovedRequest always return false because config('telescope.path').'*' is '*'
I propose check domain before check path.

[taylorotwell]

Why is this an "or" condition?

[finagin]

Because if this is a separate domain for telescope, we should not keep a record, if the domain is not separate, we check the paths

[taylorotwell]

Only needs to be sent to current branch.

[finagin]

Only needs to be sent to current branch.

Hey @taylorotwell!
I ran into this problem on a project with laravel 6 and I need these changes there because we cannot update laravel yet.
These changes don't break anything

[driesvints]

@finagin we only maintain the latest versions of our packages. Please upgrade to Laravel 8

[finagin]

@finagin we only maintain the latest versions of our packages. Please upgrade to Laravel 8

Laravel 6 is LTS right?
Branch telescope 3.x exists?
I think there is no problem to merge these changes

[driesvints]

Laravel 6 is LTS but the first party packages aren't. You can find all that info in our support policy https://laravel.com/docs/8.x/releases#support-policy

[finagin]

Sorry, but then I don't understand why the 3.x branch is needed instead of 3.0? Here and at Horizon.
Tell me if it's not difficult.