Assign the request ctx before logging.

[Gujiawei-Edinburgh]

Motivation and Context

After the request context is assigned, the logging can add extra trace info to the log content.

How Has This Been Tested?

Yes

Breaking Changes

No

Types of changes

• [x ] Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)
• Documentation update

Checklist

• [x ] I have read the MCP Documentation
• [ x] My code follows the repository's style guidelines
• New and existing tests pass locally
• I have added appropriate error handling
• I have added or updated documentation as needed

Additional context

[mcp-shadow]

Thank you for your contribution to the MCP Python SDK. After reviewing change, I'm rejecting this PR for the following reasons:

1. Insufficient justification: While you mention that logging can add extra trace info when called after the context assignment, there's no evidence provided that this is causing issues or that trace information is currently missing from logs. The PR doesn't demonstrate what specific trace information would be improved.

2. Reduced logging visibility: Your change removes a debug-level log statement entirely, which reduces the observability of the request handling process. The current approach provides two log entries (one at info level for processing, one at debug level for dispatching), which is valuable for troubleshooting.

3. Change in error logging behavior: If an exception occurs during context setup, the current implementation ensures the request is still logged, whereas your change would result in no log entry for failed requests where the context couldn't be established.

4. Maintenance cost vs. benefit: The MCP Python SDK strives to minimize unnecessary changes to reduce maintenance burden. Without a clear demonstration of the issue being solved or substantial improvement to logging functionality, this change does not meet our threshold for inclusion.