datalust / datalust/seq-extensions-logging

Capture `ExpandoObject` as dictionaries/structures

Open
#47 5 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement
Dominant language
C#
Stars
99
Forks
14
PR merge metrics
No merged PRs in 30d

Description

When logging an object that has a property of type Dictionary using @ then it's shown as expected in Seq like:

{
  DictionaryObj: {
    'ItemA': 100,
    'ItemB': 200,
  }
}

But when logging the dictionary object directly, it shows as a list of KeyValuePairs like:

[ItemA, 100], [ItemB, 200]

It is not consistent in how a Dictionary object is displayed and I highly prefer it to show as an object than a list, as in:

{
  'ItemA': 100,
  'ItemB': 200,
}

Update: dynamic objects which are dictionaries underneath already log as expected as objects. ExpandoObjects also need to be fixed to log as objects.

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

No file, test, or entry point is named. Inspect the handling of ExpandoObject and compare it with the existing dynamic-dictionary path; done means an ExpandoObject logs as an object with named properties rather than a list of key-value pairs, without regressing current dictionary logging.

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp
Domain
backend, observability-sre
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.