Skip to content

Conversation

@s1gr1d
Copy link
Member

@s1gr1d s1gr1d commented May 6, 2024

No description provided.

"@astrojs/check": "^0.5.10",
"@astrojs/node": "^8.2.5",
"@sentry/astro": "7.113.0",
"@sentry/astro": "8.0.0-beta.6",
Copy link
Member Author

Choose a reason for hiding this comment

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

That line was the only change in Astro 🙌🏻

Comment on lines -99 to -101
"hardwareConcurrency": "10",
"lcp.element": "body > h1",
"lcp.size": "[[highNumber]]",
Copy link
Member Author

Choose a reason for hiding this comment

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

LCP is missing

"cookies": {},
"headers": {},
"method": "GET",
"url": "http://localhost:4321/test-param-error/1337"
Copy link
Member Author

Choose a reason for hiding this comment

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

not parameterized

Comment on lines +151 to +152
"http.target": "/api/test-param-error/1337",
"http.url": "http://localhost:4321/api/test-param-error/1337",
Copy link
Member Author

Choose a reason for hiding this comment

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

not parameterized

Copy link
Member

Choose a reason for hiding this comment

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

this is fine, this is "only" outgoing fetch stuff!

"description": "GET http://localhost:4321/api/test-param-error/1337",
"op": "http.client",
"origin": "auto.http.node.undici",
"origin": "auto.http.otel.node_fetch",
Copy link
Member Author

Choose a reason for hiding this comment

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

Origin is different

Copy link
Member

Choose a reason for hiding this comment

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

this is fine too, to be expected :)

"transaction": "GET /test-param-error/[param]",
"transaction_info": {
"source": "custom"
"source": "route"
Copy link
Member Author

Choose a reason for hiding this comment

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

different source

Copy link
Member

Choose a reason for hiding this comment

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

seems more correct than before 🤔 so all good :D

@s1gr1d s1gr1d merged commit 96e6364 into main Jun 17, 2024
@s1gr1d s1gr1d deleted the sig/astro-v8 branch June 17, 2024 12:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants