Skip to content

[lambda_logs] fix parsing failure when message is string#17398

Open
gpop63 wants to merge 3 commits intoelastic:mainfrom
gpop63:issues/15375
Open

[lambda_logs] fix parsing failure when message is string#17398
gpop63 wants to merge 3 commits intoelastic:mainfrom
gpop63:issues/15375

Conversation

@gpop63
Copy link
Copy Markdown
Contributor

@gpop63 gpop63 commented Feb 12, 2026

Proposed commit message

this PR fixes lambda_logs parsing failure when aws.lambda.message receives a string value instead of an object.

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.
  • I have verified that any added dashboard complies with Kibana's Dashboard good practices

Author's Checklist

  • [ ]

How to test this PR locally

Related issues

Screenshots

@gpop63 gpop63 self-assigned this Feb 12, 2026
@gpop63 gpop63 added the Team:Obs-InfraObs Observability Infrastructure Monitoring team [elastic/obs-infraobs-integrations] label Feb 12, 2026
@gpop63
Copy link
Copy Markdown
Contributor Author

gpop63 commented Feb 13, 2026

/test

@elastic-vault-github-plugin-prod
Copy link
Copy Markdown

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@gpop63 gpop63 marked this pull request as ready for review February 13, 2026 13:25
@gpop63 gpop63 requested review from a team as code owners February 13, 2026 13:25
@botelastic
Copy link
Copy Markdown

botelastic Bot commented Mar 20, 2026

Hi! We just realized that we haven't looked into this PR in a while. We're sorry! We're labeling this issue as Stale to make it hit our filters and make sure we get back to it as soon as possible. In the meantime, it'd be extremely helpful if you could take a look at it as well and confirm its relevance. A simple comment with a nice emoji will be enough :+1. Thank you for your contribution!

@botelastic botelastic Bot added the Stalled label Mar 20, 2026
@botelastic botelastic Bot removed the Stalled label Mar 20, 2026
@elasticmachine
Copy link
Copy Markdown

💚 Build Succeeded

History

cc @gpop63

@agithomas
Copy link
Copy Markdown
Contributor

Could this be a breaking change? Please re-check.

@yavor-martian
Copy link
Copy Markdown

In which release can we expect this to land?

@yavor-martian
Copy link
Copy Markdown

@gpop63 could you please take a look at this one.

@gpop63
Copy link
Copy Markdown
Contributor Author

gpop63 commented Apr 16, 2026

Could this be a breaking change? Please re-check.

No field types are being changed here, just the way we check if its of type String or Map, the way we store values inside that field is changed.

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

Labels

Integration:aws AWS Team:Obs-InfraObs Observability Infrastructure Monitoring team [elastic/obs-infraobs-integrations]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[aws]: Issue with AWS lambda logs in JSON format

6 participants