|
| 1 | +<?php |
| 2 | + |
| 3 | +declare(strict_types=1); |
| 4 | + |
| 5 | +namespace Tests\Feature\Middleware; |
| 6 | + |
| 7 | +use Illuminate\Support\Facades\Config; |
| 8 | +use Illuminate\Support\Facades\Log; |
| 9 | +use Illuminate\Support\Facades\Route; |
| 10 | +use Tests\TestCase; |
| 11 | + |
| 12 | +class ApiLoggerTest extends TestCase |
| 13 | +{ |
| 14 | + protected function setUp(): void |
| 15 | + { |
| 16 | + parent::setUp(); |
| 17 | + // Ensure logging is enabled for tests |
| 18 | + Config::set('api-logger.enabled', true); |
| 19 | + // Create a test route using the middleware |
| 20 | + Route::middleware('api.logger')->post('/test-api-logger', function () { |
| 21 | + return response()->json(['success' => true, 'token' => 'shouldbemasked']); |
| 22 | + }); |
| 23 | + } |
| 24 | + |
| 25 | + public function test_logs_request_and_response_with_masking() |
| 26 | + { |
| 27 | + Log::spy(); |
| 28 | + $payload = [ |
| 29 | + 'username' => 'testuser', |
| 30 | + 'password' => 'supersecret', |
| 31 | + ]; |
| 32 | + $headers = [ |
| 33 | + 'Authorization' => 'Bearer sometoken', |
| 34 | + 'X-Request-Id' => 'test-request-id-123', |
| 35 | + ]; |
| 36 | + |
| 37 | + $response = $this->postJson('/test-api-logger', $payload, $headers); |
| 38 | + $response->assertOk(); |
| 39 | + $response->assertJson(['success' => true]); |
| 40 | + |
| 41 | + Log::shouldHaveReceived('info')->withArgs(function ($message, $context) use ($headers) { |
| 42 | + return $message === 'API Request' |
| 43 | + && $context['request_id'] === $headers['X-Request-Id'] |
| 44 | + && $context['body']['password'] === '***MASKED***' |
| 45 | + && $context['headers']['authorization'] === '***MASKED***' |
| 46 | + && $context['response_body']['token'] === '***MASKED***'; |
| 47 | + })->once(); |
| 48 | + } |
| 49 | + |
| 50 | + public function test_does_not_log_when_disabled() |
| 51 | + { |
| 52 | + Config::set('api-logger.enabled', false); |
| 53 | + Log::spy(); |
| 54 | + $response = $this->postJson('/test-api-logger', []); |
| 55 | + $response->assertOk(); |
| 56 | + Log::shouldNotHaveReceived('info'); |
| 57 | + } |
| 58 | + |
| 59 | + public function test_logs_with_custom_masked_headers_and_body_keys() |
| 60 | + { |
| 61 | + Config::set('api-logger.masked_headers', ['authorization', 'x-custom-header']); |
| 62 | + Config::set('api-logger.masked_body_keys', ['password', 'secret']); |
| 63 | + Log::spy(); |
| 64 | + $payload = [ |
| 65 | + 'username' => 'testuser', |
| 66 | + 'password' => 'supersecret', |
| 67 | + 'secret' => 'topsecret', |
| 68 | + ]; |
| 69 | + $headers = [ |
| 70 | + 'Authorization' => 'Bearer sometoken', |
| 71 | + 'X-Custom-Header' => 'customvalue', |
| 72 | + 'X-Request-Id' => 'custom-id-456', |
| 73 | + ]; |
| 74 | + $response = $this->postJson('/test-api-logger', $payload, $headers); |
| 75 | + $response->assertOk(); |
| 76 | + Log::shouldHaveReceived('info')->withArgs(function ($message, $context) { |
| 77 | + return $context['headers']['authorization'] === '***MASKED***' |
| 78 | + && $context['headers']['x-custom-header'] === '***MASKED***' |
| 79 | + && $context['body']['password'] === '***MASKED***' |
| 80 | + && $context['body']['secret'] === '***MASKED***'; |
| 81 | + })->once(); |
| 82 | + } |
| 83 | + |
| 84 | + public function test_logs_all_context_fields() |
| 85 | + { |
| 86 | + Log::spy(); |
| 87 | + $payload = ['username' => 'testuser']; |
| 88 | + $headers = [ |
| 89 | + 'X-Request-Id' => 'all-fields-id', |
| 90 | + ]; |
| 91 | + $response = $this->postJson('/test-api-logger', $payload, $headers); |
| 92 | + $response->assertOk(); |
| 93 | + Log::shouldHaveReceived('info')->withArgs(function ($message, $context) use ($headers) { |
| 94 | + return $context['request_id'] === $headers['X-Request-Id'] |
| 95 | + && isset($context['ip']) |
| 96 | + && $context['method'] === 'POST' |
| 97 | + && str_contains($context['uri'], '/test-api-logger') |
| 98 | + && $context['response_status'] === 200 |
| 99 | + && is_numeric($context['duration_ms']); |
| 100 | + })->once(); |
| 101 | + } |
| 102 | + |
| 103 | + public function test_logs_non_json_response() |
| 104 | + { |
| 105 | + Route::middleware('api.logger')->get('/test-non-json', function () { |
| 106 | + return 'plain text response'; |
| 107 | + }); |
| 108 | + Log::spy(); |
| 109 | + $response = $this->get('/test-non-json'); |
| 110 | + $response->assertOk(); |
| 111 | + $response->assertSee('plain text response'); |
| 112 | + Log::shouldHaveReceived('info')->withArgs(function ($message, $context) { |
| 113 | + return $context['response_body'] === 'plain text response'; |
| 114 | + })->once(); |
| 115 | + } |
| 116 | + |
| 117 | + public function test_logs_with_empty_headers_and_body() |
| 118 | + { |
| 119 | + Log::spy(); |
| 120 | + $response = $this->postJson('/test-api-logger', [], []); |
| 121 | + $response->assertOk(); |
| 122 | + Log::shouldHaveReceived('info')->withArgs(function ($message, $context) { |
| 123 | + return is_array($context['headers']) && is_array($context['body']); |
| 124 | + })->once(); |
| 125 | + } |
| 126 | +} |
0 commit comments