Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Panic: ClientRequest oneshot sender must not be dropped #1471

Closed
rex4539 opened this issue Dec 7, 2020 · 3 comments · Fixed by #1531
Closed

Panic: ClientRequest oneshot sender must not be dropped #1471

rex4539 opened this issue Dec 7, 2020 · 3 comments · Fixed by #1531
Assignees
Labels
A-rust Area: Updates to Rust code C-bug Category: This is a bug

Comments

@rex4539
Copy link
Contributor

rex4539 commented Dec 7, 2020

Dec 07 08:56:27.460  INFO {zebrad="5b687f87"}:sync:obtain_tips: zebra_network::peer_set::set: network request with no ready peers: finding more peers, waiting for 18 peers to answer requests
Dec 07 10:42:24.595  INFO {zebrad="5b687f87"}:peer{addr=185.153.231.82:8233}:msg_as_req{msg=getheaders}:inbound:state: zebra_state::service: responding to peer GetBlocks or GetHeaders final_height=Height(1067845) response_len=127 chain_tip_height=Height(1067845) stop_height=None intersection_height=Some(Height(1067718))
Dec 07 10:42:24.775  INFO {zebrad="5b687f87"}:peer{addr=47.253.0.14:8233}:msg_as_req{msg=getheaders}:inbound:state: zebra_state::service: responding to peer GetBlocks or GetHeaders final_height=Height(1067845) response_len=127 chain_tip_height=Height(1067845) stop_height=None intersection_height=Some(Height(1067718))
Dec 07 10:42:25.279  INFO {zebrad="5b687f87"}:peer{addr=47.94.92.186:8233}:msg_as_req{msg=getheaders}:inbound:state: zebra_state::service: responding to peer GetBlocks or GetHeaders final_height=Height(1067845) response_len=127 chain_tip_height=Height(1067845) stop_height=None intersection_height=Some(Height(1067718))

Error

ClientRequest oneshot sender must not be dropped before send: Canceled

Metadata

key value
version 3.0.0-alpha.0
git commit 5b687f8
location zebra-network/src/peer/client.rs:84:26

SpanTrace

SpanTrace:
   0: zebrad::components::sync::obtain_tips
             at zebrad/src/components/sync.rs:255
   1: zebrad::components::sync::sync
             at zebrad/src/components/sync.rs:150
   2: zebrad::commands::
           with zebrad="5b687f87"
             at zebrad/src/commands.rs:56

Backtrace

Backtrace:
   0: backtrace::backtrace::libunwind::trace
             at .cargo/registry/src/github.com-1ecc6299db9ec823/backtrace-0.3.54/src/backtrace/libunwind.rs:90:5
      backtrace::backtrace::trace_unsynchronized
             at .cargo/registry/src/github.com-1ecc6299db9ec823/backtrace-0.3.54/src/backtrace/mod.rs:66:5
   1: backtrace::backtrace::trace
             at .cargo/registry/src/github.com-1ecc6299db9ec823/backtrace-0.3.54/src/backtrace/mod.rs:53:14
   2: backtrace::capture::Backtrace::create
             at .cargo/registry/src/github.com-1ecc6299db9ec823/backtrace-0.3.54/src/capture.rs:176:9
   3: backtrace::capture::Backtrace::new
             at .cargo/registry/src/github.com-1ecc6299db9ec823/backtrace-0.3.54/src/capture.rs:140:22
   4: color_eyre::config::PanicHook::panic_report
             at .cargo/registry/src/github.com-1ecc6299db9ec823/color-eyre-0.5.10/src/config.rs:982:18
   5: color_eyre::config::PanicHook::into_panic_hook::{{closure}}
             at .cargo/registry/src/github.com-1ecc6299db9ec823/color-eyre-0.5.10/src/config.rs:961:29
   6: std::panicking::rust_panic_with_hook
             at /rustc/1c389ffeff814726dec325f0f2b0c99107df2673/library/std/src/panicking.rs:595:17
   7: std::panicking::begin_panic_handler::{{closure}}
             at /rustc/1c389ffeff814726dec325f0f2b0c99107df2673/library/std/src/panicking.rs:497:13
   8: std::sys_common::backtrace::__rust_end_short_backtrace
             at /rustc/1c389ffeff814726dec325f0f2b0c99107df2673/library/std/src/sys_common/backtrace.rs:141:18
   9: rust_begin_unwind
             at /rustc/1c389ffeff814726dec325f0f2b0c99107df2673/library/std/src/panicking.rs:493:5
  10: core::panicking::panic_fmt
             at /rustc/1c389ffeff814726dec325f0f2b0c99107df2673/library/core/src/panicking.rs:92:14
  11: core::option::expect_none_failed
             at /rustc/1c389ffeff814726dec325f0f2b0c99107df2673/library/core/src/option.rs:1268:5
  12: core::result::Result<T,E>::expect
             at .rustup/toolchains/nightly-x86_64-apple-darwin/lib/rustlib/src/rust/library/core/src/result.rs:933:23

@mpguerra mpguerra added C-bug Category: This is a bug S-needs-triage Status: A bug report needs triage labels Dec 7, 2020
@teor2345
Copy link
Contributor

teor2345 commented Dec 7, 2020

@hdevalence any insight into what's going on here?

@teor2345 teor2345 added this to the First Alpha Release milestone Dec 7, 2020
@teor2345
Copy link
Contributor

teor2345 commented Dec 7, 2020

We decided to fix this in the first alpha.

@teor2345 teor2345 assigned teor2345 and hdevalence and unassigned teor2345 Dec 7, 2020
@teor2345 teor2345 changed the title Crash during sync Panic: ClientRequest oneshot sender must not be dropped Dec 7, 2020
@teor2345 teor2345 added A-rust Area: Updates to Rust code and removed S-needs-triage Status: A bug report needs triage labels Dec 7, 2020
@teor2345
Copy link
Contributor

teor2345 commented Dec 8, 2020

I've documented this first alpha known issue in PR #1478

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-rust Area: Updates to Rust code C-bug Category: This is a bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants