Skip to content

Conversation

@maxisbey
Copy link
Contributor

@maxisbey maxisbey commented Jan 5, 2026

The client README referenced a non-existent mcp-simple-auth script that was removed when the server was refactored into separate Authorization Server and Resource Server components.

Motivation and Context

Issue #1202 reported that the simple-auth client README references mcp-simple-auth, a script that no longer exists. The server was refactored to use separate scripts:

  • mcp-simple-auth-as (Authorization Server)
  • mcp-simple-auth-rs (Resource Server)
  • mcp-simple-auth-legacy (Legacy combined server)

How Has This Been Tested?

  • Ensured port numbers and commands are consistent between both READMEs
  • Manually tested both scenarios:
    • New Architecture: AS on port 9000, RS on port 8001, client connects successfully
    • Legacy Server: Server on port 8000, client connects with fallback discovery

Breaking Changes

None - documentation only.

Types of changes

  • 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

Fixes #1202

The client README referenced a non-existent 'mcp-simple-auth' script.
Updated both server and client READMEs to correctly reference:
- mcp-simple-auth-as (Authorization Server)
- mcp-simple-auth-rs (Resource Server)
- mcp-simple-auth-legacy (Legacy combined server)

Also aligned port numbers between examples and fixed minor formatting issues.

Github-Issue:#1202
Claude-Generated-By: Claude Code (cli/claude-opus-4-5=100%)
Claude-Steers: 3
Claude-Permission-Prompts: 10
Claude-Escapes: 1
@maxisbey maxisbey added documentation Improvements or additions to documentation auth Issues and PRs related to Authentication / OAuth labels Jan 5, 2026
@maxisbey maxisbey requested a review from pcarleton January 5, 2026 18:16
@maxisbey maxisbey added the P0 Broken core functionality, security issues, critical missing feature label Jan 5, 2026
@pcarleton pcarleton merged commit 6b69f63 into main Jan 8, 2026
21 checks passed
@pcarleton pcarleton deleted the fix/issue-1202-readme-docs branch January 8, 2026 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auth Issues and PRs related to Authentication / OAuth documentation Improvements or additions to documentation P0 Broken core functionality, security issues, critical missing feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Simple Auth Client Example refers to a non existing server project script

3 participants