1 Commits

Author SHA1 Message Date
f0e07e06d6 installer: port page = check-only (no auto-open), big open/notOpen message
요청 반영: 간편설치기 포트포워딩 페이지가 포트를 직접 열지 않고 "열려 있는지"만
확인한다.
- server:portForward 를 UPnP 자동 개방/재점검 없이 외부 도달 점검만 하도록 변경,
  결과를 open/notOpen 으로 반환(PortForwardResult status 타입도 변경).
- 열려 있으면 외부 접속 주소를 크게, 안 열려 있으면 "직접 포트포워딩 해주세요"
  를 크게 표시.
- 페이지 체크에서만 쓰이던 openPortViaUpnp/removeUpnpMapping 제거(고아 코드).
- 참고: 서버 기동 시 run.bat 의 UPnP 자동 개방 주입(injectUpnpToRunBat)은 이번엔
  유지. v0.3.19.

Co-Authored-By: Claude Opus 4 <noreply@anthropic.com>
2026-07-11 00:12:55 +09:00
5 changed files with 25 additions and 122 deletions

View File

@@ -820,16 +820,20 @@ function renderSubStep35(host, back, done) {
var result = await installerApi.checkPortForward(port) var result = await installerApi.checkPortForward(port)
state.serverInstall.portStatus = result state.serverInstall.portStatus = result
var address = formatServerAddress(result.externalIp, result.port) var address = formatServerAddress(result.externalIp, result.port)
if (result.status === 'preForwarded') { if (result.status === 'open') {
resultMsg.innerHTML = tt('step3.sub35.preForwarded', { address: address }) // 열려 있으면 외부 접속 주소를 크게 표시.
resultMsg.classList.add('success') resultMsg.innerHTML =
} else if (result.status === 'upnpOk') { '<div style="margin-top:12px;padding:18px;border:1px solid var(--success);border-radius:10px;background:rgba(63,185,80,0.12);text-align:center;">' +
resultMsg.innerHTML = tt('step3.sub35.upnpOk', { address: address }) '<div style="font-size:14px;color:var(--text-muted);">' + escapeHtml(tt('step3.sub35.openTitle')) + '</div>' +
resultMsg.classList.add('success') '<div style="font-size:30px;font-weight:800;margin-top:8px;user-select:all;word-break:break-all;">' + escapeHtml(address) + '</div>' +
'</div>'
} else { } else {
resultMsg.innerHTML = (result.message || tt('step3.sub35.manualHint')) + // 안 열려 있으면 "직접 포트포워딩 해주세요" 를 크게 표시.
tt('step3.sub35.manualDetail', { address: address }) resultMsg.innerHTML =
resultMsg.classList.add('warn') '<div style="margin-top:12px;padding:18px;border:1px solid var(--danger);border-radius:10px;background:rgba(248,81,73,0.10);text-align:center;">' +
'<div style="font-size:26px;font-weight:800;">' + escapeHtml(tt('step3.sub35.notOpenBig')) + '</div>' +
'<div style="font-size:13px;color:var(--text-muted);margin-top:10px;">' + escapeHtml(tt('step3.sub35.notOpenHint', { address: address, port: result.port })) + '</div>' +
'</div>'
} }
nextBtn.disabled = false nextBtn.disabled = false
} catch (err) { } catch (err) {

View File

@@ -124,14 +124,13 @@
}, },
"sub35": { "sub35": {
"heading": "포트포워딩", "heading": "포트포워딩",
"description": "UPNP를 개방해 외부 접속을 허용합니다.", "description": "포트가 외부에서 열려 있는지 확인합니다. (설치기가 직접 열지는 않습니다)",
"portLabel": "포트", "portLabel": "포트",
"recheck": "재점검", "recheck": "재점검",
"checking": "확인 중...", "checking": "확인 중...",
"preForwarded": "포트포워딩 성공! 친구는 <strong>{{address}}</strong> 주소로 서버에 접속할 수 있습니다. (이미 외부 개방되어 있음)", "openTitle": "포트가 열려 있습니다! 친구 접속 주소",
"upnpOk": "포트포워딩 성공! 친구는 <strong>{{address}}</strong> 주소로 서버에 접속할 수 있습니다. (UPnP로 자동 개방 완료)", "notOpenBig": "직접 포트포워딩 해주세요.",
"manualHint": "직접 포트포워딩을 해주세요.", "notOpenHint": "공유기에서 외부 TCP {{port}} 를 이 PC 로 포워딩하세요. (외부 주소: {{address}})",
"manualDetail": "<br><small>외부 주소: {{address}}</small>",
"checkFailed": "점검 실패: {{message}}", "checkFailed": "점검 실패: {{message}}",
"ipUnknown": "확인 불가" "ipUnknown": "확인 불가"
} }

