Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

Companion ACKs a direct message that it then drops when the offline queue is full

Đang mở
  • Có gì không ổn?
#3,518 1 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

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
48/100
Loại issue
Lỗi
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
c
Lĩnh vực
backend, networking

Hướng nghiên cứu

Start with addToOfflineQueue() in examples/companion_radio/MyMesh.cpp (lines 224–246) and the ACK path in BaseChatMesh::onPeerDataRecv() in src/helpers/BaseChatMesh.cpp (around lines 239–254). Trace how onMessageRecv() handles direct messages when the queue is full and how that result controls ACKs. Done when a dropped message is not acknowledged, with queue eviction behavior clarified if implementing the broader suggestion.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

When the companion's offline queue is full (OFFLINE_QUEUE_SIZE, default 16) and holds no channel messages, addToOfflineQueue() drops the incoming direct message (examples/companion_radio/MyMesh.cpp, dev, lines 224-246). BaseChatMesh::onPeerDataRecv() still sends the ACK afterwards (src/helpers/BaseChatMesh.cpp, around lines 239-254), because onMessageRecv() returns void.

Result: the sender sees the message as delivered, but it never reaches the app. This happens when the phone is disconnected for a while and more than 16 DMs arrive.

Suggested fix, in increasing scope:

  1. Let onMessageRecv() return whether the message was stored, and only ACK when it was. The sender then retries later, as it would for an unreachable node.
  2. When the queue is full of DMs, evict other non-DM frames first, and only refuse (no ACK) as a last resort.
  3. Optionally, keep the offline queue in flash so a reboot or empty battery does not lose ACKed messages.

Related: #1834 (failed delivery shown but message received) is the inverse case; this one reports success while the message is lost.

Ngôn ngữ chính
C
Star
3.9k
Fork
1.5k
Merge trung bình
1 ngày 3 giờ
Pull request đã merge (30 ngày)
24

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

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. 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.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của meshcore-dev/MeshCore

Tất cả issue của meshcore-dev/MeshCore

Issue tương tự

Thêm issue về C

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.