Review: drop duplicate 404 log, scope comment wording to 4xx (Spec #164)
This commit is contained in:
@@ -274,7 +274,8 @@ const (
|
|||||||
// second taxonomy: refused is the Site holding a challenge, unreachable the
|
// second taxonomy: refused is the Site holding a challenge, unreachable the
|
||||||
// browser interrupting, noChapter a 200 with real HTML but no chapter links,
|
// browser interrupting, noChapter a 200 with real HTML but no chapter links,
|
||||||
// unfetchable the host pin or a missing fetcher, notFound a 4xx other than
|
// unfetchable the host pin or a missing fetcher, notFound a 4xx other than
|
||||||
// the 403 refusal — the page is gone — and errors everything else.
|
// the 403 refusal — the Site answered with a client status — and errors
|
||||||
|
// everything else.
|
||||||
type readOutcome int
|
type readOutcome int
|
||||||
|
|
||||||
const (
|
const (
|
||||||
@@ -663,7 +664,6 @@ func (p *Poller) checkOne(ctx context.Context, sr store.Series) (outcome readOut
|
|||||||
return outcomeUnreachable
|
return outcomeUnreachable
|
||||||
}
|
}
|
||||||
if errors.Is(err, errNotFound) {
|
if errors.Is(err, errNotFound) {
|
||||||
log.Printf("latest poll %q: page not found: %v", sr.Key(), err)
|
|
||||||
return outcomeNotFound
|
return outcomeNotFound
|
||||||
}
|
}
|
||||||
return outcomeError
|
return outcomeError
|
||||||
|
|||||||
Reference in New Issue
Block a user