Data race trên DbUpdatedAt (cron ghi, /metrics đọc) #12

New Issue

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

claudecode commented

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

Vấn đề

DbUpdatedAt time.Time (db_updater/main.go:11) được ghi bởi goroutine cron (fetchDbs) và đọc bởi HTTP handler /metrics (handler_metrics.go:11) mà không có đồng bộ → data race trên struct time.Time.

Fix đề xuất

Dùng atomic.Int64 lưu unix milli:

var DbUpdatedAt atomic.Int64
// ghi: DbUpdatedAt.Store(time.Now().UnixMilli())
// đọc: time.UnixMilli(DbUpdatedAt.Load())

Hoặc dùng mutex. Ưu tiên atomic cho đơn giản.

## Vấn đề `DbUpdatedAt time.Time` (`db_updater/main.go:11`) được **ghi bởi goroutine cron** (`fetchDbs`) và **đọc bởi HTTP handler** `/metrics` (`handler_metrics.go:11`) mà **không có đồng bộ** → data race trên struct `time.Time`. ## Fix đề xuất Dùng `atomic.Int64` lưu unix milli: ```go var DbUpdatedAt atomic.Int64 // ghi: DbUpdatedAt.Store(time.Now().UnixMilli()) // đọc: time.UnixMilli(DbUpdatedAt.Load()) ``` Hoặc dùng mutex. Ưu tiên `atomic` cho đơn giản.

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

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

internal/services/db_updater/main.go: DbUpdatedAt time.Time → atomic.Int64 (cron goroutine ghi, handler /metrics đọc); thêm DbUpdatedAtMillis() accessor + stampDbUpdatedAt().
internal/services/api/handler_metrics.go: gọi db_updater.DbUpdatedAtMillis() thay vì truy cập field trực tiếp.

Verify: go test -race PASS (race regression test trong internal/data/data_test.go).

Đã sửa trong PR #22 (merge commit `361eb40`). - `internal/services/db_updater/main.go`: `DbUpdatedAt time.Time` → `atomic.Int64` (cron goroutine ghi, handler `/metrics` đọc); thêm `DbUpdatedAtMillis()` accessor + `stampDbUpdatedAt()`. - `internal/services/api/handler_metrics.go`: gọi `db_updater.DbUpdatedAtMillis()` thay vì truy cập field trực tiếp. Verify: `go test -race` PASS (race regression test trong `internal/data/data_test.go`).

claudecode closed this issue

2026-07-22 18:35:26 +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#12