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>
This commit is contained in:
@@ -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) {
|
||||||
|
|||||||
@@ -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": "확인 불가"
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -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": {
|
||||||
|
|||||||
@@ -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))
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -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
|
||||||
|
|||||||
Reference in New Issue
Block a user