fix: mcp header auth type selection (#6246)

packages/service/common/secret/utils.ts

@@ -39,6 +39,11 @@ export const getSecretValue = ({
     const { secret, value } = val;
     const { secret, value } = val;
     const actualValue = value || decryptSecret(secret);
     const actualValue = value || decryptSecret(secret);
 
 
+    // Filter out empty values to avoid invalid headers
+    if (!actualValue || !key) {
+      return acc;
+    }
+
     if (key === HeaderSecretTypeEnum.Bearer) {
     if (key === HeaderSecretTypeEnum.Bearer) {
       acc['Authorization'] = `Bearer ${actualValue}`;
       acc['Authorization'] = `Bearer ${actualValue}`;
     } else if (key === HeaderSecretTypeEnum.Basic) {
     } else if (key === HeaderSecretTypeEnum.Basic) {

projects/app/src/components/common/secret/HeaderAuthForm.tsx

@@ -87,9 +87,9 @@ const AuthValueDisplay = ({
 };
 };
 
 
 export const getSecretType = (config: HeaderSecretConfigType): HeaderSecretTypeEnum => {
 export const getSecretType = (config: HeaderSecretConfigType): HeaderSecretTypeEnum => {
-  if (config.Bearer && (config.Bearer.secret || config.Bearer.value)) {
+  if (config.Bearer) {
     return HeaderSecretTypeEnum.Bearer;
     return HeaderSecretTypeEnum.Bearer;
-  } else if (config.Basic && (config.Basic.secret || config.Basic.value)) {
+  } else if (config.Basic) {
     return HeaderSecretTypeEnum.Basic;
     return HeaderSecretTypeEnum.Basic;
   } else if (config.customs && config.customs.length > 0) {
   } else if (config.customs && config.customs.length > 0) {
     return HeaderSecretTypeEnum.Custom;
     return HeaderSecretTypeEnum.Custom;