Skip to content

Fix test-teams-api.sh to support GitHub Codespaces - #3

Merged
pmca31 merged 2 commits into
build-octofit-appfrom
copilot/sub-pr-2
Feb 11, 2026
Merged

Fix test-teams-api.sh to support GitHub Codespaces#3
pmca31 merged 2 commits into
build-octofit-appfrom
copilot/sub-pr-2

Conversation

Copilot AI commented Feb 11, 2026

Copy link
Copy Markdown
Contributor

The test-teams-api.sh script was hard-coded to http://localhost:8000, preventing it from working in GitHub Codespaces where the backend uses a dynamic URL.

Changes

  • Added CODESPACE_NAME environment variable detection
  • Dynamic base URL construction:
    • https://${CODESPACE_NAME}-8000.app.github.dev in Codespaces
    • http://localhost:8000 locally
  • Matches the pattern used in test_api.sh
# Detect if running in GitHub Codespaces
CODESPACE_NAME=${CODESPACE_NAME:-"localhost"}

if [ "$CODESPACE_NAME" != "localhost" ]; then
    BASE_URL="https://${CODESPACE_NAME}-8000.app.github.dev"
else
    BASE_URL="http://localhost:8000"
fi

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: pmca31 <6774003+pmca31@users.noreply.github.com>

Copilot AI commented Feb 11, 2026

Copy link
Copy Markdown
Contributor Author

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • test-codespace-123-8000.app.github.dev
    • Triggering command: /usr/bin/curl curl -s REDACTED (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Copilot AI changed the title [WIP] Update registration validation logic based on feedback Fix test-teams-api.sh to support GitHub Codespaces Feb 11, 2026
Copilot AI requested a review from pmca31 February 11, 2026 19:20
@pmca31
pmca31 marked this pull request as ready for review February 11, 2026 19:46
@pmca31
pmca31 merged commit 7caf193 into build-octofit-app Feb 11, 2026
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.

2 participants