Add support for IHealthCheck

Open
  • Is something wrong?
#658 5 comments 1 reaction 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
5/5
Estimated time
Over a week
Newbie friendliness
20/100
Issue type
Feature
Clarity
Needs clarification
Activity status
Stale
Tech stack
csharp
Domain
observability

Research direction

Start by reviewing the referenced AspNetCore.Diagnostics.HealthChecks src tree and the proposed NLog.InternalLogger-events and NLog Target concepts. Define the scope for netcoreapp3.1 and net6.0, including alert counting and expiration, then establish tests showing that recent target alerts are reported and older alerts are excluded.

Written by the indexing model from the issue text.

Description

ASP.NET Core feature up-for-grabs

Something similar to these:

https://github.com/Xabaril/AspNetCore.Diagnostics.HealthChecks/tree/master/src

Maybe something like this:

  • Setup static capture of NLog.InternalLogger-events on Warning and Error-Level.
  • Keeps tracks of alert count per NLog Target within the last 10 minutes.
  • Reports all NLog Targets (or NLog Core) that have generated alerts within the last 10 minutes (Automatically expire old alerts).
Dominant language
C#
Stars
327
Forks
161
Avg merge
1d 41m
Merged PRs (30d)
7

Getting set up

This project ships no dev container, Dockerfile or contributing guide, so setting up is up to you: start from its README, and see our first-contribution guide for the general steps.

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from NLog/NLog.Web

All issues in NLog/NLog.Web

Similar issues

More C# issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.