Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

jsoriano
Copy link
Member

@jsoriano jsoriano commented Sep 6, 2024

Same change as #11016 that I forgot to apply to this package.

Adapt definitions of subfields of nested objects so they are actually installed by Fleet. New mappings are otherwise equivalent.

These mappings were correctly defined as expected by the spec, but Fleet was only installing empty nested objects. To workaround that, subfields can be moved to have their own definitions.

Issue in Fleet is fixed in elastic/kibana#191730, but we can apply this workaround for older versions of the stack.

@jsoriano jsoriano self-assigned this Sep 6, 2024
@jsoriano jsoriano requested a review from a team as a code owner September 6, 2024 15:16
@jsoriano jsoriano added Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations] Integration:proofpoint_on_demand Proofpoint On Demand labels Sep 6, 2024
@elasticmachine
Copy link

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

@andrewkroh andrewkroh added the bugfix Pull request that fixes a bug issue label Sep 6, 2024
@elasticmachine
Copy link

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@elasticmachine
Copy link

💚 Build Succeeded

cc @jsoriano

Copy link

@jsoriano jsoriano merged commit 92f726e into elastic:main Sep 9, 2024
5 checks passed
@jsoriano jsoriano deleted the proofpoint-on-demand-nested branch September 9, 2024 12:16
@elasticmachine
Copy link

Package proofpoint_on_demand - 0.1.1 containing this change is available at https://epr.elastic.co/search?package=proofpoint_on_demand

harnish-crest-data pushed a commit to chavdaharnish/integrations that referenced this pull request Feb 4, 2025
…#11031)

Adapt definitions of subfields of nested objects so they are actually installed by
Fleet. New mappings are otherwise equivalent.

These mappings were correctly defined as expected by the spec, but Fleet was
only installing empty nested objects. To workaround that, subfields can be moved
to have their own definitions.

Issue in Fleet is fixed in elastic/kibana#191730, but we can apply this workaround
for older versions of the stack.
harnish-crest-data pushed a commit to chavdaharnish/integrations that referenced this pull request Feb 5, 2025
…#11031)

Adapt definitions of subfields of nested objects so they are actually installed by
Fleet. New mappings are otherwise equivalent.

These mappings were correctly defined as expected by the spec, but Fleet was
only installing empty nested objects. To workaround that, subfields can be moved
to have their own definitions.

Issue in Fleet is fixed in elastic/kibana#191730, but we can apply this workaround
for older versions of the stack.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix Pull request that fixes a bug issue Integration:proofpoint_on_demand Proofpoint On Demand 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.

4 participants