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

fix: Add 'http.response.status_code' to documentFetch span. #928

Merged
merged 4 commits into from
Jan 20, 2025
Merged

Conversation

amertak
Copy link
Contributor

@amertak amertak commented Jan 19, 2025

Description

Add 'http.response.status_code' to documentFetch span.

Type of change

Delete options that are not relevant.

  • Fix

How has this been tested?

Delete options that are not relevant.

  • Manual testing
  • Added integration tests

@amertak amertak requested review from a team as code owners January 19, 2025 20:13
Copy link

github-actions bot commented Jan 19, 2025

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@amertak
Copy link
Contributor Author

amertak commented Jan 19, 2025

I have read the CLA Document and I hereby sign the CLA

srv-gh-o11y-gdi-cla added a commit to splunk/cla-agreement that referenced this pull request Jan 19, 2025
@@ -8,7 +8,8 @@
"pretty": true,
"resolveJsonModule": true,
"target": "ES2017",
"types": ["node"]
"types": ["node"],
"skipLibCheck": true
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we should remove this once we upgrade web-vitals package

@Joozty Joozty merged commit 74ed1a2 into main Jan 20, 2025
10 checks passed
@Joozty Joozty deleted the fix/status branch January 20, 2025 08:54
@github-actions github-actions bot locked and limited conversation to collaborators Jan 20, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants