Support `psql` as part of the server host
- Vorherrschende Sprache
- TypeScript
- Sterne
- 25
- Forks
- 22
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
According to the [CAF (Cloud Adoption Framework)](https://learn.microsoft.com/en-us/azure/cloud-adoption-framework/ready/azure-best-practices/resource-abbreviations) the official abbreviation for a PostgreSQL database is `psql`.
But the current implementation blindly replaces [`'psql'` with `''`](https://github.com/Azure/postgresql/blob/1d47d25bbb9375f1192ff806291f807dd2ce9e92/src/Utils/ActionInputs.ts#L27C1-L28C1), which breaks the ability to use this action with such a named DB.
**Input**:
```yml
- 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:
```yml
- uses: azure/postgresql@v1
with:
connection-string: host=caf-named-db-p"sql.postgres.database.azure.com ...
```
But this is far from ideal 😂
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Rechercherichtung
Beginne in src/Utils/ActionInputs.ts bei der verlinkten Logik, die `psql` und Anführungszeichen aus der Verbindungszeichenfolge entfernt. Reproduziere das Problem mit einem Host wie `caf-named-db-psql.postgres.database.azure.com` und überprüfe anschließend, dass die Action `psql` beibehält, wenn es Teil des Hosts ist, während die Verbindungszeichenfolge weiterhin korrekt geparst wird.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- github-actions, postgresql, typescript
- Bereich
- database, devops
- Issue-Typ
- Bug
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Veraltet
- Klarheit
- Klar beschrieben
- Anfängerfreundlichkeit
- 45/100