Support `psql` as part of the server host
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 45/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Estancado
- Stack tecnológico
- github-actions, postgresql, typescript
Línea de trabajo
Comienza en src/Utils/ActionInputs.ts, en la lógica enlazada que elimina psql y las comillas de la cadena de conexión. Reproduce el problema con un host como caf-named-db-psql.postgres.database.azure.com y, a continuación, verifica que la action conserve psql cuando forme parte del host, mientras sigue analizando correctamente la cadena de conexión.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
According to the CAF (Cloud Adoption Framework) the official abbreviation for a PostgreSQL database is psql.
But the current implementation blindly replaces 'psql' with '', which breaks the ability to use this action with such a named DB.
Input:
- uses: azure/postgresql@v1
with:
connection-string: host=caf-named-db-psql.postgres.database.azure.com ...
Output:
psql: error: could not translate host name "caf-named-db-.postgres.database.azure.com" to address: Name or service not known
Error: Error occurred while running action:
Error: The process '/usr/bin/psql' failed with exit code 2
Personal workaround:
The code currently not only replaces 'psql' with '', it also (as a second step) replaces '"' with ''. So a functional workaround that I use is adding a rouge " like this:
- uses: azure/postgresql@v1
with:
connection-string: host=caf-named-db-p"sql.postgres.database.azure.com ...
But this is far from ideal 😂
- Lenguaje dominante
- TypeScript
- Estrellas
- 25
- Forks
- 22
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de Azure/postgresql
-
Upgrade to Node.js 24 Abiertoneed-to-triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Azure/postgresql#58 ·
-
need-to-triage
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
Azure/postgresql#55 ·
-
need-to-triage
Dificultad 5/5 Más de una semana Aptitud para principiantes 20/100
Azure/postgresql#53 · 1 comentario ·
-
need-to-triage
Dificultad 3/5 1-2 días Aptitud para principiantes 25/100
Azure/postgresql#52 · 1 comentario ·
-
need-to-triage
Dificultad 5/5 Más de una semana Aptitud para principiantes 15/100
Azure/postgresql#41 ·
Todos los issues de Azure/postgresql
Issues similares
-
bug(cli): hapi doctor inline-media prints a fabricated B:\ helper-script path in packaged installs Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
-
Crush Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
catppuccin/catppuccin#3125 ·
-
Add a SECURITY.md Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
ElementsProject/cln-application#167 · 1 comentario · 1 reacción ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Quantco/pnpm-licenses#17 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100