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

Detail view shows disconnected peer's info #158

Closed
hebasto opened this issue Dec 20, 2020 · 6 comments
Closed

Detail view shows disconnected peer's info #158

hebasto opened this issue Dec 20, 2020 · 6 comments
Labels
Bug Something isn't working

Comments

@hebasto
Copy link
Member

hebasto commented Dec 20, 2020

To reproduce

  1. select the "Peer" tab in the "Node window"

  2. choose any peer by clicking its row in the table, the detail view shows the peers's info

  3. wait until the chosen peer disconnect

Expected behavior

The detail view stops showing the disconnected peer's info (probably, show nothing).

Actual behavior

The detail view is still showing the disconnected peer's info that is wrong.

System information

Bitcoin Core 0.21.0rc3

Linux Mint 20

@hebasto hebasto added the Bug Something isn't working label Dec 20, 2020
@hebasto
Copy link
Member Author

hebasto commented Dec 20, 2020

cc @promag

@maflcko
Copy link
Contributor

maflcko commented Dec 20, 2020

Is this a regression?

@hebasto
Copy link
Member Author

hebasto commented Dec 21, 2020

Hmm, cannot reproduce this issue on regtest connecting some nodes each to other.

@hebasto
Copy link
Member Author

hebasto commented Dec 22, 2020

Ok, I've managed to reproduce it:
Screenshot from 2020-12-22 12-01-47

Steps to reproduce: use column sorting in peer table, and then let a peer to disconnect.

@hebasto
Copy link
Member Author

hebasto commented Dec 22, 2020

@MarcoFalke

Is this a regression?

No, it isn't. The 0.20.0 has the same behavior.

@hebasto
Copy link
Member Author

hebasto commented Dec 22, 2020

Not observed on master(f4ac48d) due to the #13. But peer table logic is still broken if the table is sorted, see #18.

Closing.

@hebasto hebasto closed this as completed Dec 22, 2020
@bitcoin-core bitcoin-core locked as resolved and limited conversation to collaborators Feb 15, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants