fix: strip whitespace from user info headers to prevent MCP connection failures (#26182)
This commit is contained in:
@@ -52,9 +52,9 @@ def include_user_info_headers(headers: dict, user: Optional[Any] = None) -> dict
|
||||
|
||||
return {
|
||||
**headers,
|
||||
FORWARD_USER_INFO_HEADER_USER_NAME: quote(user.name, safe=' '),
|
||||
FORWARD_USER_INFO_HEADER_USER_NAME: quote(user.name.strip(), safe=' '),
|
||||
FORWARD_USER_INFO_HEADER_USER_ID: user.id,
|
||||
FORWARD_USER_INFO_HEADER_USER_EMAIL: user.email,
|
||||
FORWARD_USER_INFO_HEADER_USER_EMAIL: user.email.strip(),
|
||||
FORWARD_USER_INFO_HEADER_USER_ROLE: user.role,
|
||||
}
|
||||
|
||||
@@ -80,8 +80,8 @@ def get_custom_headers(custom_headers: dict, user=None, metadata: dict = None) -
|
||||
'{{FILE_CONTENT_TYPE}}': metadata.get('file_content_type', '') or '',
|
||||
'{{TASK}}': metadata.get('task', '') or '',
|
||||
'{{USER_ID}}': (user.id if user else '') or '',
|
||||
'{{USER_NAME}}': (user.name if user else '') or '',
|
||||
'{{USER_EMAIL}}': (user.email if user else '') or '',
|
||||
'{{USER_NAME}}': (user.name.strip() if user else '') or '',
|
||||
'{{USER_EMAIL}}': (user.email.strip() if user else '') or '',
|
||||
'{{USER_ROLE}}': (user.role if user else '') or '',
|
||||
}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user