Merge pull request #14 from bilfeldt/fix/duration

Rename field 'time' to 'duration'

Author: bilfeldt

database/factories/RequestLogFactory.php

@@ -24,7 +24,7 @@ public function definition()
             'payload' => $this->getPayload(),
             'response_headers' => $this->getHeaders(),
             'response_body' => $this->faker->sentence(),
-            'time' => $this->faker->numberBetween(100, 30000),
+            'duration' => $this->faker->numberBetween(100, 30000),
             'memory' => $this->faker->randomFloat(1, 1, 100),
         ];
     }

database/migrations/create_request_logs_table.php.stub

@@ -24,7 +24,7 @@ return new class extends Migration
             $table->json('payload')->nullable();
             $table->json('response_headers')->nullable();
             $table->json('response_body')->nullable();
-            $table->unsignedMediumInteger('time')->nullable(); // ms
+            $table->unsignedMediumInteger('duration')->nullable(); // ms
             $table->float('memory')->nullable(); // MB
             $table->timestamps();
 

src/Contracts/RequestLoggerInterface.php

@@ -9,13 +9,13 @@ interface RequestLoggerInterface
     /**
      * @param  Request  $request
      * @param  \Illuminate\Http\Response|\Illuminate\Http\JsonResponse|\Illuminate\Http\RedirectResponse|\Symfony\Component\HttpFoundation\Response  $response
-     * @param  int|null  $time
+     * @param  int|null  $duration
      * @param  int|null  $memory
      */
     public function log(
         Request $request,
         $response,
-        ?int $time = null,
+        ?int $duration = null,
         ?int $memory = null
     ): void;
 }

src/Listeners/LogRequest.php

@@ -19,7 +19,7 @@ class LogRequest
     public function handle(RequestHandled $event)
     {
         $startTime = defined('LARAVEL_START') ? LARAVEL_START : $event->request->server('REQUEST_TIME_FLOAT');
-        $time = $startTime ? floor((microtime(true) - $startTime) * 1000) : null;
+        $duration = $startTime ? floor((microtime(true) - $startTime) * 1000) : null;
         $memory = memory_get_peak_usage(true);
 
         if ($this->shouldLog($event->request, $event->response)) {
@@ -30,7 +30,7 @@ public function handle(RequestHandled $event)
             RequestLoggerFacade::driver($driver)->log(
                 $event->request,
                 $event->response,
-                $time,
+                $duration,
                 $memory
             );
         }

src/Models/RequestLog.php

@@ -40,7 +40,7 @@ class RequestLog extends Model implements RequestLoggerInterface
         'payload',
         'response_headers',
         'response_body',
-        'time',
+        'duration',
         'memory',
     ];
 
@@ -93,7 +93,7 @@ public function prunable()
     }
 
     /** @inerhitDoc */
-    public function log(Request $request, $response, ?int $time = null, ?int $memory = null): void
+    public function log(Request $request, $response, ?int $duration = null, ?int $memory = null): void
     {
         $model = new static();
 
@@ -109,7 +109,7 @@ public function log(Request $request, $response, ?int $time = null, ?int $memory
         $model->payload = $this->getFiltered($request->input()) ?: null;
         $model->response_headers = $this->getFiltered($response->headers->all()) ?: null;
         $model->response_body = $this->getLoggableResponseContent($response);
-        $model->time = $time;
+        $model->duration = $duration;
         $model->memory = round($memory / 1024 / 1024, 2); // [MB]
 
         if ($user = $request->user()) {

src/NullLogger.php

@@ -7,7 +7,8 @@
 
 class NullLogger implements RequestLoggerInterface
 {
-    public function log(Request $request, $response, ?int $time = null, ?int $memory = null): void
+    /** @inheritDoc */
+    public function log(Request $request, $response, ?int $duration = null, ?int $memory = null): void
     {
         // Intentionally left blank.
     }