Skip to content

Conversation

@Ankcorn
Copy link
Collaborator

@Ankcorn Ankcorn commented Oct 15, 2025

This PR enables tracing for the mcp servers and attempts to fix local dev. It seems to connect better for me now and work well in the inspector but it still regularly hangs on tool calls from claude desktop and I don't understand why

```
DEV_DISABLE_OAUTH=true
DEV_CLOUDFLARE_EMAIL=your_cloudflare_email
your_cloudflare_email
Copy link
Member

@Maximo-Guk Maximo-Guk Oct 15, 2025

Choose a reason for hiding this comment

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

Looks like a leftover comment

async ({ query }, req) => {
logger.setTags({ userAgent: req.requestInfo?.headers?.['mcp-protocol-version'] })
logger.setTags({ mcpSessionId: req.requestInfo?.headers?.['mcp-session-id'] })
logger.setTags({ userAgent: req.requestInfo?.headers?.['sec-ch-ua'] })
Copy link
Member

Choose a reason for hiding this comment

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

What happens if you set the same tag key, i.e userAgent multiple times?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Should be fine :)

@Ankcorn Ankcorn merged commit 30b77e0 into main Oct 15, 2025
6 checks passed
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.

2 participants