mirror of
https://github.com/samanhappy/mcphub.git
synced 2025-12-23 18:29:21 -05:00
Fix tool and prompt description update not calling API
The issue was that toolService.ts and promptService.ts were setting Authorization: Bearer header manually, but the backend expects the x-auth-token header. The auth interceptor already handles adding x-auth-token automatically, so the fix is to remove the manual header and let the interceptor do its job. Co-authored-by: samanhappy <2755122+samanhappy@users.noreply.github.com>
This commit is contained in:
@@ -126,14 +126,10 @@ export const updatePromptDescription = async (
|
|||||||
): Promise<{ success: boolean; error?: string }> => {
|
): Promise<{ success: boolean; error?: string }> => {
|
||||||
try {
|
try {
|
||||||
// URL-encode server and prompt names to handle slashes (e.g., "com.atlassian/atlassian-mcp-server")
|
// URL-encode server and prompt names to handle slashes (e.g., "com.atlassian/atlassian-mcp-server")
|
||||||
|
// Auth header is automatically added by the interceptor
|
||||||
const response = await apiPut<any>(
|
const response = await apiPut<any>(
|
||||||
`/servers/${encodeURIComponent(serverName)}/prompts/${encodeURIComponent(promptName)}/description`,
|
`/servers/${encodeURIComponent(serverName)}/prompts/${encodeURIComponent(promptName)}/description`,
|
||||||
{ description },
|
{ description },
|
||||||
{
|
|
||||||
headers: {
|
|
||||||
Authorization: `Bearer ${localStorage.getItem('mcphub_token')}`,
|
|
||||||
},
|
|
||||||
},
|
|
||||||
);
|
);
|
||||||
|
|
||||||
return {
|
return {
|
||||||
|
|||||||
@@ -30,11 +30,8 @@ export const callTool = async (
|
|||||||
? `/tools/${encodeURIComponent(server)}/${encodeURIComponent(request.toolName)}`
|
? `/tools/${encodeURIComponent(server)}/${encodeURIComponent(request.toolName)}`
|
||||||
: '/tools/call';
|
: '/tools/call';
|
||||||
|
|
||||||
const response = await apiPost<any>(url, request.arguments, {
|
// Auth header is automatically added by the interceptor
|
||||||
headers: {
|
const response = await apiPost<any>(url, request.arguments);
|
||||||
Authorization: `Bearer ${localStorage.getItem('mcphub_token')}`, // Add bearer auth for MCP routing
|
|
||||||
},
|
|
||||||
});
|
|
||||||
|
|
||||||
if (response.success === false) {
|
if (response.success === false) {
|
||||||
return {
|
return {
|
||||||
@@ -66,14 +63,10 @@ export const toggleTool = async (
|
|||||||
): Promise<{ success: boolean; error?: string }> => {
|
): Promise<{ success: boolean; error?: string }> => {
|
||||||
try {
|
try {
|
||||||
// URL-encode server and tool names to handle slashes (e.g., "com.atlassian/atlassian-mcp-server")
|
// URL-encode server and tool names to handle slashes (e.g., "com.atlassian/atlassian-mcp-server")
|
||||||
|
// Auth header is automatically added by the interceptor
|
||||||
const response = await apiPost<any>(
|
const response = await apiPost<any>(
|
||||||
`/servers/${encodeURIComponent(serverName)}/tools/${encodeURIComponent(toolName)}/toggle`,
|
`/servers/${encodeURIComponent(serverName)}/tools/${encodeURIComponent(toolName)}/toggle`,
|
||||||
{ enabled },
|
{ enabled },
|
||||||
{
|
|
||||||
headers: {
|
|
||||||
Authorization: `Bearer ${localStorage.getItem('mcphub_token')}`,
|
|
||||||
},
|
|
||||||
},
|
|
||||||
);
|
);
|
||||||
|
|
||||||
return {
|
return {
|
||||||
@@ -99,14 +92,10 @@ export const updateToolDescription = async (
|
|||||||
): Promise<{ success: boolean; error?: string }> => {
|
): Promise<{ success: boolean; error?: string }> => {
|
||||||
try {
|
try {
|
||||||
// URL-encode server and tool names to handle slashes (e.g., "com.atlassian/atlassian-mcp-server")
|
// URL-encode server and tool names to handle slashes (e.g., "com.atlassian/atlassian-mcp-server")
|
||||||
|
// Auth header is automatically added by the interceptor
|
||||||
const response = await apiPut<any>(
|
const response = await apiPut<any>(
|
||||||
`/servers/${encodeURIComponent(serverName)}/tools/${encodeURIComponent(toolName)}/description`,
|
`/servers/${encodeURIComponent(serverName)}/tools/${encodeURIComponent(toolName)}/description`,
|
||||||
{ description },
|
{ description },
|
||||||
{
|
|
||||||
headers: {
|
|
||||||
Authorization: `Bearer ${localStorage.getItem('mcphub_token')}`,
|
|
||||||
},
|
|
||||||
},
|
|
||||||
);
|
);
|
||||||
|
|
||||||
return {
|
return {
|
||||||
|
|||||||
@@ -12,7 +12,7 @@
|
|||||||
"tools": {
|
"tools": {
|
||||||
"amap-maps_regeocode": {
|
"amap-maps_regeocode": {
|
||||||
"enabled": true,
|
"enabled": true,
|
||||||
"description": "My custom description"
|
"description": "Updated via UI test"
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
},
|
},
|
||||||
|
|||||||
Reference in New Issue
Block a user