Skip to content

Fix GitHub Enterprise host handling #70

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
gkaleta merged 2 commits into from
May 1, 2026
+6 −2
Merged

Fix GitHub Enterprise host handling #70

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
0678412
Fix GitHub Enterprise host handling
Mar 31, 2026
0ed168d
review: apply review feedback for PR #70
jkordick Apr 17, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 6 additions & 2 deletions doctor.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,10 @@ CYAN='\033[0;36m'
MAGENTA='\033[0;35m'
BOLD='\033[1m'
NC='\033[0m' # No Color
# Variable for the GitHub host, defaults to 'github.com'
GITHUB_HOST="${GITHUB_HOST:-github.com}"
GITHUB_HOST="${GITHUB_HOST#https://}"
GITHUB_HOST="${GITHUB_HOST#http://}"

# Resolve the sqlite3 command, handling Windows (Git Bash / MSYS2) paths
SQLITE3_CMD=""
Expand Down Expand Up @@ -1168,7 +1172,7 @@ run_setup() {
echo -e " ${BLUE}Classic PAT (fine-grained PATs do not currently support Copilot):${NC}"
echo " • copilot"
echo ""
echo -e "${YELLOW}Create one at: https://github.com/settings/tokens${NC}"
echo -e "${YELLOW}Create one at: https://${GITHUB_HOST}/settings/tokens${NC}"
echo ""
# Read from /dev/tty explicitly to ensure correct capture in all terminal environments
echo -n "Please provide the PAT and press Enter: "
Expand All @@ -1185,7 +1189,7 @@ run_setup() {
# --- CLI authentication (existing flow) ---
echo -e "${BLUE}🔐 Authenticating with GitHub Copilot...${NC}"
echo ""
if ! copilot login; then
if ! copilot login --host "https://${GITHUB_HOST}"; then
Copy link

Copilot AI Apr 17, 2026

Choose a reason for hiding this comment

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

copilot login --host "https://${GITHUB_HOST}" will produce an invalid URL if GITHUB_HOST is provided with a scheme already (e.g., https://ghe.example.com → https://https://...). Consider normalizing GITHUB_HOST (accept either hostname or full URL) before building the --host argument, and reuse the same host for the PAT instructions URL in this setup flow so GitHub Enterprise users aren’t directed to github.com.

Copilot uses AI. Check for mistakes.
echo ""
echo -e "${RED}❌ Authentication failed. Please try again.${NC}"
return 1
Expand Down