Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issues with JSON formatted logs #32

Open
jalenplayvs opened this issue Feb 12, 2021 · 3 comments
Open

Issues with JSON formatted logs #32

jalenplayvs opened this issue Feb 12, 2021 · 3 comments
Labels
enhancement New feature or request

Comments

@jalenplayvs
Copy link

Migrating from LogDNA-agent to the Lambda integration, our JSON logs are not properly formatted in the LogDNA web app. The JSON body gets converted to a string, and log lines displayed in the web app as a string.

In the interim, we made our own fork with a hacky fix that would break other use-cases. But we would like the upstream package to have better support for JSON formatted logs so that we do not have to maintain a patch.

Setup:
Follow these instructions (NOTE: The link to the zip file is an old version sitting in an S3 bucket, we downloaded the latest from github). We did port the instructions to Terraform, it is possible that something is different than what the manual instructions would produce, but I could not find a meaningful difference.

Steps to reproduce:

  • Run an application with JSON formatted logs, ingest logs using LogDNA agent
  • Run the same application, use the Cloudwatch->Lambda->LogDNA integration to ingest

Expected result:
The log lines look the same in the LogDNA web app

Actual result:
The logs lines look different, are not properly parsed as JSON data

@smusali smusali self-assigned this May 3, 2021
@smusali smusali added the enhancement New feature or request label May 3, 2021
@smusali
Copy link
Contributor

smusali commented May 3, 2021

Sorry for the late reply, @jalenplayvs, and thanks a lot for reporting this use case!

We will bring this enhancement to our lambda function as soon as possible and keep you updated!

@smusali smusali removed their assignment Sep 6, 2021
@madarve
Copy link

madarve commented Jun 23, 2022

Hi, Is there any movement on this?

@steve-scx
Copy link

steve-scx commented Feb 9, 2023

This does seem to be fixed in version v2.2.1. Try using LOG_RAW_EVENT=TRUE.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants