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

[UX] blockchain explorer silently "hangs" if no chain is running #129

Open
ixje opened this issue Mar 9, 2022 · 5 comments
Open

[UX] blockchain explorer silently "hangs" if no chain is running #129

ixje opened this issue Mar 9, 2022 · 5 comments

Comments

@ixje
Copy link

ixje commented Mar 9, 2022

Opening the block chain explorer while the chain is not running gives the following screen
image
The Loading... message suggests it's trying to do something while in reality it will hang there indefinitely. I think the screen should give a different view when the chain is not running or becomes non-responsive.

@robliou
Copy link

robliou commented May 27, 2022

I wasn’t able to recreate this error. When I tried, the blockchain explorer opened just fine for me…?

@ixje
Copy link
Author

ixje commented May 28, 2022 via email

@robliou
Copy link

robliou commented May 28, 2022

Weird. I think I may have encountered that error before. However, I just tried again but again, wasn't able to recreate the issue. For the record, I made sure that I was not connected to any blockchain, and then tried opening the blockchain explorer for 0-N3 Mainnet, 1-N3 TestNet, and 3-Single-Node.Neo-Express. All worked OK (see screenshot below). Note that I can't connect to the other two blockchains, so connecting to the blockchain explorer for them would be a moot point..

A6

@ixje
Copy link
Author

ixje commented May 30, 2022

Mainnet and Testnet are expected to always work (assuming internet connectivity) because they're always online. The single-node is working because from the Terminal tab at the bottom I can see the chain is running (the name this terminal belongs to is also displayed at the far right (same height as the Terminal tab) and confirms it is the single node.neo-express that is running. If you press the trash bin icon on the far right while having this terminal open, then close the block chain explorer tab and re-open the explorer it should fail. Here's a video displaying the issue

explorer-2022-05-30_08.22.37.mp4

I think what the above really shows is that this "Neo: Not connected" status bar at the bottom is not what we think it is. If it should represent which chain is running, then it is malfunctioning. If it should represent something else, then whatever it is supposed to be it is not clear and leads to confusion.

@ixje
Copy link
Author

ixje commented May 30, 2022

So what I'm trying to suggest with the original issue is to probably look here
https://github.com/ngdenterprise/neo3-visual-tracker/blob/b50ab36f58d98497b4d2e2989937ba7dcbf677a1/src/extension/panelControllers/trackerPanelController.ts#L53

then when blockchainMonitor.state.isHealthy is false display some other panel. isHealthy is controlled here
https://github.com/ngdenterprise/neo3-visual-tracker/blob/b50ab36f58d98497b4d2e2989937ba7dcbf677a1/src/extension/blockchainMonitor/blockchainMonitorInternal.ts#L262-L267
if we can't get the blockcount (which is also an RPC command) then we can't get any blocks either and there should probably be some logic added there that doesn't try to fetch any blocks while we know it is going to fail anyway.

The following screenshot shows that we're failing to connect to the RPC node and that's why we hang indefinitely

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants