melosh/mal
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Handle HTML response errors

Browse Source
This commit is contained in:
mkelvers
2026-06-20 18:28:34 +02:00
committed by Milas Holsting
parent 9e8e49691c
commit ce41785ffa
1 changed files with 8 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
pkg/net/document.go
View File

@@ -4,6 +4,7 @@ import (
"context" "context"
"fmt" "fmt"
"io" "io"
"mal/pkg/errlog"
"net/http" "net/http"
"github.com/PuerkitoBio/goquery" "github.com/PuerkitoBio/goquery"
@@ -43,10 +44,15 @@ func FetchHTMLDocument(
if err != nil { if err != nil {
return nil, "", fmt.Errorf("request failed: %w", err) return nil, "", fmt.Errorf("request failed: %w", err)
} }
defer func() { _ = response.Body.Close() }() defer func() {
errlog.Log("failed to close html response body", response.Body.Close())
}()
if response.StatusCode != http.StatusOK { if response.StatusCode != http.StatusOK {
body, _ := io.ReadAll(io.LimitReader(response.Body, Bytes512)) body, readErr := io.ReadAll(io.LimitReader(response.Body, Bytes512))
if readErr != nil {
return nil, responseURL(response, request), fmt.Errorf("failed to read error response body: %w", readErr)
}
return nil, responseURL(response, request), buildStatusError(response, body) return nil, responseURL(response, request), buildStatusError(response, body)
} }