jleony/nanoreth
1
0
Fork 0
mirror of https://github.com/hl-archive-node/nanoreth.git synced 2025-12-06 10:59:55 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: log actual notification id in exex manager (#7874)

Browse Source
This commit is contained in:
Oliver Nordbjerg
2024-04-25 15:55:37 +02:00
committed by GitHub
parent ed45c3c10d
commit 421888d22f
1 changed files with 7 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
crates/exex/src/manager.rs
View File

@ -280,12 +280,16 @@ impl Future for ExExManager {
// it is a logic error for this to ever underflow since the manager manages the
// notification IDs
let notification_id = exex
let notification_index = exex
.next_notification_id
.checked_sub(self.min_id)
.expect("exex expected notification ID outside the manager's range");
if let Some(notification) = self.buffer.get(notification_id) {
debug!(exex.id, notification_id, "sent notification to exex");
if let Some(notification) = self.buffer.get(notification_index) {
debug!(
exex.id,
notification_id = exex.next_notification_id,
"sent notification to exex"
);
if let Poll::Ready(Err(err)) = exex.send(cx, notification) {
// the channel was closed, which is irrecoverable for the manager
return Poll::Ready(Err(err.into()))