Task #207 Stage 3: Firefox interceptor에 공통 판정 적용 + 이중 콜백 유지

claude · claude · commit 5381ca8fb25f · 2026-04-20T02:09:12.000Z
- rhwp-firefox/sw/download-interceptor.js - 로컬 HWP_EXTENSIONS 정규식 제거 → 공통 모듈 shouldInterceptDownload 사용 - onCreated: url 외에 mime/referrer 까지 자동 평가 (Chrome 동등) - onChanged: filename 매칭만 하던 단순 체크 → 공통 판정 함수로 교체 · DEXT5 referrer 차단, MIME 힌트, finalUrl 검사 자동 적용 (#198 패리티) - onCreated+onChanged 이중 콜백 구조는 그대로 유지 (Firefox MV3 제약) Refs #207
diff --git a/rhwp-firefox/sw/download-interceptor.js b/rhwp-firefox/sw/download-interceptor.js
@@ -1,38 +1,37 @@
 // 다운로드 가로채기 (Firefox 버전)
-// - onCreated: URL 기반 즉시 감지 (1차 판정)
-// - onChanged: filename 확정 시 재판정 (2차 판정)
-// - browser.downloads.search로 최신 DownloadItem 재조회
+// - onCreated: url/mime/referrer 기반 1차 판정 (filename 미확정일 수 있음)
+// - onChanged: filename 확정 시 2차 판정
+// - browser.downloads.search 로 최신 DownloadItem 재조회
 // - handled 집합으로 동일 다운로드 중복 처리 방지
+//
+// #207: 판정 로직은 rhwp-shared/sw/download-interceptor-common.js 와 공유.
+// Chrome 과 동일한 DEXT5 블랙리스트 / MIME 힌트 / finalUrl 검사 자동 적용.
 
 import { openViewer } from './viewer-launcher.js';
+import { shouldInterceptDownload } from './download-interceptor-common.js';
 
-const HWP_EXTENSIONS = /\.(hwp|hwpx)(\?.*)?$/i;
 const handled = new Set();     // 이미 처리된 downloadId
 
 export function setupDownloadInterceptor() {
-  // 1차: 다운로드 시작 시 URL 기반 즉시 감지
+  // 1차: 다운로드 시작 시 url/mime/referrer 로 즉시 판정
   browser.downloads.onCreated.addListener((item) => {
     if (handled.has(item.id)) return;
 
-    if (HWP_EXTENSIONS.test(item.url || '')) {
+    if (shouldInterceptDownload(item)) {
       handled.add(item.id);
       handleHwpDownload(item);
     }
-    // URL로 판별 불가 시: filename 확정될 때 onChanged에서 재판정
+    // 판정 불가 시: filename 확정될 때 onChanged에서 재판정
   });
 
   // 2차: filename 확정 시 재판정
   browser.downloads.onChanged.addListener(async (delta) => {
-    // filename 확정으로 HWP가 처음 판정되는 경우에만 처리
-    if (!handled.has(delta.id)
-      && delta.filename?.current
-      && HWP_EXTENSIONS.test(delta.filename.current)) {
-      handled.add(delta.id);
-
+    // 아직 미처리 + filename 확정 → 최신 DownloadItem 으로 재판정
+    if (!handled.has(delta.id) && delta.filename?.current) {
       try {
-        // 최신 DownloadItem 재조회 (url, fileSize 등 완전한 정보 확보)
         const [item] = await browser.downloads.search({ id: delta.id });
-        if (item) {
+        if (item && shouldInterceptDownload(item)) {
+          handled.add(delta.id);
           handleHwpDownload(item);
         }
       } catch (err) {
@@ -61,7 +60,7 @@ async function handleHwpDownload(item) {
 
     openViewer({
       url: item.url,
-      filename: item.filename
+      filename: item.filename,
     });
   } catch (err) {
     console.error('[rhwp] 다운로드 인터셉터 오류:', err);
