--caproverUrl ignores port; cannot deploy to CapRover instances not on 443

Abierto
  • ¿Algo no está bien?
#153 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
58/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Estancado
Stack tecnológico
typescript
Área
cli

Línea de trabajo

Localiza el punto de entrada cleanAdminDomainUrl utilizado por caprover deploy --caproverUrl e inspecciona cómo se analiza y reconstruye la URL de entrada. Verifica que una URL como https://captain.example.com:8443 conserve el puerto 8443, preserve el comportamiento del prefijo captain y se conecte al endpoint proporcionado.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

Description

When passing a URL with an explicit port to caprover deploy --caproverUrl, the CLI strips the port and forces a request to port 443. This breaks deployments where CapRover is not exposed on 443.

Example

caprover deploy --caproverUrl https://captain.example.com:8443

Ensuring authentication...
Deploying *** to https://captain.example.com...

Something bad happened: cannot deploy *** at https://captain.example.com.
Expected Behavior

The CLI would respect the provided port (8443) and connect to it.

Actual Behavior

The CLI silently removes the :8443 and connects to https://captain.example.com on port 443.

Justification

CapRover may be deployed behind shared ingress/load balancers, reverse proxies, or on alternate ports, internal environments, or constrained hosting (It's sort of supported). In these setups, CapRover isn’t always a first-class service on 443.

For example, I've been recently deploying a HTTPS load balancing endpoint, using CapRover to deploy it. As a result CapRover has to be moved to a different port (.e.g 8443).

Proposed Solution

Preserve port in cleanAdminDomainUrl:
Parse the input (url.parse), keep hostname and port, enforce 'captain.' prefix if missing, and when rebuilding the URL, append :${port} if present.

Lenguaje dominante
TypeScript
Estrellas
78
Forks
42
Sin PR fusionados en 30 d

Preparar el entorno

  • Incluye un Dockerfile o un archivo de Docker Compose
  • Tiene una plantilla de pull request
  • Sin guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de caprover/caprover-cli

Todos los issues de caprover/caprover-cli

Issues similares

Más issues de TypeScript

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.