HeuJung
ea58c606a2
공개 제보 — 파일 스토리지에서 S3 를 선택해 저장해도 실제 파일 저장이 동작하지 않던 결함의 전면 수정. - S3 어댑터(league/flysystem-aws-s3-v3)·predis 를 코어 기본 의존성으로 포함 — 어댑터 부재 즉사, phpredis 확장 없는 서버의 redis 선택 전면 다운 차단 (부트 시 확장 부재 감지 → predis 자동 폴백) - storage_driver=s3 저장 시 코어 첨부 업로드 디스크를 s3 로 전환 (ATTACHMENT_DISK env 명시가 항상 우선, 기존 행은 저장 당시 disk 로 서빙) - 첨부·템플릿 레이아웃 첨부 서빙을 행 disk 를 따르는 스토리지 스트림으로 교체 — 로컬 절대 경로 전제 fileResponse 는 S3 행에서 filemtime stat 500 (streamedFileResponse: 행 메타 기반 ETag/304/Cache-Control) - S3 호환 스토리지(R2/MinIO/NCP) 연결 지원: 엔드포인트 URL·path-style 설정 신설, 리전 목록 선택 → 자유 입력 전환, 연결 테스트를 실제 저장 경로와 동일 설정(endpoint/path-style)으로 정렬 - 사용 불능 드라이버(어댑터·PHP 확장 부재)의 저장/테스트 요청을 사유와 함께 422 로 차단하는 서버 게이트 신설 (DriverRegistryService 능력 판정) - 웹소켓 연결 테스트에 서버(백엔드 발송용) endpoint 검사 추가 — 클라이언트만 검사해 테스트 성공 + 실제 발송 실패가 가능하던 비대칭 해소 - env 빈 값(`KEY=`) 함정 정규화: AWS_URL/AWS_ENDPOINT/ATTACHMENT_DISK 빈 문자열을 미설정으로 취급 (config 정규화 + 예시 파일 주석 처리) - 플러그인 드라이버 폴백의 log 카테고리 죽은 키(logging.default) 정정 및 websocket 유령 설정 키 제거 - 실 AWS S3 종단 검증 완료 (설정 저장 → 업로드 S3 실저장 → 서빙 200/304)
237 lines
9.2 KiB
PHP
237 lines
9.2 KiB
PHP
<?php
|
|
|
|
namespace Tests\Unit\Services;
|
|
|
|
use App\Services\DriverConnectionTester;
|
|
use Illuminate\Support\Facades\Http;
|
|
use League\Flysystem\AwsS3V3\AwsS3V3Adapter;
|
|
use Tests\TestCase;
|
|
|
|
/**
|
|
* DriverConnectionTester 테스트 (공개 #99 / 내부 #563, A5·A6)
|
|
*
|
|
* testS3 의 endpoint/path-style 반영·어댑터 선검사, testWebsocket 의
|
|
* client/server 양측 endpoint probe 를 검증합니다.
|
|
*
|
|
* 한계: Http::fake 는 요청 URL 계약까지만 검증한다 (요청 본문 미검증 —
|
|
* feedback_http_fake_does_not_validate_request_body). S3 종단 검증은
|
|
* Chrome MCP MinIO 매트릭스가 담당한다.
|
|
*/
|
|
class DriverConnectionTesterTest extends TestCase
|
|
{
|
|
private DriverConnectionTester $tester;
|
|
|
|
protected function setUp(): void
|
|
{
|
|
parent::setUp();
|
|
|
|
$this->tester = new DriverConnectionTester;
|
|
}
|
|
|
|
/**
|
|
* 필수 설정 누락 시 s3_missing_config 메시지를 반환하는지 테스트합니다.
|
|
*/
|
|
public function test_s3_missing_config_returns_failure(): void
|
|
{
|
|
$result = $this->tester->testS3(['s3_bucket' => 'only-bucket']);
|
|
|
|
$this->assertFalse($result['success']);
|
|
$this->assertSame(__('settings.s3_missing_config'), $result['message']);
|
|
}
|
|
|
|
/**
|
|
* 커스텀 endpoint 가 실제 아웃바운드 대상에 반영되는지 테스트합니다.
|
|
*
|
|
* 도달 불가한 로컬 endpoint 로 시도 → 연결 실패의 error 문자열에 해당 endpoint
|
|
* 호스트가 나타나야 한다 (AWS 리전 도메인이 아니라). 이것이 endpoint 미반영
|
|
* 결함(#99 ③ — s3_url 만 있고 SDK 요청 대상은 amazonaws.com 고정)의 회귀 가드다.
|
|
*
|
|
* @scenario region_shape=custom_minio,endpoint=path_style,driver_usability=adapter_present,attachment_disk=local_default
|
|
*
|
|
* @effects endpoint_injected
|
|
*/
|
|
public function test_s3_custom_endpoint_reaches_outbound_target(): void
|
|
{
|
|
$result = $this->tester->testS3([
|
|
's3_bucket' => 'test-bucket',
|
|
's3_region' => 'us-east-1',
|
|
's3_access_key' => 'test-key',
|
|
's3_secret_key' => 'test-secret',
|
|
's3_endpoint' => 'http://127.0.0.1:9',
|
|
's3_use_path_style' => true,
|
|
]);
|
|
|
|
$this->assertFalse($result['success']);
|
|
$this->assertArrayHasKey('error', $result);
|
|
$this->assertStringContainsString(
|
|
'127.0.0.1',
|
|
$result['error'],
|
|
'아웃바운드가 커스텀 endpoint 를 향해야 함 — amazonaws.com 고정이면 endpoint 미반영 회귀'
|
|
);
|
|
$this->assertStringNotContainsString('amazonaws.com', $result['error']);
|
|
}
|
|
|
|
/**
|
|
* path-style 토글이 아웃바운드 주소 형태를 실제로 바꾸는지 테스트합니다.
|
|
*
|
|
* IP endpoint 는 SDK 가 path-style 을 자동 강제해 off↔on 이 구분되지 않으므로
|
|
* hostname endpoint 로 검증한다: off = virtual-hosted(버킷이 호스트에 접두) /
|
|
* on = path-style(버킷이 경로로 이동). off 상태로도 통과하던 endpoint 단언의
|
|
* 사각을 메우는 회귀 가드다.
|
|
*
|
|
* @scenario region_shape=custom_minio,endpoint=path_style,driver_usability=adapter_present,attachment_disk=local_default
|
|
*
|
|
* @effects endpoint_injected
|
|
*/
|
|
public function test_s3_path_style_toggle_changes_outbound_address_shape(): void
|
|
{
|
|
$config = [
|
|
's3_bucket' => 'test-bucket',
|
|
's3_region' => 'us-east-1',
|
|
's3_access_key' => 'test-key',
|
|
's3_secret_key' => 'test-secret',
|
|
's3_endpoint' => 'http://s3-parity.invalid:9',
|
|
];
|
|
|
|
$virtualHosted = $this->tester->testS3($config + ['s3_use_path_style' => false]);
|
|
$pathStyle = $this->tester->testS3($config + ['s3_use_path_style' => true]);
|
|
|
|
$this->assertFalse($virtualHosted['success']);
|
|
$this->assertFalse($pathStyle['success']);
|
|
|
|
// off: 버킷명이 호스트에 접두된 virtual-hosted 주소를 향해야 한다
|
|
$this->assertStringContainsString(
|
|
'test-bucket.s3-parity.invalid',
|
|
$virtualHosted['error'],
|
|
'path-style off 는 virtual-hosted 주소여야 함'
|
|
);
|
|
|
|
// on: 버킷 접두 호스트가 사라져야 한다 — off 와 동일하면 path-style 주입 회귀
|
|
$this->assertStringNotContainsString(
|
|
'test-bucket.s3-parity.invalid',
|
|
$pathStyle['error'],
|
|
'path-style on 인데 버킷 접두 호스트가 남아 있으면 use_path_style_endpoint 미반영 회귀'
|
|
);
|
|
$this->assertStringContainsString('s3-parity.invalid', $pathStyle['error']);
|
|
}
|
|
|
|
/**
|
|
* 어댑터 선검사 계약 — 어댑터가 설치된 환경에서는 missing 분기가 발동하지 않고,
|
|
* 코드가 어댑터 존재를 검사하고 있는지(클래스 상수 참조) 검증합니다.
|
|
*
|
|
* 어댑터는 composer 필수 의존성이라 부재 상태를 만들 수 없으므로 구성 검증으로 대체한다
|
|
* (계획서 A5 — "mock 불가 시 구성 검증으로 대체").
|
|
*/
|
|
public function test_s3_adapter_precheck_present_in_code(): void
|
|
{
|
|
$this->assertTrue(
|
|
class_exists(AwsS3V3Adapter::class),
|
|
'어댑터는 코어 필수 의존성 (#99 A1)'
|
|
);
|
|
|
|
$source = file_get_contents(app_path('Services/DriverConnectionTester.php'));
|
|
$this->assertStringContainsString(
|
|
'AwsS3V3Adapter::class',
|
|
$source,
|
|
'testS3 는 어댑터 존재 선검사를 유지해야 함 — 테스트(SDK 직접)와 실경로(Flysystem)의 분리가 결함을 가리는 구조 방지 (A5)'
|
|
);
|
|
$this->assertStringContainsString('s3_adapter_missing', $source);
|
|
}
|
|
|
|
/**
|
|
* testWebsocket 이 client 와 server endpoint 를 모두 probe 하는지 테스트합니다.
|
|
*
|
|
* @scenario region_shape=aws_region,endpoint=absent,driver_usability=adapter_present,attachment_disk=local_default
|
|
*
|
|
* @effects test_covers_server_websocket
|
|
*/
|
|
public function test_websocket_probes_both_client_and_server_endpoints(): void
|
|
{
|
|
Http::fake([
|
|
'client.test:8443*' => Http::response('ok', 200),
|
|
'server.test:8080*' => Http::response('ok', 200),
|
|
]);
|
|
|
|
$result = $this->tester->testWebsocket([
|
|
'websocket_host' => 'client.test',
|
|
'websocket_port' => 8443,
|
|
'websocket_scheme' => 'https',
|
|
'websocket_server_host' => 'server.test',
|
|
'websocket_server_port' => 8080,
|
|
'websocket_server_scheme' => 'http',
|
|
]);
|
|
|
|
$this->assertTrue($result['success']);
|
|
Http::assertSent(fn ($request) => str_starts_with($request->url(), 'https://client.test'));
|
|
Http::assertSent(fn ($request) => str_starts_with($request->url(), 'http://server.test'));
|
|
}
|
|
|
|
/**
|
|
* server endpoint 실패 시 어느 쪽 실패인지 구분된 메시지를 반환하는지 테스트합니다.
|
|
*/
|
|
public function test_websocket_server_endpoint_failure_is_distinguished(): void
|
|
{
|
|
Http::fake([
|
|
'client.test:8443*' => Http::response('ok', 200),
|
|
'server.test:8080*' => Http::response('error', 500),
|
|
]);
|
|
|
|
$result = $this->tester->testWebsocket([
|
|
'websocket_host' => 'client.test',
|
|
'websocket_port' => 8443,
|
|
'websocket_scheme' => 'https',
|
|
'websocket_server_host' => 'server.test',
|
|
'websocket_server_port' => 8080,
|
|
'websocket_server_scheme' => 'http',
|
|
]);
|
|
|
|
$this->assertFalse($result['success']);
|
|
$this->assertSame(
|
|
__('settings.websocket_server_test_failed'),
|
|
$result['message'],
|
|
'백엔드 발송은 server endpoint 를 쓴다 — client 만 검사하면 "테스트 성공 + 실제 발송 실패" (A6)'
|
|
);
|
|
}
|
|
|
|
/**
|
|
* server 필드가 비어 있으면 client 값으로 폴백하며 중복 probe 를 생략하는지 테스트합니다.
|
|
*/
|
|
public function test_websocket_server_fields_fallback_to_client(): void
|
|
{
|
|
Http::fake([
|
|
'client.test:8443*' => Http::response('ok', 200),
|
|
]);
|
|
|
|
$result = $this->tester->testWebsocket([
|
|
'websocket_host' => 'client.test',
|
|
'websocket_port' => 8443,
|
|
'websocket_scheme' => 'https',
|
|
]);
|
|
|
|
$this->assertTrue($result['success']);
|
|
Http::assertSentCount(1);
|
|
}
|
|
|
|
/**
|
|
* client endpoint 실패 시 기존 메시지 계약이 유지되는지 테스트합니다 (하위호환).
|
|
*/
|
|
public function test_websocket_client_failure_keeps_existing_contract(): void
|
|
{
|
|
Http::fake([
|
|
'client.test:8443*' => Http::response('error', 500),
|
|
]);
|
|
|
|
$result = $this->tester->testWebsocket([
|
|
'websocket_host' => 'client.test',
|
|
'websocket_port' => 8443,
|
|
'websocket_scheme' => 'https',
|
|
'websocket_server_host' => 'server.test',
|
|
'websocket_server_port' => 8080,
|
|
'websocket_server_scheme' => 'http',
|
|
]);
|
|
|
|
$this->assertFalse($result['success']);
|
|
$this->assertSame(__('settings.websocket_test_failed'), $result['message']);
|
|
}
|
|
}
|