KronosClock returns stale cached time after device reboot if syncInBackground is called before getCurrentTime
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức phù hợp với người mới
- 45/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Ít trao đổi
- Công nghệ
- kotlin
- Lĩnh vực
- networking
Hướng nghiên cứu
Bắt đầu trong SntpService bằng cách theo dõi syncInBackground, getCurrentTime, state và isCachedFromPreviousBoot trong quá trình xử lý khởi động lại. Xem lại PR #51 và issue #61 để hiểu hành vi liên quan của state; được xem là hoàn tất khi thời gian đã lưu trong cache bị cũ không được trả về trong quá trình đồng bộ hóa và cache không bị xóa sai sau các thay đổi thời gian thủ công.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
PR https://github.com/lyft/Kronos-Android/pull/51 attempted to fix an issue where the local cache was not cleared after a device reboot, but it overlooked another edge case.
If the user calls syncInBackground before getCurrentTime, the SntpService.state variable will be set to State.SYNCING. As a result, the isCachedFromPreviousBoot check always evaluates to false while the sync is in progress. This creates a short window (a few seconds) during which getCurrentTime may return an incorrectly computed time based on a stale local cache.
Additionally, PR https://github.com/lyft/Kronos-Android/pull/51 removes State.INIT, which introduces another issue described in https://github.com/lyft/Kronos-Android/issues/61, where the local cache is cleared every time the user manually changes the system time.
- Ngôn ngữ chính
- Kotlin
- Star
- 287
- Fork
- 17
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Chuẩn bị môi trường
Dự án này không cung cấp dev container, Dockerfile hay hướng dẫn đóng góp, nên bạn cần tự thiết lập môi trường: hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của lyft/Kronos-Android
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
lyft/Kronos-Android#66 ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 30/100
lyft/Kronos-Android#65 ·
-
Wrong TimeĐang mở
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 25/100
lyft/Kronos-Android#62 ·
-
getCurrentTimeMs() return the wrong date time everytime I manually change the device clockCó thể đã có người làm @Rajin9601 đã nhận 1710 ngày trước. Đang mở
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 25/100
lyft/Kronos-Android#61 · 10 bình luận · 2 reaction ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 25/100
lyft/Kronos-Android#53 ·
Tất cả issue của lyft/Kronos-Android
Issue tương tự
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 80/100
software-mansion/enriched-markdown#1033 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
enhancement
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
-
enhancement
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 67/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Improve navigation menuĐang mởapp
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
LawnchairLauncher/lawnicons#4180 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Kotlin extra warnings in GeneratedAppGlideModuleImplCó thể đã có người làm @bacecek đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
Maintainer thường phản hồi trong vòng 18 ngày