docs: 방화벽 · 프록시 점검 가이드 페이지 추가 - #18
Merged
Merged
Conversation
Add docs/firewall.html, a standalone bilingual (KO/EN) guide listing every outbound endpoint CopilotWatchTower contacts, so admins can pre-clear firewall/proxy rules before installing. - Required domains (Entra ID auth, Microsoft Graph, auth CDNs, blob storage) and optional ones (Purview eDiscovery, Power Platform/Copilot Studio, GitHub release download), each with port and requesting component (Node vs Chromium vs OS browser). - Copy-to-clipboard allowlist block. - Warns that the Node collection path uses global fetch and ignores system proxy/PAC settings while the embedded Chromium windows honour them, so interactive sign-in can succeed while background collection silently fails. - Warns that TLS-inspection appliances break the Node path since it trusts only bundled root CAs; suggests bypass or NODE_EXTRA_CA_CERTS. - Loopback note for the admin-consent callback, PowerShell connectivity test snippets, and a symptom/cause troubleshooting table. Link it from index.html via the nav, a banner in the download section, and the footer, and fix a pre-existing horizontal overflow of the header on narrow viewports. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
요약
설치 전에 점검·개방해야 할 방화벽/프록시 항목을 정리한 별도 페이지
docs/firewall.html을 추가하고, 소개 사이트(docs/index.html)에서 링크했습니다.소스 코드를 전수 조사해 앱이 실제로 호출하는 아웃바운드 엔드포인트만 수록했습니다.
페이지 내용
login.microsoftonline.com,graph.microsoft.com,*.msftauth.net/*.msauth.net,login.live.com,*.blob.core.windows.netadmin.powerplatform.microsoft.com,licensing.powerplatform.microsoft.com,*.bap.microsoft.com,*.powerplatform.com,*.dynamics.com등), GitHub 릴리스 다운로드127.0.0.1임시 포트특히 강조한 두 가지 함정
src/main/http.ts의 전역fetch를 쓰므로 시스템 프록시·HTTP(S)_PROXY·PAC를 무시합니다. 반면 포털 로그인 창(Chromium)은 프록시를 따릅니다. 그래서 로그인은 되는데 수집만 조용히 실패하는 증상이 납니다. 해당 도메인을 프록시 바이패스에 넣거나 직접 443 아웃바운드를 허용해야 합니다.unable to verify the first certificate가 발생합니다. 인증·Graph 도메인 검사 제외를 권장하고, 대안으로NODE_EXTRA_CA_CERTS를 안내합니다.docs/index.html변경상단 내비게이션, 다운로드 섹션 배너, 푸터 3곳에 링크를 추가했습니다(한/영 번역 포함). 기존 사이트의
data-i18n방식과cwt-langlocalStorage 키를 그대로 사용해 두 페이지 간 언어 선택이 유지됩니다.작업 중 발견한, 좁은 화면에서 헤더가 가로로 넘치던 기존 문제도 함께 고쳤습니다.
검증
로컬 정적 서버로 두 페이지를 데스크톱(1280px)·모바일(390px), 한국어·영어 전 조합에서 확인했습니다. 콘솔 에러 0건, 전체 표 렌더링 정상, 언어 토글·복사 버튼 정상 동작, 가로 스크롤 없음.