Skip to content

Conversation

@marvion2
Copy link
Contributor

  • Fix MCP test connection authentication issues by updating frontend request format
  • Add variables field with authorization_token for template substitution
  • Change headers to use proper Authorization Bearer format with template variable

🤖 Generated with Claude Code

What problem does this PR solve?

correct MCP server authentication header format in frontend

Type of change

  • Bug Fix (non-breaking change which fixes an issue)

- Fix MCP test connection authentication issues by updating frontend request format
- Add variables field with authorization_token for template substitution
- Change headers to use proper Authorization Bearer format with template variable
- Add CLAUDE.md with essential development guidance for future Claude Code instances

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. 🐞 bug Something isn't working, pull request that fix bug. labels Aug 29, 2025
@KevinHuSh KevinHuSh requested a review from buua436 October 22, 2025 04:26
@marvion2 marvion2 requested a review from buua436 October 23, 2025 05:17
@marvion2 marvion2 closed this Oct 24, 2025
@marvion2 marvion2 reopened this Oct 24, 2025
@marvion2
Copy link
Contributor Author

@buua436 request to merge

@buua436 buua436 added the ci Continue Integration label Oct 31, 2025
@buua436 buua436 requested a review from yongtenglei October 31, 2025 07:56
@KevinHuSh KevinHuSh merged commit 395ce16 into infiniflow:main Nov 3, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐞 bug Something isn't working, pull request that fix bug. ci Continue Integration size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants