Skip to content

Commit 05bca41

Browse files
panics for resourceTemplates as well
1 parent ea2df9c commit 05bca41

File tree

1 file changed

+7
-4
lines changed

1 file changed

+7
-4
lines changed

mcp/server.go

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -371,10 +371,13 @@ func (s *Server) AddResourceTemplate(t *ResourceTemplate, h ResourceHandler) {
371371
// Ensure the URI template has a valid scheme
372372
u, err := url.Parse(t.URITemplate)
373373
if err != nil {
374-
panic(err) // url.Parse includes the URI in the error
375-
}
376-
if !u.IsAbs() {
377-
panic(fmt.Errorf("URI template %q needs a scheme", t.URITemplate))
374+
//panic(err) // url.Parse includes the URI in the error
375+
fmt.Fprintf(os.Stderr, "invalid resource template uri %q: %v\n", t.URITemplate, err)
376+
} else {
377+
if !u.IsAbs() {
378+
//panic(fmt.Errorf("URI template %q needs a scheme", t.URITemplate))
379+
fmt.Fprintf(os.Stderr, "invalid resource template uri %q: %v\n", t.URITemplate, err)
380+
}
378381
}
379382
s.resourceTemplates.add(&serverResourceTemplate{t, h})
380383
return true

0 commit comments

Comments
 (0)