Xabaril/AspNetCore.Diagnostics.HealthChecks

Redis Health Check

Open

#750 建立於 2021年3月30日

在 GitHub 查看
 (3 留言) (8 反應) (0 負責人)C# (3,776 star) (737 fork)batch import
help wanted

描述

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

貢獻者指南

技術棧
csharpredis
領域
backendobservability
議題類型
feature
難度面向新貢獻者的預計實作難度，1 表示很小改動，5 表示專家級工作。
2
預計時間有經驗貢獻者完成調查、實作、測試並準備 pull request 的粗略時間範圍。
1-3 hours
活動狀態議題目前的可參與程度：新鮮、活躍、陳舊、阻塞或等待維護者輸入。
fresh
清晰度議題是否清楚說明預期改動、驗收標準和下一步。
clear
前置要求
C#ASP.NET CoreRedis
新手友善度1-100 的估計分數，表示該議題對首次貢獻者的友善程度。
70
研究方向
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.