Skip to content

feat: add Astro CLI reverse proxy discovery for local Airflow#165

Draft
josh-fell wants to merge 1 commit intomainfrom
josh-fell/af-reverse-proxy
Draft

feat: add Astro CLI reverse proxy discovery for local Airflow#165
josh-fell wants to merge 1 commit intomainfrom
josh-fell/af-reverse-proxy

Conversation

@josh-fell
Copy link
Contributor

Summary

  • Adds reverse proxy route discovery to LocalDiscoveryBackend, checking ~/.astro/proxy/routes.json and global config for proxy port before falling back to port scanning
  • Introduces helper functions in constants.py for resolving ASTRO_HOME, proxy routes path, and global config path
  • Refactors discover_airflow_url in __main__.py to return proxy-matched URLs with source indication
  • Adds comprehensive tests for proxy discovery, port fallback, and coexistence scenarios

Test plan

  • Unit tests for proxy port reading (default and custom)
  • Unit tests for missing/invalid routes.json handling
  • Tests for proxy + port-scan coexistence
  • Tests for discover_airflow_url proxy match and fallthrough paths

🤖 Generated with Claude Code

Add proxy route discovery to LocalDiscoveryBackend and
discover_airflow_url, checking ~/.astro/proxy/routes.json and global
config before falling back to port scanning.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@josh-fell josh-fell force-pushed the josh-fell/af-reverse-proxy branch from 1cf2d64 to eda600f Compare March 20, 2026 17:22
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.

1 participant