NLog / NLog/NLog.Web

Add support for IHealthCheck for netcoreapp3.1 + net6.0

Open
#658 5 comments 1 reaction 0 assignees View on GitHub

Nobody has claimed this yet.

ASP.NET Core feature up-for-grabs
Dominant language
C#
Stars
328
Forks
161
Avg merge
13h 1m
Merged PRs (30d)
5

Description

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).

Contributor guide

No contributing guide indexed for this repository

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.

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.

Assessment

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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.