View File

@@ -1,6 +1,6 @@
{ {
"name": "minecraft-music-quiz-installer", "name": "minecraft-music-quiz-installer",
"version": "0.3.18", "version": "0.3.19",
"description": "마인크래프트 음악퀴즈 간편설치기 + 관리 사이트", "description": "마인크래프트 음악퀴즈 간편설치기 + 관리 사이트",
"main": "dist/installer/main.js", "main": "dist/installer/main.js",
"scripts": { "scripts": {

View File

@@ -860,12 +860,7 @@ ipcMain.handle('server:portForward', async (_event, port: number): Promise<PortF
const targetPort = Number.isFinite(port) && port > 0 ? port : 25565 const targetPort = Number.isFinite(port) && port > 0 ? port : 25565
sendLog(t('log.portCheckStart', { port: targetPort })) sendLog(t('log.portCheckStart', { port: targetPort }))
// 1차 점검 전에 우리가 이전 실행에서 만든 UPnP 매핑이 남아 있으면 먼저 제거한다. // 설치기는 포트를 직접 열지 않는다. 외부에서 이미 열려 있는지만 확인한다.
// 이렇게 해야 "사용자 라우터 규칙이 활성화돼서 외부 접근이 가능한 상태" 와 "UPnP 매핑 덕분에 접근 가능한 상태" 가 구별된다.
// 사용자 규칙이 비활성/없으면 1차 점검은 false 가 되어 UPnP 시도 단계로 자연스럽게 넘어간다.
sendLog(t('log.upnpCleanup'))
await removeUpnpMapping(targetPort)
// 외부 IP 확보: 공용 API → 실패 시 UPnP 게이트웨이의 외부 IP로 폴백. // 외부 IP 확보: 공용 API → 실패 시 UPnP 게이트웨이의 외부 IP로 폴백.
let externalIp = await detectExternalIpHttp() let externalIp = await detectExternalIpHttp()
if (externalIp) { if (externalIp) {
@@ -877,9 +872,9 @@ ipcMain.handle('server:portForward', async (_event, port: number): Promise<PortF
else sendLog(t('log.externalIpUpnpFail')) else sendLog(t('log.externalIpUpnpFail'))
} }
// 1차 점검: 외부에서 이미 접근 가능한지 (서버가 떠 있거나, 우리가 임시 리스너 띄워서 검증). // 외부에서 포트가 닿는지 점검(서버가 떠 있거나, 임시 리스너 검증).
sendLog(t('log.probeStart')) sendLog(t('log.probeStart'))
let probe = await probePortFromOutside(targetPort, externalIp) const probe = await probePortFromOutside(targetPort, externalIp)
if (!externalIp && probe.detectedIp) externalIp = probe.detectedIp if (!externalIp && probe.detectedIp) externalIp = probe.detectedIp
const verdict = probe.reachable === true const verdict = probe.reachable === true
? t('log.probeVerdictSuccess') ? t('log.probeVerdictSuccess')
@@ -888,47 +883,9 @@ ipcMain.handle('server:portForward', async (_event, port: number): Promise<PortF
if (probe.reachable === true) { if (probe.reachable === true) {
sendLog(t('log.probePreForwarded', { addr: externalIp || t('log.ipUnknown'), port: targetPort })) sendLog(t('log.probePreForwarded', { addr: externalIp || t('log.ipUnknown'), port: targetPort }))
return { status: 'preForwarded', externalIp, port: targetPort } return { status: 'open', externalIp, port: targetPort }
} }
return { status: 'notOpen', externalIp, port: targetPort }
// UPnP 시도.
sendLog(t('log.upnpTryOpen', { port: targetPort }))
try {
await openPortViaUpnp(targetPort)
sendLog(t('log.upnpReqOk'))
} catch (error) {
const msg = (error as Error).message || String(error)
sendLog(t('log.upnpTryFail', { message: msg }))
return {
status: 'upnpFailed',
externalIp,
port: targetPort,
message: t('log.upnpFailDetail', { message: msg })
}
}
// NAT 반영 지연을 고려해 최대 3회 재점검.
for (let attempt = 1; attempt <= 3; attempt++) {
await sleep(1500)
sendLog(t('log.upnpRecheck', { attempt }))
probe = await probePortFromOutside(targetPort, externalIp)
if (!externalIp && probe.detectedIp) externalIp = probe.detectedIp
if (probe.reachable === true) {
sendLog(t('log.upnpDone', { port: targetPort }))
await removeUpnpMapping(targetPort)
return { status: 'upnpOk', externalIp, port: targetPort }
}
}
// 테스트 목적으로 만든 매핑 정리. 실제 개방은 run.bat 이 담당.
sendLog(t('log.upnpCleanupTest'))
await removeUpnpMapping(targetPort)
const reason = probe.reachable === false
? t('log.upnpFailReason1')
: t('log.upnpFailReason2', { detail: probe.detail })
sendLog(reason)
return { status: 'upnpFailed', externalIp, port: targetPort, message: reason }
}) })
async function detectExternalIpHttp(): Promise<string> { async function detectExternalIpHttp(): Promise<string> {
@@ -1124,64 +1081,6 @@ function fetchIfconfigCoPortOnce(port: number): Promise<IfconfigPortResult> {
}) })
} }
function removeUpnpMapping(port: number): Promise<void> {
return new Promise((resolve) => {
let settled = false
const done = () => { if (!settled) { settled = true; resolve() } }
let client: ReturnType<typeof natUpnp.createClient> | null = null
try {
client = natUpnp.createClient()
} catch (err) {
sendLog(t('log.upnpClientFailRemove', { message: (err as Error).message }))
done()
return
}
const timer = setTimeout(() => {
try { client && client.close() } catch {}
sendLog(t('log.upnpRemoveTimeout'))
done()
}, 8000)
client.portUnmapping({ public: port, protocol: 'tcp' }, (err: Error | null) => {
clearTimeout(timer)
try { client && client.close() } catch {}
if (err) sendLog(t('log.upnpRemoveAttempt', { message: err.message }))
else sendLog(t('log.upnpRemoveDone', { port }))
done()
})
})
}
function openPortViaUpnp(port: number): Promise<void> {
return new Promise((resolve, reject) => {
let settled = false
const done = (err?: Error) => {
if (settled) return
settled = true
if (err) reject(err)
else resolve()
}
let client: ReturnType<typeof natUpnp.createClient> | null = null
try {
client = natUpnp.createClient()
} catch (err) {
done(err as Error)
return
}
const timer = setTimeout(() => {
try { client && client.close() } catch {}
done(new Error(t('errors.upnpTimeout')))
}, 15000)
client.portMapping(
{ public: port, private: port, ttl: 0, description: 'MusicQuiz Server', protocol: 'tcp' },
(error: Error | null) => {
clearTimeout(timer)
try { client && client.close() } catch {}
done(error || undefined)
}
)
})
}
function sleep(ms: number): Promise<void> { function sleep(ms: number): Promise<void> {
return new Promise((resolve) => setTimeout(resolve, ms)) return new Promise((resolve) => setTimeout(resolve, ms))
} }

View File

@@ -36,7 +36,8 @@ export interface RamCheckResult {
} }
export interface PortForwardResult { export interface PortForwardResult {
status: 'preForwarded' | 'upnpOk' | 'upnpFailed' // 설치기는 포트를 직접 열지 않고 "열려 있는지"만 확인한다.
status: 'open' | 'notOpen'
externalIp?: string externalIp?: string
port: number port: number
message?: string message?: string