Skip to content

feat: added support for log format using NEW_RELIC_FORMAT_LOGS.#128

Merged
nr-rkallempudi merged 8 commits intonewrelic:masterfrom
Sashwatdas123:master
Oct 30, 2025
Merged

feat: added support for log format using NEW_RELIC_FORMAT_LOGS.#128
nr-rkallempudi merged 8 commits intonewrelic:masterfrom
Sashwatdas123:master

Conversation

@Sashwatdas123
Copy link
Contributor

@Sashwatdas123 Sashwatdas123 commented Sep 23, 2025

Detail: The log-ingestion Lambda was unable to process the unique payload formats generated by .NET Lambda functions, leading to integration issues with New Relic. These payloads differed from those initially supported, causing disruptions in data parsing and instrumentation.

Solution: This PR updates the log-ingestion lambda function to support the specific .NET payload formats, resolving the compatibility issue. By implementing the necessary changes, .NET Lambda functions can now integrate seamlessly with New Relic, restoring proper functionality and instrumentation.

@Sashwatdas123 Sashwatdas123 changed the title feat: added support for payload V2 feat: added support for log format using NEW_RELIC_FORMAT_LOGS. Oct 27, 2025
@hrai-nr hrai-nr self-requested a review October 30, 2025 06:23
hrai-nr
hrai-nr previously approved these changes Oct 30, 2025
Copy link
Contributor

@hrai-nr hrai-nr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving based on comments made earlier and discussion with @Sashwatdas123

hrai-nr
hrai-nr previously approved these changes Oct 30, 2025
@nr-rkallempudi nr-rkallempudi merged commit 336cf93 into newrelic:master Oct 30, 2025
5 checks passed

LAMBDA_LOG_GROUP_PREFIX = os.getenv("NR_LAMBDA_LOG_GROUP_PREFIX", "/aws/lambda")
VPC_LOG_GROUP_PREFIX = os.getenv("NR_VPC_LOG_GROUP_PREFIX", "/aws/vpc/flow-logs")
NEW_RELIC_FORMAT_LOGS = os.getenv("NEW_RELIC_FORMAT_LOGS", "false").lower() == "true"
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why did we add this environment variable? Do we want to disable this by default?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants