-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Feature/port parameterization and external supabase option #129
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.
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
Open
tazmon95
wants to merge
10
commits into
coleam00:main
Choose a base branch
from
tazmon95:feature/port-parameterization-and-external-supabase
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Feature/port parameterization and external supabase option #129
tazmon95
wants to merge
10
commits into
coleam00:main
from
tazmon95:feature/port-parameterization-and-external-supabase
Conversation
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
* Remove all port mappings between the host machine and the containers * Add expose elements for informative purposes * Add concept of environment and reset Supabase ports in public environment * Remove network_mode host from Caddy service * Open ports on localhost, when in private environment * Update README.md to include explanation of environment argument * Comment out endpoints without authentication in Caddyfile * Fix Caddy configuration to use container names and not localhost * Update README.md Added "-b stable" as this is now the release state * Updating .env.example to go along with Supabase .env changes * Note in README for deploying to VPS without Docker Compose --------- Co-authored-by: Benny Tops <[email protected]> Co-authored-by: Leex <[email protected]>
- **Files not Found in Supabase Folder** to troubleshooting section
# Pool size for internal metadata storage used by Supavisor # This is separate from client connections and used only by Supavisor itself POOLER_DB_POOL_SIZE=5
## Summary - Add centralized port configuration in .env file for all services - Add support for external Supabase installations via --ext-supabase flag - Improve script output with cleaner logging and warning suppression - Add comprehensive CLAUDE.md documentation for development guidance ## Features Added ### Port Parameterization - Centralized PORTS section in .env.example with environment variables for all service ports - External ports (host access) and internal ports (inter-service) separately configurable - Configurable bind IP address for private environment (default: 127.0.0.1) - Updated docker-compose.yml, override files, and Caddyfile to use port variables - Backward compatibility maintained with sensible defaults ### External Supabase Support - New --ext-supabase PATH flag to use existing Supabase installations - Dual-mode operation: internal (original) vs external Supabase - Safety checks to prevent conflicts between internal/external modes - Smart output filtering with orphan warning suppression for external mode - Quiet mode for cleaner Docker Compose output in external mode ### Enhanced Script Functionality - Improved start_services.py with better error handling and user guidance - Safety warnings when switching between internal/external modes - Clean separation of concerns with dedicated functions for each mode - Enhanced logging with selective verbosity control ### Documentation - Comprehensive CLAUDE.md with architecture overview and development commands - Port configuration guidance and troubleshooting information - Clear explanation of dual-mode operation and use cases ## Benefits - Easy port conflict resolution through centralized configuration - Flexible deployment options for users with existing Supabase instances - Improved developer experience with better documentation and tooling - Cleaner output and reduced noise in external Supabase mode - Maintains full backward compatibility for existing users 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
- Keep port parameterization changes in all files - Preserve external Supabase functionality in start_services.py - Maintain enhanced Caddyfile with environment variable support - Add SearXNG troubleshooting note from upstream - Resolve whitespace and formatting conflicts 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
…tures - Document --ext-supabase flag usage and requirements - Explain benefits of external Supabase mode - Add port parameterization configuration section - Include examples and best practices 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
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.
🔧 Port Parameterization
🔗 External Supabase Support
📚 Documentation & Polish