Skip to content

Support adding arbitrary headers #249

Closed

Description

@yuedong111

Why not replace the authorization field with headers? This makes the test more flexible.

Activity

  1. pulkitsharma07 commented on Apr 2, 2025

    @pulkitsharma07
    Contributor

    Can you explain our use case ?

  2. ericlabouve-sn commented on Apr 8, 2025

    @ericlabouve-sn

    @pulkitsharma07 I agree with @yuedong111 . Instead of a Bearer Token field, a user should be allowed more flexibility in which headers gets included in the http request.

    For example, what if the MCP server is not using a Bearer Token for authentication but is rather using an API Key which requires the Ocp-Apim-Subscription-Key header?

    A generic text field (or something similar to Postman) would allow for this flexibility. This design would still allow for Bearer tokens by specifying Authorization: Bearer <token>

    Here is where the Bearer Header is currently being set. We should make this generic so the user can set any arbitrary header.

  3. anthonychu commented on Apr 10, 2025

    @anthonychu

    +1 on this. It could just be an optional section for additional HTTP headers. The existing bearer token input is easier to use and I think it would be nice if it stayed.

  4. charlesr1971 commented on Apr 14, 2025

    @charlesr1971

    +1 on this. I need to pass three custom headers, that are vital for my application working

  5. iFurySt commented on Apr 17, 2025

    @iFurySt
    Image

    what about this? we're converting a lot of restful api to mcp servers via an mcp-gateway service. it'll be helpful if inspector could support custom headers instead of forcing jwt auth. if this sounds broadly useful, i'm happy to submit a pr.

  6. charlesr1971 commented on Apr 17, 2025

    @charlesr1971

    @iFurySt Yes. This sounds great!😊

  7. cristipufu commented on Apr 22, 2025

    @cristipufu

    @iFurySt looking forward to your PR 😄

  8. Blackbaud-JamesZhang commented on May 13, 2025

    @Blackbaud-JamesZhang

    @iFurySt looking forward to this addition, any update on this? 😁

  9. anandwilbert commented on May 15, 2025

    @anandwilbert

    @iFurySt +1 on this. I will be helpful to have this feature

  10. changed the title [-]Inspector should add a header support[/-] [+]Support adding arbitrary headers[/+] on May 28, 2025
  11. ashubham commented on Jun 2, 2025

    @ashubham

    +1, I need to send more headers in addition to auth like x-tenant-name etc

  12. SteffanDefinely commented on Jun 6, 2025

    @SteffanDefinely

    +1

  13. dahaipeng commented on Jun 13, 2025

    @dahaipeng

    +++++1 this would be MORE flexible for developers who wanna suport customized headers to connect to mcp servers; looking forward to your updates!!!

  14. bozhlab commented on Jun 17, 2025

    @bozhlab

    Need for a custom header to pass some user information when initialize the connection.

  15. 14 remaining items

  16. AyatKhraisat commented on Aug 30, 2025

    @AyatKhraisat

    +1

  17. josephkalayci commented on Sep 1, 2025

    @josephkalayci

    +1

  18. Gabswim commented on Sep 3, 2025

    @Gabswim

    +1

  19. Gabswim commented on Sep 3, 2025

    @Gabswim

    Note: While the PR is pending #549, you can test the changes by running:

    npx github:popomore/inspector

    ⚠️
    This runs code from an unmerged PR. Please review the changes before running to ensure they're safe.

  20. palash1611 commented on Sep 4, 2025

    @palash1611

    @Gabswim I believe these headers are only sent during the connection request. There are MCPs which expects MCP during the tool call as well. Is there a possibility of sending these headers along with each tool call as well?

  21. Gabswim commented on Sep 4, 2025

    @Gabswim

    There are MCPs which expects MCP during the tool call as well. Is there a possibility of sending these headers along with each tool call as well?

    @palash1611 Yes, this is what it does from what I tested, because we have that exact use case where we are sending the organization id as a header

  22. daltonwallace commented on Sep 8, 2025

    @daltonwallace

    +1

  23. michaelshewbread commented on Sep 8, 2025

    @michaelshewbread

    +1

  24. rishid commented on Sep 8, 2025

    @rishid

    There is a PR here: #751

  25. ryanmadsenn commented on Sep 9, 2025

    @ryanmadsenn

    +1

  26. craigmwright commented on Sep 9, 2025

    @craigmwright

    +1

  27. ZacharyBaker commented on Sep 9, 2025

    @ZacharyBaker

    +1

  28. Devinbeth commented on Sep 9, 2025

    @Devinbeth

    +1

  29. dvalsagna89 commented on Sep 9, 2025

    @dvalsagna89

    +1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions