jkocik / laravel-profiler

Profiler for Laravel Framework
MIT License
188 stars 14 forks source link

fix: make artisan command package:discover turn off for processors by default. #2

Closed chrisLeeTW closed 4 years ago

chrisLeeTW commented 5 years ago

This pull request will make artisan command package:discover bypass for the processors.

Original config will make curl exception during the composer install if laravel-profiler is not started.

   GuzzleHttp\Exception\ConnectException  : cURL error 6: Could not resolve: profiler (Domain name not found) (see http://curl.haxx.se/libcurl/c/libcurl-errors.html)

  at /Users/chris_lee/WorkZone/GitZone/GetokenCode/revolution/revolution-backend/vendor/guzzlehttp/guzzle/src/Handler/CurlFactory.php:185
    181|         );
    182|
    183|         // Create a connection exception if it was a specific error code.
    184|         $error = isset($connectionErrors[$easy->errno])
  > 185|             ? new ConnectException($message, $easy->request, null, $ctx)
    186|             : new RequestException($message, $easy->request, $easy->response, null, $ctx);
    187|
    188|         return \GuzzleHttp\Promise\rejection_for($error);
    189|     }

  Exception trace:

  1   GuzzleHttp\Handler\CurlFactory::createRejection(Object(GuzzleHttp\Handler\EasyHandle))
      /Users/chris_lee/WorkZone/GitZone/GetokenCode/revolution/revolution-backend/vendor/guzzlehttp/guzzle/src/Handler/CurlFactory.php:149

  2   GuzzleHttp\Handler\CurlFactory::finishError(Object(GuzzleHttp\Handler\CurlHandler), Object(GuzzleHttp\Handler\EasyHandle), Object(GuzzleHttp\Handler\CurlFactory))
      /Users/chris_lee/WorkZone/GitZone/GetokenCode/revolution/revolution-backend/vendor/guzzlehttp/guzzle/src/Handler/CurlFactory.php:102

  Please use the argument -v to see more details.
coveralls commented 5 years ago

Pull Request Test Coverage Report for Build 63


Totals Coverage Status
Change from base Build 59: 0.0%
Covered Lines: 820
Relevant Lines: 820

💛 - Coveralls
jkocik commented 4 years ago

Thanks for that PR but I don't want to exclude any framework actions by default.