SigNoz/signoz

Move db calls to prepared statements with context

Offen

#1.353 geöffnet am 04.07.2022

Auf GitHub ansehenContributor Guide
 (11 Kommentare) (0 Reaktionen) (1 zugewiesene Person)TypeScript (976 Forks)batch import
backendgood first issue

Repository-Metriken

Stars
 (16.037 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 5T 22h) (224 gemergte PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Untersuchen Sie die Datenbankzugriffsschicht des Repositorys, um alle Stellen zu identifizieren, an denen SQL Abfragen mithilfe von String Formatierung (z. B. fmt.Sprintf) erstellt werden. Überprüfen Sie die Issue Kommentare auf spezifische Muster, die von den Maintainern erwähnt wurden. Konzentrieren Sie sich auf Dateien in den Verzeichnissen 'pkg' oder 'server', die Datenbankoperationen verarbeiten. Suchen Sie nach vorhandener Verwendung von vorbereiteten Anweisungen, um das bevorzugte Muster zu verstehen. Erwägen Sie, mit einem Modul zu beginnen, um den Ansatz zu demonstrieren, bevor Sie ihn auf die gesamte Codebasis ausweiten.
Tech Stack
typescript
Domain
backend
Issue Type
Sicherheit
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitTypeScript
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70