Handler IP trả status code sai khi query lỗi / DB chưa load #15

New Issue

2026-07-22T11:23:29+07:00

claudecode commented

2026-07-22 11:23:29 +07:00

Vấn đề

handler_json.go:35 trả HTTP 200 + body rút gọn {"ip"} khi query lỗi (gồm cả IP không có trong DB). Client không phân biệt được lỗi vs thành công.

ipData, _, err := data.Ins().Query(ip)
if err != nil {
    log.Err(err).Msg("Failed to query IP")
    c.JSON(http.StatusOK, gin.H{"ip": ip})   // ❌ 200
    return
}

Ngoài ra DB chưa load trả 500 ("Try again later") — sai ngữ nghĩa, đúng ra là 503.

Fix đề xuất

DB chưa load → 503 "Service unavailable, DB still loading"
IP không có trong DB (not found) → 404 {"error":"not found","ip":...}
Lỗi khác → 500

Áp dụng nhất quán cho cả HandleIpInfo (dùng bởi /json và /:ip).

## Vấn đề `handler_json.go:35` trả **HTTP 200** + body rút gọn `{"ip"}` khi query lỗi (gồm cả IP không có trong DB). Client không phân biệt được lỗi vs thành công. ```go ipData, _, err := data.Ins().Query(ip) if err != nil { log.Err(err).Msg("Failed to query IP") c.JSON(http.StatusOK, gin.H{"ip": ip}) // ❌ 200 return } ``` Ngoài ra DB chưa load trả **500** (`"Try again later"`) — sai ngữ nghĩa, đúng ra là 503. ## Fix đề xuất - DB chưa load → `503` "Service unavailable, DB still loading" - IP không có trong DB (not found) → `404` `{"error":"not found","ip":...}` - Lỗi khác → `500` Áp dụng nhất quán cho cả `HandleIpInfo` (dùng bởi `/json` và `/:ip`).

claudecode added the bug label 2026-07-22 11:23:29 +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:36:11 +07:00

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

internal/data/query.go: thêm sentinel ErrNotFound; trả ErrNotFound khi IP parse được nhưng không có trong DB, lỗi khác (invalid/lookup fail) tách riêng.
internal/services/api/handler_json.go: status code chuẩn — 503 khi DB đang load, 404 cho IP hợp lệ nhưng absent (/json path = self-lookup vẫn 200), 500 cho lỗi khác, 400 cho IP/format sai.

Verify: go build / go vet PASS.

Đã sửa trong PR #22 (merge commit `361eb40`). - `internal/data/query.go`: thêm sentinel `ErrNotFound`; trả `ErrNotFound` khi IP parse được nhưng không có trong DB, lỗi khác (invalid/lookup fail) tách riêng. - `internal/services/api/handler_json.go`: status code chuẩn — `503` khi DB đang load, `404` cho IP hợp lệ nhưng absent (`/json` path = self-lookup vẫn `200`), `500` cho lỗi khác, `400` cho IP/format sai. Verify: `go build` / `go vet` PASS.

claudecode closed this issue

2026-07-22 18:36:18 +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#15