Merge branch 'main' into web-client-oauth-proxy-fetch
This commit is contained in:
commit
c04cd9958d
2 changed files with 14 additions and 4 deletions
|
|
@ -55,7 +55,12 @@ const JsonView = memo(
|
||||||
}, [toast, normalizedData, setCopied]);
|
}, [toast, normalizedData, setCopied]);
|
||||||
|
|
||||||
return (
|
return (
|
||||||
<div className={clsx("p-4 border rounded relative", className)}>
|
<div
|
||||||
|
className={clsx(
|
||||||
|
"p-4 border overflow-x-auto rounded relative",
|
||||||
|
className,
|
||||||
|
)}
|
||||||
|
>
|
||||||
{withCopyButton && (
|
{withCopyButton && (
|
||||||
<Button
|
<Button
|
||||||
size="icon"
|
size="icon"
|
||||||
|
|
|
||||||
|
|
@ -148,10 +148,15 @@ export class InspectorOAuthClientProvider implements OAuthClientProvider {
|
||||||
}
|
}
|
||||||
|
|
||||||
get redirect_uris() {
|
get redirect_uris() {
|
||||||
// Normally register both redirect URIs to support both normal and debug flows
|
// Always register both redirect URIs to support both normal and debug flows.
|
||||||
// In debug subclass, redirectUrl may be the same as debugRedirectUrl, so remove duplicates
|
// Use the base URLs directly (not this.redirectUrl) to avoid the debug subclass
|
||||||
|
// override collapsing both URIs into one, which causes the normal flow's
|
||||||
|
// /authorize request to use an unregistered redirect_uri.
|
||||||
// See: https://github.com/modelcontextprotocol/inspector/issues/825
|
// See: https://github.com/modelcontextprotocol/inspector/issues/825
|
||||||
return [...new Set([this.redirectUrl, this.debugRedirectUrl])];
|
// See: https://github.com/modelcontextprotocol/inspector/issues/930
|
||||||
|
const callbackUrl = window.location.origin + "/oauth/callback";
|
||||||
|
const debugCallbackUrl = window.location.origin + "/oauth/callback/debug";
|
||||||
|
return [callbackUrl, debugCallbackUrl];
|
||||||
}
|
}
|
||||||
|
|
||||||
get clientMetadata(): OAuthClientMetadata {
|
get clientMetadata(): OAuthClientMetadata {
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue