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

Cherry-pick #20500 to 7.13: [filebeat] clone value when copy fields in processors #26382

Merged
merged 1 commit into from
Jun 22, 2021

Conversation

jsoriano
Copy link
Member

@jsoriano jsoriano commented Jun 21, 2021

Cherry-pick of PR #20500 to 7.13 branch. Original message:

  • Bug

fix the bug mentioned in #19206

What does this PR do?

clone value when copy field in processors to avoid crash.

Why is it important?

As explained in #19206, a child-document of a map may point to it's root without deep copy.

Related issues

@jsoriano jsoriano added [zube]: In Review backport Team:Services (Deprecated) Label for the former Integrations-Services team labels Jun 21, 2021
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Jun 21, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/integrations-services (Team:Services)

@botelastic botelastic bot removed the needs_team Indicates that the issue/PR needs a Team:* label label Jun 21, 2021
@jsoriano jsoriano added the Team:Elastic-Agent Label for the Agent team label Jun 21, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/agent (Team:Agent)

@jsoriano jsoriano self-assigned this Jun 21, 2021
@elasticmachine
Copy link
Collaborator

elasticmachine commented Jun 21, 2021

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

  • Build Cause: jsoriano commented: /test

  • Start Time: 2021-06-21T12:41:46.390+0000

  • Duration: 29 min 16 sec

  • Commit: 3eba5e3

Test stats 🧪

Test Results
Failed 0
Passed 542
Skipped 5
Total 547

Trends 🧪

Image of Build Times

Image of Tests

💚 Flaky test report

Tests succeeded.

Expand to view the summary

Test stats 🧪

Test Results
Failed 0
Passed 542
Skipped 5
Total 547

@jsoriano
Copy link
Member Author

/test

@jsoriano jsoriano merged commit 51c97b9 into elastic:7.13 Jun 22, 2021
@jsoriano jsoriano deleted the backport_20500_7.13 branch June 22, 2021 07:38
@zube zube bot removed the [zube]: Done label Sep 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport Team:Elastic-Agent Label for the Agent team Team:Services (Deprecated) Label for the former Integrations-Services team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants