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.

Sign up for GitHub

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

Jump to bottom

bugfix: prevent ETIMEOUT error from shuting down explorer #258

Merged
merged 1 commit into from Jul 28, 2021
Merged

bugfix: prevent ETIMEOUT error from shuting down explorer #258

merged 1 commit into from Jul 28, 2021

Conversation

ThomasRalee
Copy link
Contributor

This PR:

  • prevents the explorer from shutting down on ETIMEOUT error.

Error stack:

[2021-07-21T14:26:47.644] [ERROR] FabricGateway - Failed to get chain info from channel defaultchannel :  FabricError: Query failed. Errors: ["Error: 13 INTERNAL: Received RST_STREAM with code 2 triggered by internal client error: read ETIMEDOUT"]
    at SingleQueryHandler.evaluate (.../blockchain-explorer/node_modules/fabric-network/lib/impl/query/singlequeryhandler.js:47:23)
    at processTicksAndRejections (internal/process/task_queues.js:97:5)
    at async Transaction.evaluate (.../blockchain-explorer/node_modules/fabric-network/lib/transaction.js:276:25)
[2021-07-21T14:26:47.644] [INFO] SyncServices - syncBlocks: Failed to retrieve channelInfo >> defaultchannel

@nekia
Copy link
Contributor

nekia commented Jul 27, 2021

Thanks @ThomasRalee . Are you still work in progress? All initial commits seem to be reverted.

@ankitm123 @ThomasRalee
docs: add code snippet for admin cert modification (#257)
8501380 
Signed-off-by: Thomas <thomas.leera@gmail.com>
@sonarcloud
Copy link

sonarcloud bot commented Jul 28, 2021

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@ThomasRalee
Copy link
Contributor Author

@nekia, updated the PR. Sorry for the inconvenience caused.

nekia
nekia approved these changes Jul 28, 2021
View reviewed changes
Copy link
Contributor

@nekia nekia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks. Looks good to me.

@nekia nekia merged commit 3ed0435 into hyperledger-labs:main Jul 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nekia nekia nekia approved these changes

@JeevaSang JeevaSang Awaiting requested review from JeevaSang

@nfrunza nfrunza Awaiting requested review from nfrunza

@vchinoy-da vchinoy-da Awaiting requested review from vchinoy-da

@xspeedcruiser xspeedcruiser Awaiting requested review from xspeedcruiser

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@ThomasRalee @nekia @ankitm123