zlei6/openserp
1
0
Fork 0
mirror of https://github.com/karust/openserp.git synced 2026-08-15 21:15:56 +08:00
Code Issues Packages Projects Releases Wiki Activity

refactor: use errInvalidParam helper and add per-URL took_ms to batch extract

Browse Source
This commit is contained in:
AIWintermuteAI
2026-07-18 09:56:20 +02:00
parent aec0d9c42a
commit e430584324
1 changed files with 5 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
core/server_extract.go
View File

@@ -446,10 +446,10 @@ func (s *Server) handleBatchExtract(c *fiber.Ctx) error {
var body batchExtractPayload
if len(c.Body()) == 0 {
return &APIError{HTTPStatus: fiber.StatusBadRequest, ErrorCode: "invalid_request", Message: "request body is required"}
return errInvalidParam("request body is required")
}
if err := c.BodyParser(&body); err != nil {
return &APIError{HTTPStatus: fiber.StatusBadRequest, ErrorCode: "invalid_request", Message: "invalid JSON body"}
return errInvalidParam("invalid JSON body")
}
// Deduplicate and normalize URLs.
@@ -467,14 +467,10 @@ func (s *Server) handleBatchExtract(c *fiber.Ctx) error {
urls = append(urls, u)
}
if len(urls) == 0 {
return &APIError{HTTPStatus: fiber.StatusBadRequest, ErrorCode: "invalid_request", Message: "urls array is required and must contain at least one valid URL"}
return errInvalidParam("urls array is required and must contain at least one valid URL")
}
if len(urls) > maxBatchExtractURLs {
return &APIError{
HTTPStatus: fiber.StatusBadRequest,
ErrorCode: "invalid_request",
Message: fmt.Sprintf("urls array exceeds maximum of %d", maxBatchExtractURLs),
}
return errInvalidParam(fmt.Sprintf("urls array exceeds maximum of %d", maxBatchExtractURLs))
}
// Validate all URLs upfront.
@@ -539,6 +535,7 @@ func (s *Server) handleBatchExtract(c *fiber.Ctx) error {
"canonical": result.Canonical,
"mode_used": result.Meta.ModeUsed,
"fetched_at": result.Meta.FetchedAt,
"took_ms": fmt.Sprintf("%d", result.Meta.TookMs),
},
}
}(i, u)