Merge pull request #999 from thinhtpt-dev/main
fix: detect 401 errors from `StreamableHTTP` transport in client
This commit is contained in:
commit
fc8066efe6
1 changed files with 5 additions and 1 deletions
|
|
@ -7,6 +7,7 @@ import {
|
||||||
import {
|
import {
|
||||||
StreamableHTTPClientTransport,
|
StreamableHTTPClientTransport,
|
||||||
StreamableHTTPClientTransportOptions,
|
StreamableHTTPClientTransportOptions,
|
||||||
|
StreamableHTTPError,
|
||||||
} from "@modelcontextprotocol/sdk/client/streamableHttp.js";
|
} from "@modelcontextprotocol/sdk/client/streamableHttp.js";
|
||||||
import {
|
import {
|
||||||
ClientNotification,
|
ClientNotification,
|
||||||
|
|
@ -358,8 +359,11 @@ export function useConnection({
|
||||||
const is401Error = (error: unknown): boolean => {
|
const is401Error = (error: unknown): boolean => {
|
||||||
return (
|
return (
|
||||||
(error instanceof SseError && error.code === 401) ||
|
(error instanceof SseError && error.code === 401) ||
|
||||||
|
(error instanceof StreamableHTTPError && error.code === 401) ||
|
||||||
(error instanceof Error && error.message.includes("401")) ||
|
(error instanceof Error && error.message.includes("401")) ||
|
||||||
(error instanceof Error && error.message.includes("Unauthorized"))
|
(error instanceof Error && error.message.includes("Unauthorized")) ||
|
||||||
|
(error instanceof Error &&
|
||||||
|
error.message.includes("Missing Authorization header"))
|
||||||
);
|
);
|
||||||
};
|
};
|
||||||
|
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue