download() không timeout/retry → cron daily có thể treo #13

New Issue

2026-07-22T11:22:32+07:00

claudecode commented

2026-07-22 11:22:32 +07:00

Vấn đề

downloader.go:19 tạo resty.New() không SetTimeout và không retry. Nếu CDN jsDelivr treo/chậm, fetchDbs() (chạy @daily + lúc start) bị kẹt vô thời hạn.

Fix đề xuất

client := resty.New().
    SetTimeout(2 * time.Minute).
    SetRetryCount(2).
    SetRetryWaitTime(5 * time.Second)

(Bổ sung import time.)

## Vấn đề `downloader.go:19` tạo `resty.New()` **không SetTimeout** và không retry. Nếu CDN jsDelivr treo/chậm, `fetchDbs()` (chạy `@daily` + lúc start) bị kẹt vô thời hạn. ## Fix đề xuất ```go client := resty.New(). SetTimeout(2 * time.Minute). SetRetryCount(2). SetRetryWaitTime(5 * time.Second) ``` (Bổ sung import `time`.)

claudecode added the bug label 2026-07-22 11:22:32 +07:00

claudecode referenced this issue from a commit

2026-07-22 13:38:01 +07:00

Fix data races, robustness, and cleanup across codebase (#10-#21)

claudecode referenced this issue

2026-07-22 13:39:25 +07:00

Fix data races, robustness & cleanup (#10-#21) #22

claudecode commented

2026-07-22 18:35:30 +07:00

Đã sửa trong PR #22 (merge commit 361eb40).

internal/services/db_updater/downloader.go: resty client thêm SetTimeout(2*time.Minute) + SetRetryCount(2) + SetRetryWaitTime(5*time.Second) — daily cron không còn treo vô hạn.

Verify: go build / go vet PASS.

Đã sửa trong PR #22 (merge commit `361eb40`). - `internal/services/db_updater/downloader.go`: resty client thêm `SetTimeout(2*time.Minute)` + `SetRetryCount(2)` + `SetRetryWaitTime(5*time.Second)` — daily cron không còn treo vô hạn. Verify: `go build` / `go vet` PASS.

claudecode closed this issue

2026-07-22 18:35:35 +07:00

This repo is archived. You cannot comment on issues.

1 Participants

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: public/ip-info#13