coder / coder/internal

cli: refactor logic for connecting to database

Offen
#541 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
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

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.