🛡️ Sentinel: [security improvement] 민감한 파일 확장자 정보 노출 방지

.jules/sentinel.md

@@ -83,3 +83,7 @@
 **Vulnerability:** 정적 HTML 생성 도구에서 매번 다른 Nonce를 동적으로 생성하여 CSP에 적용하는 것은, 캐싱 효율을 저하시킬 뿐만 아니라 정적 배포 환경(예: GitHub Pages 등)에서 올바른 보안 정책 수립을 방해할 수 있는 안티 패턴입니다.
 **Learning:** 정적으로 고정된 인라인 스타일이나 스크립트에는 난수화된 Nonce보다 콘텐츠 자체의 해시(SHA-256 등)를 사용하는 것이 안전하고 일관된 방식임을 배웠습니다.
 **Prevention:** 자동 생성되는 정적 HTML의 콘텐츠 보안 정책(CSP)에는 `style-src 'sha256-<HASH>'` 방식을 적용하고, `<style>` 태그에서 불필요한 `nonce` 속성을 제거하여 브라우저의 무결성 검증 기능을 적극 활용하십시오.
+## 2024-07-14 - Information Exposure via Sensitive Extensions
+**Vulnerability:** The `process_ignore_file` function only excluded files starting with a dot, allowing sensitive files with common extensions (e.g. `.pem`, `.key`, `.p12`, `.bak`) to be indexed and exposed to users via the generated HTML directory tree.
+**Learning:** Hardcoding sensitive exclusions without dynamically evaluating standard security patterns leaves gaps in protection, specifically for files like backups or authentication keys.
+**Prevention:** Implement defense-in-depth dynamic filters for known sensitive file extensions when generating directory listings.

src/main/kotlin/html4tree/main.kt

@@ -219,9 +219,18 @@ fun process_ignore_file(curr_dir: File, dirFilesNames: Array<String>? = null): S
     val defaultSensitiveFiles = listOf(".git", ".env", ".ssh", ".htpasswd", ".htaccess", "id_rsa", "id_ed25519", "secrets.yml", ".html4ignore", ".DS_Store", ".aws", ".kube", ".npmrc", ".gnupg", "config.json", "credentials.json")
     files_to_exclude.addAll(defaultSensitiveFiles)
 
-    // 보안 향상: .env, .git 등 민감한 정보가 포함될 수 있는 숨김 파일(.으로 시작하는 모든 항목)을 기본적으로 노출하지 않도록 제외 (정보 노출 방지)
+    // 보안 향상: .env, .git 등 숨김 파일과 인증 키, 백업 파일 등 민감한 정보가 포함될 수 있는 파일을 동적으로 제외하여 정보 노출(Information Exposure) 방지
     (dirFilesNames ?: curr_dir.list())?.forEach {
-        if (it.startsWith(".")) {
+        val lowerName = it.toLowerCase()
+        if (it.startsWith(".") ||
+            lowerName.endsWith(".pem") ||
+            lowerName.endsWith(".key") ||
+            lowerName.endsWith(".p12") ||
+            lowerName.endsWith(".pfx") ||
+            lowerName.endsWith(".keystore") ||
+            lowerName.endsWith(".bak") ||
+            lowerName.endsWith(".swp") ||
+            it.endsWith("~")) {
             files_to_exclude.add(it)
         }
     }

src/test/kotlin/html4tree/MainTest.kt

@@ -570,6 +570,30 @@ class MainTest {
         assertFalse(excluded.contains("test.txt"))
     }
 
+    @Test
+    fun testProcessIgnoreFileSensitiveExtensions() {
+        File(tempDir, "secret.pem").createNewFile()
+        File(tempDir, "key.KEY").createNewFile() // test case-insensitivity
+        File(tempDir, "cert.p12").createNewFile()
+        File(tempDir, "cert.pfx").createNewFile()
+        File(tempDir, "app.keystore").createNewFile()
+        File(tempDir, "backup.bak").createNewFile()
+        File(tempDir, "vim.swp").createNewFile()
+        File(tempDir, "emacs~").createNewFile()
+        File(tempDir, "normal.txt").createNewFile()
+
+        val excluded = process_ignore_file(tempDir)
+        assertTrue(excluded.contains("secret.pem"))
+        assertTrue(excluded.contains("key.KEY"))
+        assertTrue(excluded.contains("cert.p12"))
+        assertTrue(excluded.contains("cert.pfx"))
+        assertTrue(excluded.contains("app.keystore"))
+        assertTrue(excluded.contains("backup.bak"))
+        assertTrue(excluded.contains("vim.swp"))
+        assertTrue(excluded.contains("emacs~"))
+        assertFalse(excluded.contains("normal.txt"))
+    }
+
     @Test
     fun testIgnoreFileIsDirectory() {
         val ignoreDir = File(tempDir, ".html4ignore")