padosoft / laravel-composer-security

src/ComposerSecurityCheckServiceProvider.php

Doc Comments +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
     /**
      * Get the services provided by the provider.
      *
-     * @return array
+     * @return string[]
      */
     public function provides()
     {

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 namespace Padosoft\LaravelComposerSecurity;
 
 use Illuminate\Support\ServiceProvider;
-use GuzzleHttp\Client;
 
 class ComposerSecurityCheckServiceProvider extends ServiceProvider
 {

Spacing +4 added lines, -4 removed lines

@@ -21,13 +21,13 @@ 
     public function boot()
     {
         $this->publishes([
-            __DIR__ . '/config/composer-security-check.php' => config_path('composer-security-check.php'),
+            __DIR__.'/config/composer-security-check.php' => config_path('composer-security-check.php'),
         ], 'config');
 
-        $this->loadViewsFrom(__DIR__ . '/views', 'composer-security-check');
+        $this->loadViewsFrom(__DIR__.'/views', 'composer-security-check');
 
         $this->publishes([
-            __DIR__ . '/views' => base_path('resources/views/vendor/composer-security-check'),
+            __DIR__.'/views' => base_path('resources/views/vendor/composer-security-check'),
         ]);
     }
 
@@ -39,7 +39,7 @@ 
     public function register()
     {
         $this->app['command.composer-security:check'] = $this->app->share(
-            function ($app) {
+            function($app) {
                 return new ComposerSecurityCheck(new client);
             }
         );

src/FileHelper.php

Spacing +3 added lines, -3 removed lines

@@ -13,12 +13,12 @@ 
      */
     public function findFiles($path, $fileName)
     {
-        if ($path=='') {
+        if ($path == '') {
             $path = base_path();
         }
 
         if (is_dir($path)) {
-            $path=str_finish($path, '/');
+            $path = str_finish($path, '/');
 
         }
         $path .= $fileName;
@@ -35,7 +35,7 @@ 
 
         $p = explode(",", str_replace('\\', '/', $path));
 
-        $pathList = array_map(function ($item) {
+        $pathList = array_map(function($item) {
             return str_finish($item, '/');
         },
             $p

src/config/composer-security-check.php

Indentation +1 added lines, -1 removed lines

@@ -12,4 +12,4 @@
     'mailFromName' => env('SECURITY_CHECK_MESSAGE_FROM_NAME', 'Info Example'),
     'mailViewName' => env('SECURITY_CHECK_MAIL_VIEW_NAME', 'composer-security-check::mail'),
     'logFilePath' => env('SECURITY_CHECK_LOG_FILE_PATH', storage_path().'/composersecurityCheck.log')
- );
+    );

src/SensiolabHelper.php

Doc Comments +2 added lines, -2 removed lines

@@ -129,7 +129,7 @@ 
     /**
      * @param $key
      * @param $vulnerability
-     * @param $tuttoOk
+     * @param boolean $tuttoOk
      * @return array
      */
     public function checkResponse($key, $vulnerability, $tuttoOk)
@@ -144,7 +144,7 @@ 
     }
 
     /**
-     * @param            $msg
+     * @param            string $msg
      * @param bool|false $error
      */
     private function addVerboseLog($msg, $error = false)

Unused Use Statements +1 added lines, -1 removed lines

@@ -8,8 +8,8 @@
 
 namespace Padosoft\LaravelComposerSecurity;
 
-use Illuminate\Console\Command;
 use GuzzleHttp\Client;
+use Illuminate\Console\Command;
 
 class SensiolabHelper
 {

Spacing +12 added lines, -12 removed lines

@@ -42,19 +42,19 @@ 
      */
     public function getSensiolabVulnerabilties($fileLock)
     {
-        $this->addVerboseLog('Send request to sensiolab: <info>' . $fileLock . '</info>');
+        $this->addVerboseLog('Send request to sensiolab: <info>'.$fileLock.'</info>');
 
-        $debug = false;//set to true to log into console output
+        $debug = false; //set to true to log into console output
         $headers = [
             //OPTIONS
             'allow_redirects' => [
-                'max' => 3,        // allow at most 10 redirects.
-                'strict' => true,      // use "strict" RFC compliant redirects.
-                'referer' => true,      // add a Referer header
+                'max' => 3, // allow at most 10 redirects.
+                'strict' => true, // use "strict" RFC compliant redirects.
+                'referer' => true, // add a Referer header
                 'protocols' => ['http', 'https'], // only allow http and https URLs
                 'track_redirects' => false
             ],
-            'connect_timeout' => 20,//Use 0 to wait connection indefinitely
+            'connect_timeout' => 20, //Use 0 to wait connection indefinitely
             'timeout' => 30, //Use 0 to wait response indefinitely
             'debug' => $debug,
             //HEADERS
@@ -76,17 +76,17 @@ 
             $responseBody = $iResponse->getBody()->getContents();
             $response = json_decode($responseBody, true);
         } catch (\GuzzleHttp\Exception\ClientException $e) {
-            $this->command->error("ClientException!\nMessage: " . $e->getMessage());
+            $this->command->error("ClientException!\nMessage: ".$e->getMessage());
             $colorTag = $this->getColorTagForStatusCode($e->getResponse()->getStatusCode());
-            $this->command->line("HTTP StatusCode: <{$colorTag}>" . $e->getResponse()->getStatusCode() . "<{$colorTag}>");
+            $this->command->line("HTTP StatusCode: <{$colorTag}>".$e->getResponse()->getStatusCode()."<{$colorTag}>");
             $this->printMessage($e->getResponse());
             $this->printMessage($e->getRequest());
         } catch (\GuzzleHttp\Exception\RequestException $e) {
-            $this->command->error("RequestException!\nMessage: " . $e->getMessage());
+            $this->command->error("RequestException!\nMessage: ".$e->getMessage());
             $this->printMessage($e->getRequest());
             if ($e->hasResponse()) {
                 $colorTag = $this->getColorTagForStatusCode($e->getResponse()->getStatusCode());
-                $this->command->line("HTTP StatusCode: <{$colorTag}>" . $e->getResponse()->getStatusCode() . "<{$colorTag}>");
+                $this->command->line("HTTP StatusCode: <{$colorTag}>".$e->getResponse()->getStatusCode()."<{$colorTag}>");
                 $this->printMessage($e->getResponse());
             }
         }
@@ -119,7 +119,7 @@ 
                 'advisories' => $data2["title"]
             ];
 
-            $this->addVerboseLog($data['name'] . " " . $data['version'] . " " . $data2["title"], true);
+            $this->addVerboseLog($data['name']." ".$data['version']." ".$data2["title"], true);
             $this->tableVulnerabilities[] = $dataTable;
         }
 
@@ -174,7 +174,7 @@ 
         $this->command->info("$type:");
         $headers = '';
         foreach ($message->getHeaders() as $name => $values) {
-            $headers .= $name . ': ' . implode(', ', $values) . "\r\n";
+            $headers .= $name.': '.implode(', ', $values)."\r\n";
         }
         $this->command->comment($headers);
         if ($type == 'REQUEST') {

src/ComposerSecurityCheck.php

Spacing +8 added lines, -8 removed lines

@@ -82,9 +82,9 @@ 
     private function hardWork($argument, $option)
     {
         $path = $argument['path'];
-        $this->line('path: <info>' . $path . '</info>.\nCheck composer.lock files...');
+        $this->line('path: <info>'.$path.'</info>.\nCheck composer.lock files...');
         $lockFiles = $this->findFilesComposerLock($path);
-        $this->line('Find <info>' . count($lockFiles) . '</info> composer.lock files.');
+        $this->line('Find <info>'.count($lockFiles).'</info> composer.lock files.');
 
         $this->tableVulnerabilities = [];
         $tuttoOk = true;
@@ -94,7 +94,7 @@ 
 
         foreach ($lockFiles as $fileLock) {
 
-            $this->line("Analizing <info>" . ($numLock + 1) . "</info> di <info>" . count($lockFiles) . "</info>");
+            $this->line("Analizing <info>".($numLock + 1)."</info> di <info>".count($lockFiles)."</info>");
 
             $tuttoOk = $this->checkFile($fileLock, $whitelist);
 
@@ -117,12 +117,12 @@ 
 
         $nomailok_bool = false;
 
-        if ($nomailok!='' && strtolower($nomailok)=='true') {
+        if ($nomailok != '' && strtolower($nomailok) == 'true') {
             $nomailok_bool = true;
         }
 
         //send email
-        if(!$nomailok_bool || !$tuttoOk) {
+        if (!$nomailok_bool || !$tuttoOk) {
             $this->sendEmail($mail, $tuttoOk);
         }
 
@@ -168,11 +168,11 @@ 
         $file = new FileHelper();
         $lockFiles = array();
         foreach ($file->adjustPath($path) as $item) {
-            $lockFiles = array_merge($lockFiles,$file->findFiles($item, 'composer.lock'));
+            $lockFiles = array_merge($lockFiles, $file->findFiles($item, 'composer.lock'));
         }
 
 
-        if(!is_array($lockFiles)){
+        if (!is_array($lockFiles)) {
             $lockFiles = array();
         }
 
@@ -205,7 +205,7 @@ 
         if (count($response) == 0) {
             return true;
         }
-        $this->error("Trovate " . count($response) . " vulnerabilita' in $fileLock");
+        $this->error("Trovate ".count($response)." vulnerabilita' in $fileLock");
 
         $tuttoOk = in_array(rtrim(str_replace('\\', '/', $fileLock), 'composer.lock'), $whitelist);
 

src/MailHelper.php

Spacing +3 added lines, -3 removed lines

@@ -43,10 +43,10 @@ 
      */
     public function sendEmail($tuttoOk, $mail, $vul)
     {
-        $soggetto=Config::get('composer-security-check.mailSubjectSuccess');
+        $soggetto = Config::get('composer-security-check.mailSubjectSuccess');
 
         if (!$tuttoOk) {
-            $soggetto=Config::get('composer-security-check.mailSubjetcAlarm');
+            $soggetto = Config::get('composer-security-check.mailSubjetcAlarm');
         }
 
         $validator = Validator::make(['email' => $mail], [
@@ -61,7 +61,7 @@ 
         Mail::send(
             Config::get('composer-security-check.mailViewName'),
             ['vul' => $vul],
-            function ($message) use ($mail, $soggetto) {
+            function($message) use ($mail, $soggetto) {
                 $message->from(
                     Config::get('composer-security-check.mailFrom'),
                     Config::get('composer-security-check.mailFromName')