elastic / apm

Elastic Application Performance Monitoring - resources and general issue tracking for Elastic APM.
https://www.elastic.co/apm
Apache License 2.0
384 stars 114 forks source link

Add elasticsearch_capture_body_urls option #670

Closed felixbarny closed 2 years ago

felixbarny commented 2 years ago

Refs: https://github.com/elastic/apm-agent-nodejs/issues/2019 Closes: #665

We're only expecting to implement this in the Node.js agent in the near future. Other agents may add this to their roadmap as they see fit but we won't create a cross-agent tracking issue.

/schedule 2022-08-18

apmmachine commented 2 years ago

:green_heart: 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 preview

Expand to view the summary

#### Build stats * Start Time: 2022-08-16T11:14:28.625+0000 * Duration: 3 min 55 sec

trentm commented 2 years ago

Closes elastic/apm-agent-nodejs#2019 and #665

@felixbarny Are you okay with me changing that "Closes" to "Refs", so that the former issue isn't actually closed when this spec goes in? The issue should be closed when we implement the spec.

Update:

Oh, actually, I think what's wanted is:

Refs: https://github.com/elastic/apm-agent-nodejs/issues/2019
Closes: #665
felixbarny commented 2 years ago

As this affects all agents at least in the way the default behaves, I have created implementation issues for all agents and I'm waiting two more days for objections.