공개 저장소 이슈 gnuboard/g7 (@bigmsg) 제보에서 출발한 작업이다. 브라우저가 화면을 그리려고 제3자 CDN 에 도달해야 하면, 그 도달 실패는 예외도 로그도 남기지 않고 화면 기능만 조용히 사라진다. 폐쇄망·방화벽·광고차단기에서 재현되는데 자체 서버 로그에는 흔적이 없어 운영자가 원인을 특정할 수 없다. 제보된 것은 편집기 하나였지만 같은 구조가 아이콘·글꼴·코드편집기·압축 라이브러리·설치 마법사·개발 대시보드에 똑같이 있었으므로, 번들 확장과 템플릿 전체를 자체 제공으로 옮겼다. 런타임에 외부로 나가는 것은 주소 검색 서비스 하나만 남았다. 자체 제공만으로는 부족하다 — 자기 서버에서 받는 파일도 실패할 수 있고, 종전에는 그 실패가 무음이었다. CSS 경로에 재시도 계층을 세우고(스크립트 경로와 동형), 서버가 HTML 에 직접 심는 externals 까지 실패를 붙잡아 안내 배너와 [다시 시도]로 표면화했다. 편집기·코드편집기는 확보 실패 시 평문 입력으로 내려앉되 저장 계약을 유지한다. 두 번째 축은 운영자가 CSS 를 덧붙일 자리가 없던 문제다(sir.kr 문의). 확장 디렉토리의 custom/ 을 운영자 소유로 정해, 확장 교체가 그 디렉토리만은 보존하게 했다. 출처에 의존하지 않는 서술자로 해석하므로 나중에 다른 출처가 붙어도 소비자는 바뀌지 않는다. 확장 자산과 같은 메커니즘으로 정적 게시되어 CSS 내부 상대 url 도 해석되고, 파일을 고치면 그 변경을 감지해 재게시까지 예약된다. FTP 접근이 없는 운영자에게는 그 자리도 없는 것과 같으므로 레이아웃 편집기에서 직접 넣고 고칠 수 있게 했다. 모듈·플러그인·템플릿이 한 엔드포인트를 공유한다 — 타입별로 나누면 같은 검증이 세 벌로 갈리고 그중 약한 하나가 우회로가 된다. 여기서 올린 스크립트는 그 레이아웃 한 장이 아니라 사이트 전 화면에서 실행되므로 레이아웃 편집과 분리된 전용 권한으로 연다. 운영자 CSS 가 화면을 조작 불능으로 만들면 그것을 고칠 화면에도 같은 CSS 가 실려 스스로 갇히므로, 서버가 목록을 비우는 탈출구(?custom=off)를 함께 뒀다. 동봉 자산은 재생성 경로에 버전 대조 가드를 붙였다. 선언과 다른 버전을 버전 디렉토리에 써 넣는 조용한 거짓말은 배포된 뒤에는 드러나지 않는다.
160 lines
5.5 KiB
PHP
160 lines
5.5 KiB
PHP
<?php
|
|
|
|
namespace Tests\Feature\Extension;
|
|
|
|
use App\Extension\Helpers\ExtensionInstallRollbackHelper;
|
|
use Illuminate\Support\Facades\File;
|
|
use Tests\TestCase;
|
|
|
|
/**
|
|
* 회귀: 실패한 설치가 활성 디렉토리를 고아로 남기지 않는다
|
|
*
|
|
* 설치 흐름은 원본(`_pending`/`_bundled`)을 활성 디렉토리로 **먼저 복사한 뒤** 확장을
|
|
* 로드해 코어 버전·의존성·언어 경로를 검증한다. 검증에 로드된 인스턴스가 필요해 순서를
|
|
* 뒤집을 수 없는데, 검증이 실패하면 방금 만든 디렉토리가 그대로 남았다.
|
|
*
|
|
* 남은 디렉토리는 DB 행이 없어 `plugin:list` 에도 뜨지 않고 번들 병합에도 참여하지 않는다.
|
|
* 오류도 경고도 없이 디스크만 점유하는 고아이고, 다음 설치 시도가 그것을 "이미 있는
|
|
* 설치본" 으로 보고 원본 복사를 건너뛸 수 있다.
|
|
*/
|
|
class FailedInstallRollsBackActiveDirectoryTest extends TestCase
|
|
{
|
|
private string $sandbox;
|
|
|
|
protected function setUp(): void
|
|
{
|
|
parent::setUp();
|
|
|
|
$this->sandbox = storage_path('framework/testing/install-rollback-'.uniqid());
|
|
File::ensureDirectoryExists($this->sandbox);
|
|
}
|
|
|
|
protected function tearDown(): void
|
|
{
|
|
File::deleteDirectory($this->sandbox);
|
|
|
|
parent::tearDown();
|
|
}
|
|
|
|
/**
|
|
* 이번 설치가 만든 디렉토리는 제거한다.
|
|
*/
|
|
public function test_directory_created_by_this_install_is_removed(): void
|
|
{
|
|
$active = $this->sandbox.DIRECTORY_SEPARATOR.'vendor-sample';
|
|
File::ensureDirectoryExists($active);
|
|
File::put($active.DIRECTORY_SEPARATOR.'plugin.json', '{}');
|
|
|
|
$removed = ExtensionInstallRollbackHelper::removeIfCreatedByThisInstall(
|
|
$active,
|
|
existedBefore: false,
|
|
identifier: 'vendor-sample',
|
|
type: 'plugin',
|
|
);
|
|
|
|
$this->assertTrue($removed);
|
|
$this->assertDirectoryDoesNotExist($active);
|
|
}
|
|
|
|
/**
|
|
* 이미 있던 설치본은 건드리지 않는다.
|
|
*
|
|
* `--force` 재설치가 검증에서 실패한 경우다. 그 디렉토리는 운영자의 기존 설치본이므로
|
|
* 여기서 지우면 실패한 재설치가 멀쩡히 돌던 확장을 없애버린다.
|
|
*/
|
|
public function test_preexisting_directory_is_left_untouched(): void
|
|
{
|
|
$active = $this->sandbox.DIRECTORY_SEPARATOR.'vendor-existing';
|
|
File::ensureDirectoryExists($active);
|
|
File::put($active.DIRECTORY_SEPARATOR.'plugin.json', '{"keep":true}');
|
|
|
|
$removed = ExtensionInstallRollbackHelper::removeIfCreatedByThisInstall(
|
|
$active,
|
|
existedBefore: true,
|
|
identifier: 'vendor-existing',
|
|
type: 'plugin',
|
|
);
|
|
|
|
$this->assertFalse($removed);
|
|
$this->assertDirectoryExists($active);
|
|
$this->assertStringContainsString('keep', File::get($active.DIRECTORY_SEPARATOR.'plugin.json'));
|
|
}
|
|
|
|
/**
|
|
* 디렉토리가 아예 없으면 조용히 넘어간다 (복사 이전에 실패한 경우).
|
|
*/
|
|
public function test_absent_directory_is_a_no_op(): void
|
|
{
|
|
$this->assertFalse(ExtensionInstallRollbackHelper::removeIfCreatedByThisInstall(
|
|
$this->sandbox.DIRECTORY_SEPARATOR.'never-created',
|
|
existedBefore: false,
|
|
identifier: 'never-created',
|
|
type: 'module',
|
|
));
|
|
}
|
|
|
|
/**
|
|
* 세 관리자의 설치 경로가 모두 이 되돌리기를 경유한다.
|
|
*
|
|
* 한 곳만 배선하면 나머지 두 유형에서 같은 고아가 계속 생기고, 그 사실은 오류로
|
|
* 드러나지 않는다. 배선이 나중에 조용히 빠지는 것도 여기서 막는다.
|
|
*
|
|
* @return void
|
|
*/
|
|
public function test_all_three_managers_wire_the_rollback(): void
|
|
{
|
|
$targets = [
|
|
[app_path('Extension/PluginManager.php'), 'installPlugin'],
|
|
[app_path('Extension/ModuleManager.php'), 'installModule'],
|
|
[app_path('Extension/TemplateManager.php'), 'installTemplate'],
|
|
];
|
|
|
|
foreach ($targets as [$path, $method]) {
|
|
$source = File::get($path);
|
|
$body = $this->methodBody($source, $method);
|
|
|
|
$this->assertNotSame('', $body, "{$method} 본문을 찾지 못했습니다.");
|
|
$this->assertStringContainsString(
|
|
'ExtensionInstallRollbackHelper::removeIfCreatedByThisInstall',
|
|
$body,
|
|
"{$method} 이 실패 시 활성 디렉토리를 되돌리지 않습니다."
|
|
);
|
|
}
|
|
}
|
|
|
|
/**
|
|
* 소스에서 메서드 본문을 잘라냅니다 (중괄호 균형).
|
|
*
|
|
* @param string $source PHP 소스 전문
|
|
* @param string $method 메서드명
|
|
* @return string 본문 (못 찾으면 빈 문자열)
|
|
*/
|
|
private function methodBody(string $source, string $method): string
|
|
{
|
|
$start = strpos($source, 'public function '.$method.'(');
|
|
|
|
if ($start === false) {
|
|
return '';
|
|
}
|
|
|
|
$depth = 0;
|
|
$began = false;
|
|
$length = strlen($source);
|
|
|
|
for ($i = $start; $i < $length; $i++) {
|
|
if ($source[$i] === '{') {
|
|
$depth++;
|
|
$began = true;
|
|
} elseif ($source[$i] === '}') {
|
|
$depth--;
|
|
|
|
if ($began && $depth === 0) {
|
|
return substr($source, $start, $i - $start + 1);
|
|
}
|
|
}
|
|
}
|
|
|
|
return '';
|
|
}
|
|
}
|