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.x: [filebeat] clone value when copy fields in processors #26381

Merged
merged 1 commit into from
Jun 21, 2021

Conversation

jsoriano
Copy link
Member

@jsoriano jsoriano commented Jun 21, 2021

Cherry-pick of PR #20500 to 7.x 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
@jsoriano jsoriano self-assigned this Jun 21, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/agent (Team:Agent)

@elasticmachine
Copy link
Collaborator

💚 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: Pull request #26381 opened

  • Start Time: 2021-06-21T10:23:12.973+0000

  • Duration: 143 min 26 sec

  • Commit: 854acce

Test stats 🧪

Test Results
Failed 0
Passed 47535
Skipped 5221
Total 52756

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 47535
Skipped 5221
Total 52756

@jsoriano jsoriano merged commit 56a909e into elastic:7.x Jun 21, 2021
@jsoriano jsoriano deleted the backport_20500_7.x branch June 21, 2021 12:52
@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