log4js-node / log4js-node/loggly

Include timestamp field

Open
#13 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
JavaScript
Stars
5
Forks
3
PR merge metrics
No merged PRs in 30d

Description

Loggly supports sending a timestamp in the logs. If you dont, it will use the http(s) reception time of the log as the timestamp as described here : https://documentation.solarwinds.com/en/success_center/loggly/content/admin/timestamps.htm
The issue with the reception time of the log is that when log events are very close timewise they can arrive out of order on the loggly side which can be confusing when tracking down issues.

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

The issue names no file, test, or entry point. Start by locating the Loggly appender implementation and its existing tests, then trace how log events are assembled and sent. Done means the appender supplies a timestamp and tests verify the resulting Loggly payload and ordering-related behavior.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
observability
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.