Skip to content

Conversation

@efd6
Copy link
Contributor

@efd6 efd6 commented Oct 22, 2025

Proposed commit message

aws_bedrock: tolerate non-object elements in invocation output.outputBodyJson lists

Some models will include string elements in this list in chat streaming
responses, so promote non-object elements to an object with a single
"value" field holding the original element value. Without this change
those documents are otherwise rejected during ingest since flattened
fields may not contain non-object elements.

The test case is a copy of an existing test case, with the final element
in the list replaced with the string "[DONE]" to reflect the original
report.

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

@efd6 efd6 self-assigned this Oct 22, 2025
@efd6 efd6 added enhancement New feature or request Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations] Integration:aws_bedrock Amazon Bedrock labels Oct 22, 2025
…BodyJson lists

Some models will include string elements in this list in chat streaming
responses, so promote non-object elements to an object with a single
"value" field holding the original element value. Without this change
those documents are otherwise rejected during ingest since flattened
fields may not contain non-object elements.

The test case is a copy of an existing test case, with the final element
in the list replaced with the string "[DONE]" to reflect the original
report.
@efd6 efd6 force-pushed the s6547-aws_bedrock branch from 1bbb805 to 1c72909 Compare October 22, 2025 20:09
@elastic-vault-github-plugin-prod

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@elasticmachine
Copy link

💚 Build Succeeded

cc @efd6

@efd6 efd6 marked this pull request as ready for review October 22, 2025 21:11
@efd6 efd6 requested review from a team as code owners October 22, 2025 21:11
@elasticmachine
Copy link

Pinging @elastic/security-service-integrations (Team:Security-Service Integrations)

Copy link
Contributor

@agithomas agithomas left a comment

Choose a reason for hiding this comment

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

LGTM!

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

Labels

enhancement New feature or request Integration:aws_bedrock Amazon Bedrock Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants