elastic / beats

:tropical_fish: Beats - Lightweight shippers for Elasticsearch & Logstash
https://www.elastic.co/products/beats
Other
103 stars 4.92k forks source link

[8.x](backport #40909) [filebeat] Fix long filepaths in diagnostics exceeding max path limits on Windows #40923

Closed mergify[bot] closed 1 month ago

mergify[bot] commented 1 month ago

Proposed commit message

Fix long filepaths in diagnostics exceeding max path limits on Windows.

Following the first proposal here https://github.com/elastic/elastic-agent/issues/3758#issuecomment-1830229924 Added IDWithoutName that doesn't include the source name and using that instead.

Checklist

Related issues

Screenshots

The screenshot that shows the diagnostics collected before and after this change. The earlier diagnostics shows the long path that includes the sanitized okta log collection URL. The diagnostics after this change have shorter path that doesn't include the sanitized URL in the path.

Screenshot 2024-09-19 at 2 36 12 PM


This is an automatic backport of pull request #40909 done by Mergify.

elasticmachine commented 1 month ago

Pinging @elastic/sec-deployment-and-devices (Team:Security-Deployment and Devices)

mergify[bot] commented 1 month ago

This pull request has not been merged yet. Could you please review and merge it @aleksmaus? 🙏

elasticmachine commented 1 month ago

Pinging @elastic/elastic-agent-data-plane (Team:Elastic-Agent-Data-Plane)