Skip to content
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

Simplify and Optimize Self-Hosted VPN Installation Script #3148

Open
masterbpro opened this issue Jan 3, 2025 · 0 comments
Open

Simplify and Optimize Self-Hosted VPN Installation Script #3148

masterbpro opened this issue Jan 3, 2025 · 0 comments

Comments

@masterbpro
Copy link

Is your feature request related to a problem? Please describe.
The installation script for the self-hosted VPN is overly complex and challenging to integrate effectively. Key issues observed include:

  1. Overly intricate logic and redundant code, making maintenance and customization difficult.
  2. The Caddy server occupies default ports 80/443, potentially causing conflicts with other services.
  3. Integration with Zitadel is cumbersome and error-prone, requiring manual interventions and additional configurations.

Describe the solution you'd like

  1. Refactor the script to remove redundant code and simplify logic, ensuring maintainability and ease of use.
  2. Allow customization of Caddy's default ports to avoid conflicts with existing services.
  3. Provide a streamlined and automated integration process with Zitadel to minimize user effort and potential errors.

Describe alternatives you've considered

  • Manually adjusting the script and configurations for our use case.
  • Exploring alternative tools or approaches for self-hosted VPN deployment.
  • Using custom scripts to bypass integration challenges, which increases complexity.

Additional context
Notable issues within the code:

  • Functions such as initEnvironment and init_crdb are bloated with unnecessary steps.
  • Port handling for Caddy is rigid, causing conflicts in multi-service environments.
  • Integration logic for Zitadel (e.g., init_zitadel, create_new_application) is verbose and prone to errors.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant