cli: refactor logic for connecting to database
- Vorherrschende Sprache
- Keine Sprachdaten
- Sterne
- 3
- Forks
- 0
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
I'm quite sure I have seen this pattern in other CLI commands:
startBuiltinPostgres - codersdk.PostgresAuth - ConnectToPostgres
we may consider some base pattern for all commands using the database
_Originally posted by @mtojek in https://github.com/coder/coder/pull/17091#discussion_r2014730400_
We have a few commands now that can connect to the database directly and need to handle the conditional logic of builtin postgres:
- server
- server create-admin-user
- server regenerate-vapid-keypair
We should refactor and unify this.
We also have the `server dbcrypt` commands under `enterprise/` that could also benefit from this standardized logic.
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Rechercherichtung
Lies die Implementierungen der Befehle server, server create-admin-user und server regenerate-vapid-keypair und vergleiche sie mit den enterprise-server-dbcrypt-Befehlen. Verfolge, wie jeder Befehl mit integriertem Postgres und Datenbankverbindungen umgeht; abgeschlossen ist die Aufgabe, wenn die Befehle ein einheitliches Standardmuster verwenden, ohne diese bedingte Logik zu duplizieren.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Bereich
- cli, database
- Issue-Typ
- Refactoring
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 45/100