Skip to content

internal/testing: fake auth server#296

Merged
jba merged 4 commits intomodelcontextprotocol:mainfrom
jba:fake-auth-server
Sep 17, 2025
Merged

internal/testing: fake auth server#296
jba merged 4 commits intomodelcontextprotocol:mainfrom
jba:fake-auth-server

Conversation

@jba
Copy link
Copy Markdown
Contributor

@jba jba commented Aug 13, 2025

This is a fake OAuth authentication server, for use in testing.

@jba jba requested a review from neild August 13, 2025 17:27
@findleyr
Copy link
Copy Markdown
Contributor

Do you want me to try to review?

Comment thread internal/testing/fake_auth_server.go Outdated
Comment thread internal/testing/fake_auth_server.go
Comment thread internal/testing/fake_auth_server.go
Comment thread internal/testing/fake_auth_server.go
Comment thread internal/testing/fake_auth_server.go
Comment thread internal/testing/fake_auth_server.go
Comment thread internal/testing/fake_auth_server.go Outdated
@jba jba force-pushed the fake-auth-server branch from 6bd3764 to acae328 Compare September 3, 2025 19:43
@jba jba requested a review from neild September 9, 2025 10:13
// type FakeAuthServer struct {
// server *http.Server
// authCodes map[string]authCodeInfo
// }
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Leftover commented-out code?

@jba jba merged commit b3fb83f into modelcontextprotocol:main Sep 17, 2025
5 checks passed
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.

3 participants