[5.3] Change server variables when bootstrapping the app request from artisan.

[j3j5]

This change fixes #14139 .

I am not sure where would tests for this belong. Should they go on tests/Routing/RoutingUrlGeneratorTest.php or on tests\Console\ConsoleApplicationTest.php? I think the former is better but I'd like to have somebody else opinion.

[taylorotwell]

I'm not able to even recreate this: http://d.pr/i/1bqOj

[vlakoff]

A better fix in SetRequestForConsole might be:

    public function bootstrap(Application $app)
    {
        $url = $app->make('config')->get('app.url', 'http://localhost');

        // Add this line:
        $app->make('url')->forceRootUrl($url);

        $app->instance('request', Request::create($url, 'GET', [], [], [], $_SERVER));
    }

The user is still able to call again forceRootUrl later if he wishes so.

Alternatively, this could be added to RoutingServiceProvider:

if ($app->runningInConsole()) {
    $url->forceRootUrl($app['config']->get('app.url', 'http://localhost'));
}

Though in both suggestions, we're introducing new dependencies into the service providers:

1. 'url' into SetRequestForConsole
2. 'config' into RoutingServiceProvider / 'url'