Xabaril/AspNetCore.Diagnostics.HealthChecks

Redis Health Check

Open

#750 geöffnet am 30. März 2021

Auf GitHub ansehen
 (3 Kommentare) (8 Reaktionen) (0 zugewiesene Personen)C# (3.776 Stars) (737 Forks)batch import
help wanted

Beschreibung

What would you like to be added:

I have already registered a singleton IConnectionMultiplexer to my IoC container and I would like my health check to use that instead.

// Func<IServiceProvider, IConnectionMultiplexer>
.AddRedis(sp => sp.GetRequiredService<IConnectionMultiplexer>())

Why is this needed:

To use the same connection as the application

Contributor Guide

Tech Stack
csharpredis
Domain
backendobservability
Issue Type
feature
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 hours
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
fresh
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
C#ASP.NET CoreRedis
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70
Research-Richtung
The issue requests a new overload of the AddRedis extension method that accepts an IConnectionMultiplexer parameter. The change is likely in the HealthChecks.Redis project, specifically in the RedisHealthCheckBuilderExtensions class. A new method should be created that uses the provided connection multiplexer instead of creating a new one, following the existing pattern for dependency injection. Review the current AddRedis method signatures and the HealthCheckBuilderExtensions file to understand the expected implementation.