HeuJung
f32729e20e
KVE-2026-2191 대응에서 계획 안 세부를 자율 결정한 지점을 회귀 전수조사해 정합 결함 5건을 고쳤다. 넷은 같은 미출시 버전이 만든 자기교정이고, 마지막 하나는 7.0.10 및 그 이전부터 있던 결함이라 공개 CHANGELOG 에 항목을 넣었다. - 적중 통계의 레이아웃명이 항상 비어 화면별 표가 성립하지 않던 것 → 페이지 캐시 값에 레이아웃명을 함께 담고 적중 경로가 그것으로 귀속한다. 이전 버전이 문자열로만 저장한 항목은 그대로 읽어 배포 직후 살아 있는 캐시를 버리지 않는다. 인터페이스는 불변이고 코어 구현일 때만 항목 전체를 읽는다. - 통계 url 컬럼(255)이 캐시 키 URL(정규화 쿼리 최대 512바이트)보다 짧아 긴 주소의 기록이 엄격 모드에서 실패하고 그 예외를 서비스가 삼키던 것 → 768 로 확장(utf8mb4 인덱스 키 상한 3072바이트). down 은 잘리는 행을 먼저 지운다. - 경로당 변종 상한이 언어를 합산해 다국어 사이트의 실효 상한이 언어 수만큼 줄던 것 → 인덱스 항목이 url|locale 별이므로 경로·언어 조합 단위로 판정한다. - 그릴 게 없는 요청(미라우트 404·SEO 비활성)이 렌더 예산을 소모해, 캐시에 남지 않는 죽은 주소 재크롤이 정상 페이지의 예산을 태우던 것 → null 렌더는 차감을 환급한다. 렌더 도중 예외는 비용을 이미 치른 것이라 환급하지 않는다. - 병합 단계에서 건너뛴 확장이 있는 결과가 캐시로 굳어 버전 bump 전까지 그 확장 자산이 사라진 채 고정되던 것 → 병합 루프를 mergeJs/mergeCss 로 분리해 건너뛴 확장을 함께 돌려주고, 하나라도 있으면 캐시하지 않고 Log::error 로 남긴다. 캐시 없이 돌면 호출측이 매 요청 다시 병합하므로 원인이 사라지는 즉시 회복한다. 기본 로그 수준이 error 라 종전 per-extension warning 은 기록되지 않아 흔적이 없었다. 회귀 테스트 8건은 red 확인 후 green 으로 전환했고, 변경 심볼의 소비자 모집단(테스트 클래스 9종)을 프로세스 분할로 재실행해 189건 통과를 확인했다.
98 lines
3.6 KiB
PHP
98 lines
3.6 KiB
PHP
<?php
|
|
|
|
namespace Tests\Feature\Database;
|
|
|
|
use Illuminate\Foundation\Testing\RefreshDatabase;
|
|
use Illuminate\Support\Facades\Artisan;
|
|
use Illuminate\Support\Facades\DB;
|
|
use Illuminate\Support\Facades\Schema;
|
|
use Tests\TestCase;
|
|
|
|
/**
|
|
* `seo_cache_stats.url` 확장 마이그레이션의 왕복(up → down → up) 안전성.
|
|
*
|
|
* 캐시 키 URL(경로 + 정규화 쿼리 최대 512바이트)이 255자 컬럼에 들어가지 않아 긴 주소의
|
|
* 통계 기록이 조용히 실패하던 것을 768자로 넓힌 마이그레이션이다. 되돌릴 때는 잘리는 행을
|
|
* 먼저 지워야 엄격 모드에서 ALTER 가 성공한다.
|
|
*/
|
|
class ModifyUrlInSeoCacheStatsMigrationTest extends TestCase
|
|
{
|
|
use RefreshDatabase;
|
|
|
|
private const MIGRATION = 'database/migrations/2026_09_08_000001_modify_url_in_seo_cache_stats_table.php';
|
|
|
|
/**
|
|
* up → down → up 왕복 뒤 컬럼 길이가 복원되고 url 인덱스가 보존된다.
|
|
*
|
|
* @scenario bot_state=bot, cache_state=hit, ip_budget=within, query_shape=normal, store_state=under_caps
|
|
*
|
|
* @effects stats_url_column_fits_normalized_cache_url
|
|
*/
|
|
public function test_round_trip_restores_column_length_and_keeps_index(): void
|
|
{
|
|
$this->assertSame(768, $this->urlLength());
|
|
|
|
Artisan::call('migrate:rollback', ['--path' => [self::MIGRATION], '--force' => true]);
|
|
$this->assertSame(255, $this->urlLength());
|
|
|
|
Artisan::call('migrate', ['--path' => [self::MIGRATION], '--force' => true]);
|
|
$this->assertSame(768, $this->urlLength());
|
|
|
|
$this->assertContains(
|
|
'idx_seo_cache_stats_url',
|
|
array_column(Schema::getIndexes('seo_cache_stats'), 'name'),
|
|
'url 인덱스는 컬럼 변경 뒤에도 남아야 한다'
|
|
);
|
|
}
|
|
|
|
/**
|
|
* down 은 255자를 넘는 행을 먼저 지운다 — 엄격 모드에서는 잘리는 값이 하나라도 있으면
|
|
* ALTER 자체가 실패해 되돌리기가 막힌다.
|
|
*
|
|
* @scenario bot_state=bot, cache_state=hit, ip_budget=within, query_shape=normal, store_state=under_caps
|
|
*
|
|
* @effects stats_url_column_fits_normalized_cache_url
|
|
*/
|
|
public function test_down_removes_rows_that_would_not_fit_before_shrinking(): void
|
|
{
|
|
$long = '/shop?'.str_repeat('a=0123456789&', 30);
|
|
|
|
$this->assertGreaterThan(255, strlen($long));
|
|
|
|
DB::table('seo_cache_stats')->insert([
|
|
['url' => $long, 'locale' => 'ko', 'type' => 'hit'],
|
|
['url' => '/short', 'locale' => 'ko', 'type' => 'hit'],
|
|
]);
|
|
|
|
try {
|
|
Artisan::call('migrate:rollback', ['--path' => [self::MIGRATION], '--force' => true]);
|
|
|
|
$this->assertSame(255, $this->urlLength());
|
|
$this->assertDatabaseMissing('seo_cache_stats', ['url' => $long]);
|
|
$this->assertDatabaseHas('seo_cache_stats', ['url' => '/short']);
|
|
} finally {
|
|
// DDL 이 테스트 트랜잭션을 암묵 커밋하므로 남는 행은 직접 치운다.
|
|
Artisan::call('migrate', ['--path' => [self::MIGRATION], '--force' => true]);
|
|
DB::table('seo_cache_stats')->where('url', '/short')->delete();
|
|
}
|
|
}
|
|
|
|
/**
|
|
* url 컬럼의 선언 길이를 읽습니다.
|
|
*
|
|
* @return int 문자 수 (varchar(N) 의 N)
|
|
*/
|
|
private function urlLength(): int
|
|
{
|
|
foreach (Schema::getColumns('seo_cache_stats') as $column) {
|
|
if ($column['name'] === 'url') {
|
|
preg_match('/\((\d+)\)/', (string) $column['type'], $matches);
|
|
|
|
return (int) ($matches[1] ?? 0);
|
|
}
|
|
}
|
|
|
|
$this->fail('seo_cache_stats.url 컬럼이 없다');
|
|
}
|
|
}
|