cli: refactor logic for connecting to database
- Lingua principale
- Nessun dato sulla lingua
- Stelle
- 3
- Fork
- 0
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Descrizione
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.
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Direzione di ricerca
Leggi le implementazioni dei comandi server, server create-admin-user e server regenerate-vapid-keypair, quindi confrontale con i comandi enterprise server dbcrypt. Traccia il modo in cui ciascuno gestisce Postgres integrato e le connessioni al database; il lavoro è completato quando i comandi usano un modello standard unificato senza duplicare quella logica condizionale.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Ambito
- cli, database
- Tipo di issue
- Refactoring
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 45/100