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

event.id -> http.request.id in formatHttpRequest #76

Closed
trentm opened this issue Mar 30, 2021 · 0 comments · Fixed by #163
Closed

event.id -> http.request.id in formatHttpRequest #76

trentm opened this issue Mar 30, 2021 · 0 comments · Fixed by #163
Assignees
Labels
agent-nodejs Make available for APM Agents project planning.

Comments

@trentm
Copy link
Member

trentm commented Mar 30, 2021

ECS 1.9.0 formalizes http.request.id: https://github.com/elastic/ecs/releases/tag/v1.9.0

Currently formatHttpRequest will map req.id to event.id. That isn't documented or in the tests. Judgement call if that mandates a breaking change to remove.

@github-actions github-actions bot added the agent-nodejs Make available for APM Agents project planning. label Mar 30, 2021
@trentm trentm self-assigned this Oct 26, 2023
trentm added a commit that referenced this issue Oct 26, 2023
trentm added a commit that referenced this issue Oct 27, 2023
@trentm trentm mentioned this issue Oct 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
agent-nodejs Make available for APM Agents project planning.
Projects
None yet
1 participant