skip operations with a request body with no supported MIME type

g-linville · g-linville · commit 1a1523977834 · 2024-04-02T09:24:20.000-04:00
Signed-off-by: Grant Linville &lt;grant@acorn.io&gt;
diff --git a/pkg/loader/openapi.go b/pkg/loader/openapi.go
@@ -66,6 +66,7 @@ func getOpenAPITools(t *openapi3.T, defaultHost string) ([]types.Tool, error) {
 			}
 		}
 
+	operations:
 		for method, operation := range pathObj.Operations() {
 			// Handle operation-level server override, if one exists
 			operationServer := pathServer
@@ -170,6 +171,11 @@ func getOpenAPITools(t *openapi3.T, defaultHost string) ([]types.Tool, error) {
 					tool.Parameters.Arguments.Properties["requestBodyContent"] = &openapi3.SchemaRef{Value: arg}
 					break
 				}
+
+				if bodyMIME == "" {
+					// No supported MIME types found, so just skip this operation and move on.
+					continue operations
+				}
 			}
 
 			// See if there is any auth defined for this operation

Original file line number	Diff line number	Diff line change
`@@ -66,6 +66,7 @@ func getOpenAPITools(t *openapi3.T, defaultHost string) ([]types.Tool, error) {`
`66`	`66`	`}`
`67`	`67`	`}`
`68`	`68`
	`69`	`+ operations:`
`69`	`70`	`for method, operation := range pathObj.Operations() {`
`70`	`71`	`// Handle operation-level server override, if one exists`
`71`	`72`	`operationServer := pathServer`
`@@ -170,6 +171,11 @@ func getOpenAPITools(t *openapi3.T, defaultHost string) ([]types.Tool, error) {`
`170`	`171`	`tool.Parameters.Arguments.Properties["requestBodyContent"] = &openapi3.SchemaRef{Value: arg}`
`171`	`172`	`break`
`172`	`173`	`}`
	`174`	`+`
	`175`	`+ if bodyMIME == "" {`
	`176`	`+ // No supported MIME types found, so just skip this operation and move on.`
	`177`	`+ continue operations`
	`178`	`+ }`
`173`	`179`	`}`
`174`	`180`
`175`	`181`	`// See if there is any auth defined for this operation`