Enable Authentication header name configuration

[idosal]

This change adds a Header Name field to the sidebar. It overrides the "Authorization" default name.

Motivation and Context

Some servers use a custom header name (e.g., Vercel Preview). The current feature doesn't support these servers.

How Has This Been Tested?

I've added unit tests and tested it manually

Breaking Changes

No breaking changes.

Types of changes

• 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

• I have read the MCP Documentation
• 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

Please let me know if it requires any changes, and thanks for the great project!

[pulkitsharma07]

You can add tests here https://github.com/modelcontextprotocol/inspector/blob/main/client/src/components/__tests__/Sidebar.test.tsx

[idosal]

@pulkitsharma07 fixed comments and added tests :)

[cliffhall]

Some servers use a custom header name (e.g., Vercel Preview). The current feature doesn't support these servers.

Hi @idosal

Can you add a video (or loom if it's too long) of using an alternative header and an easy set of steps to reproduce?

[idosal]

Some servers use a custom header name (e.g., Vercel Preview). The current feature doesn't support these servers.

Hi @idosal

Can you add a video (or loom if it's too long) of using an alternative header and an easy set of steps to reproduce?

Hey @cliffhall ,
I'm sorry to say I no longer use Vercel, so I don't have an example handy.

[cliffhall]

Some servers use a custom header name (e.g., Vercel Preview). The current feature doesn't support these servers.

Hi @idosal
Can you add a video (or loom if it's too long) of using an alternative header and an easy set of steps to reproduce?

Hey @cliffhall , I'm sorry to say I no longer use Vercel, so I don't have an example handy.

Seems like a worthwhile change if there are places out there that do this. Maybe you'd like to find a site that uses something other than "Authorization" header? I wouldn't want to merge without testability.

[idosal]

Some servers use a custom header name (e.g., Vercel Preview). The current feature doesn't support these servers.

Hi @idosal
Can you add a video (or loom if it's too long) of using an alternative header and an easy set of steps to reproduce?

Hey @cliffhall , I'm sorry to say I no longer use Vercel, so I don't have an example handy.

Seems like a worthwhile change if there are places out there that do this. Maybe you'd like to find a site that uses something other than "Authorization" header? I wouldn't want to merge without testability.

That makes complete sense. I'm afraid I couldn't find a good example in the wild. Attaching a screenshot that shows the custom header being added to the request instead of Authorization if that helps:

If you feel the feature should wait until there are live examples, that's perfectly understandable.

[idosal]

@pulkitsharma07 @cliffhall I've pushed all requested changes.

[cliffhall]

Could we amend the README?

[cliffhall]

Weird. My review page was out of sync and it looked as if the README wasn't amended. LGTM! 👍