github/Fabric
1
1
Fork 0
mirror of https://github.com/danielmiessler/Fabric.git synced 2026-04-24 03:00:15 -04:00
Code Issues Packages Projects Releases Wiki Activity

fix: improve error message truncation in DirectlyGetModels method

Browse Source 
## CHANGES

- Add proper bounds checking for response body truncation
- Prevent slice out of bounds errors in error messages
- Add ellipsis indicator when response body is truncated
- Improve error message clarity for debugging purposes
This commit is contained in:
Kayvan Sylvan
2025-07-11 13:33:16 -07:00
parent ecac2b4c34
commit e1945a0b62
1 changed files with 7 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
internal/plugins/ai/openai_compatible/direct_models_call.go
View File

@@ -88,7 +88,13 @@ func (c *Client) DirectlyGetModels(ctx context.Context) ([]string, error) {
return extractModelIDs(directArray), nil
}
return nil, fmt.Errorf("unable to parse models response; raw response: %s", string(bodyBytes)[:errorResponseLimit])
var truncatedBody string
if len(bodyBytes) > errorResponseLimit {
truncatedBody = string(bodyBytes[:errorResponseLimit]) + "..."
} else {
truncatedBody = string(bodyBytes)
}
return nil, fmt.Errorf("unable to parse models response; raw response: %s", truncatedBody)
}
func extractModelIDs(models []Model) []string {