Skip to content

Allow the MCP server to start without authorization for inspection#62

Merged
derek-etherton-opslevel merged 1 commit intomainfrom
13611-start-with-invalid-token
Jul 2, 2025
Merged

Allow the MCP server to start without authorization for inspection#62
derek-etherton-opslevel merged 1 commit intomainfrom
13611-start-with-invalid-token

Conversation

@derek-etherton-opslevel
Copy link
Copy Markdown
Contributor

@derek-etherton-opslevel derek-etherton-opslevel commented Jul 2, 2025

Problem

Third parties are unable to inspect the MCP server and its available tools without having a valid OpsLevel API token.

To better support aggregators like Glama.ai, security scanners, contributors, etc. who might use MCP Inspector or other tools without a valid OpsLevel API key, we should allow this.

Screenshot 2025-07-02 at 9 31 12 AM

Solution

Allow the server to start without authorizing, but still return 401 Unauthorized for any tool requests.

Screenshot 2025-07-02 at 9 30 51 AM

With invalid token:

Screenshot 2025-07-02 at 9 32 07 AM Screenshot 2025-07-02 at 9 32 59 AM

With valid token:

Screenshot 2025-07-02 at 9 33 41 AM

Checklist

  • I have run this code, and it appears to resolve the stated issue.
  • This PR has no user interface changes or has already received approval from product management to change the interface.
  • Make a changie entry that explains the customer facing outcome of this change.

@derek-etherton-opslevel derek-etherton-opslevel force-pushed the 13611-start-with-invalid-token branch from c11fd3f to e9c2132 Compare July 2, 2025 16:37
@derek-etherton-opslevel derek-etherton-opslevel force-pushed the 13611-start-with-invalid-token branch from e9c2132 to e40f31d Compare July 2, 2025 16:42
Comment thread src/cmd/client.go
@derek-etherton-opslevel derek-etherton-opslevel requested review from BigBlue79 and Farjaad and removed request for BigBlue79 July 2, 2025 16:47
@derek-etherton-opslevel derek-etherton-opslevel force-pushed the 13611-start-with-invalid-token branch from e40f31d to 38de88c Compare July 2, 2025 18:59
@derek-etherton-opslevel derek-etherton-opslevel merged commit a26bded into main Jul 2, 2025
2 checks passed
@derek-etherton-opslevel derek-etherton-opslevel deleted the 13611-start-with-invalid-token branch July 2, 2025 19:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants