Skip to content

업데이트 설치 파일 선택 조건 수정 - #310

Merged
rkttu merged 1 commit into
yourtablecloth:mainfrom
john-ver:fix/update-installer-selection
Oct 2, 2026
Merged

rkttu merged 1 commit into
yourtablecloth:mainfrom
john-ver:fix/update-installer-selection

Conversation

@john-ver

@john-ver john-ver commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor

TableCloth 1.16.3에서 업데이트 시 SporkBootstrap이 다운로드됐습니다. TableCloth 업데이트 파일로 알고 실행했는데, Spork가 Windows Sandbox가 아닌 실제 Windows 환경에서 실행됐고 이후 선택한 은행 사이트의 보안 프로그램까지 실제 환경에 설치됐습니다.

이미 #283에서 인지하신 문제로 보이지만, 최신 버전의 코드에도 자동 업데이트 실패 시 사용하는 GitHub 릴리스 다운로드 경로에 제품명 확인이 빠져 있습니다. 같은 릴리스에 TableCloth 외에 Spork·SporkBootstrap 실행 파일도 함께 게시되므로, 이 경로를 사용하는 경우 같은 문제가 발생할 가능성이 남아 있습니다.

설치 파일 선택 조건에 TableCloth 제품명을 추가하고, 아키텍처와 배포 채널이 정확히 일치하도록 수정했습니다. 맞는 설치 파일이 없을 때 첫 번째 exe 파일을 선택하던 코드도 제거했습니다. 이 경우에는 기존 릴리스 페이지로 안내됩니다. 이렇게 수정하면 해당 문제를 막을 수 있을 것 같아 제안드립니다.

@CLAassistant

CLAassistant commented Oct 2, 2026 •

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@rkttu rkttu left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

제품, 아키텍처, Retail/Preview 채널 선택과 일치하는 설치 파일이 없을 때의 동작을 검토했습니다. main과 develop의 로컬 전체 Release 테스트 및 x64/ARM64 Debug/Release CI를 통과했으며 병합을 막을 문제를 발견하지 못했습니다.

rkttu commented Oct 2, 2026

Copy link
Copy Markdown
Member

상세한 제보와 수정 PR에 감사드립니다. 최신 코드에도 남아 있던 제품명 확인 누락과 첫 번째 EXE 선택 경로를 확인했으며, 제안하신 수정 방향에 동의합니다.

main 병합 결과를 .NET 10에서 검증하여 TableCloth.Test 146개와 Spork.Test 43개를 통과했습니다. 현재 develop에도 충돌 없이 적용한 뒤 .NET 11에서 TableCloth.Test 243개, ManagedAi.Test 130개, Spork.Test 43개를 통과했습니다. 두 환경 모두 이번 PR의 회귀 테스트 11개를 포함합니다. GitHub Actions도 x64와 ARM64의 Debug/Release 빌드, 테스트 및 패키징을 모두 통과했습니다.

현재 Preview.5의 설치 파일명과 preview-x64/preview-arm64 채널 처리에도 충돌을 발견하지 못했습니다. main에 병합하고 develop에도 같은 수정을 반영하겠습니다.

다만 이미 설치한 1.16.x의 업데이트 로직까지 바꾸는 수정은 아니므로, 해당 버전 사용자는 GitHub Releases에서 TableCloth 설치 파일을 직접 받아 업그레이드하면 됩니다. 이번 수정본을 배포하기 전까지는 기존 버전의 GitHub 대체 다운로드 경로에도 결함이 남아 있습니다.

재현 사례와 회귀 테스트를 함께 제공해 주셔서 감사합니다.

@rkttu
rkttu merged commit faeb0e9 into yourtablecloth:main Oct 2, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants