SigNoz/signoz

Move db calls to prepared statements with context

Aperta

#1353 aperta il 4 lug 2022

Vedi su GitHubGuida contributor
 (11 commenti) (0 reazioni) (1 assegnatario)TypeScript (976 fork)batch import
backendgood first issue

Metriche repository

Star
 (16.037 stelle)
Metriche merge PR
 (Merge medio 5g 22h) (224 PR mergiate in 30 g)

Descrizione

Move all db calls to prepared statements and specifically with context if possible to make signoz more secure from sql injections. A query should not be a string prepared from fmt.sprintf(...) if it has args to pass. We should try to avoid string formatting for args.

Guida contributor

Direzione di ricerca
Esamina il livello di accesso al database del repository per identificare tutte le posizioni in cui le query SQL vengono costruite utilizzando la formattazione di stringhe (ad es. fmt.Sprintf). Rivedi i commenti dell'issue per i modelli specifici menzionati dai manutentori. Concentrati sui file nelle directory 'pkg' o 'server' che gestiscono le operazioni del database. Cerca l'uso esistente di istruzioni preparate per comprendere il modello preferito. Considera di iniziare con un modulo per dimostrare l'approccio prima di espanderlo all'intera codebase.
Tech stack
typescript
Dominio
backend
Tipo issue
Sicurezza
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitTypeScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70