Skip to content

Commit 7e47690

Browse files
committed
chore: store request class and method
1 parent 61461c9 commit 7e47690

File tree

6 files changed

+44
-0
lines changed

6 files changed

+44
-0
lines changed
Lines changed: 29 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,29 @@
1+
<?php
2+
3+
declare(strict_types=1);
4+
5+
use Illuminate\Database\Migrations\Migration;
6+
use Illuminate\Database\Schema\Blueprint;
7+
use Illuminate\Support\Facades\Schema;
8+
9+
return new class extends Migration
10+
{
11+
/**
12+
* Run the migrations.
13+
*/
14+
public function up(): void
15+
{
16+
Schema::table('saloon_requests', function (Blueprint $table) {
17+
$table->string('request')->after('connector');
18+
$table->string('method')->after('endpoint');
19+
});
20+
}
21+
22+
/**
23+
* Reverse the migrations.
24+
*/
25+
public function down(): void
26+
{
27+
Schema::table('saloon_requests', fn (Blueprint $table) => $table->dropColumn(['method', 'request']));
28+
}
29+
};

readme.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,11 @@ Batteries for [Saloon](https://docs.saloon.dev/).
88

99
- Request logging (to database by default or in-memory)
1010

11+
Supported:
12+
- Laravel 11/12
13+
- PHP 8.2/8.3/8.4
14+
- Saloon 3
15+
1116
## Installation
1217

1318
Via Composer

src/Logger/SaloonRequest.php

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,8 @@ class SaloonRequest extends Model
2020
*/
2121
protected $fillable = [
2222
'connector',
23+
'request',
24+
'method',
2325
'endpoint',
2426
'request_headers',
2527
'request_query',

src/Logger/Stores/DatabaseLogger.php

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,8 @@ public function create(PendingRequest $request, Connector $connector): mixed
1717
{
1818
$log = SaloonRequest::create([
1919
'connector' => get_class($connector),
20+
'request' => get_class($request->getRequest()),
21+
'method' => $request->getRequest()->getMethod(),
2022
'endpoint' => $request->getRequest()->resolveEndpoint(),
2123
'request_headers' => $request->getRequest()->headers()->all(),
2224
'request_query' => $request->getRequest()->query()->all(),

src/Logger/Stores/MemoryLogger.php

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,8 @@ public function create(PendingRequest $request, Connector $connector): mixed
4747
$data[$requestId] = [
4848
'id' => $requestId,
4949
'connector' => get_class($connector),
50+
'request' => get_class($request->getRequest()),
51+
'method' => $request->getRequest()->getMethod(),
5052
'endpoint' => $request->getRequest()->resolveEndpoint(),
5153
'request_headers' => $request->getRequest()->headers()->all(),
5254
'request_query' => $request->getRequest()->query()->all(),

tests/Unit/Logger/DatabaseLoggerTest.php

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -36,10 +36,14 @@ public function logs_response(): void
3636

3737
$this->assertDatabaseCount((new SaloonRequest)->getTable(), 1);
3838

39+
/** @var SaloonRequest $log */
3940
$log = SaloonRequest::first();
4041
$request = new GoogleSearchRequest('saloon');
4142

4243
$this->assertEquals(200, $log->status_code);
44+
$this->assertEquals(ConnectorGeneric::class, $log->connector);
45+
$this->assertEquals(GoogleSearchRequest::class, $log->request);
46+
$this->assertEquals($request->getMethod()->value, $log->method);
4347
$this->assertEquals($request->query()->all(), $log->request_query);
4448
$this->assertEquals($request->resolveEndpoint(), $log->endpoint);
4549
}

0 commit comments

Comments
 (0)