openobserve/openobserve

Unify the usage of reqwest/awc in the codebase.

Open

#3.700 geöffnet am 9. Juni 2024

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)Rust (537 Forks)batch import
good first issue✏️ Feature

Repository-Metriken

Stars
 (14.265 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 1T 2h) (399 gemergte PRs in 30 T)

Beschreibung

Which OpenObserve functionalities are relevant/related to the feature request?

other

Description

Throughout the codebase both reqwest and awc are being used as http clients. Need to pick one and use it throughout.

Proposed solution

Use reqwest and remove awc. Add tests for the corresponding change.

Alternatives considered

None

Contributor Guide

Research-Richtung
Identifizieren Sie alle Verwendungen von reqwest und awc im Code. Bestimmen Sie den besten Ansatz für die Migration von awc zu reqwest, um die Funktionsgleichheit sicherzustellen. Schreiben Sie Tests, um zu überprüfen, ob alle HTTP Aufrufe nach der Migration noch korrekt funktionieren.
Tech Stack
rust
Domain
backendobservability sre
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
3-5 Tage
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
RustGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65