coder / coder/internal

cli: refactor logic for connecting to database

Aperta
#541 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
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

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.