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

Make the HTTP module throw a "Not connected" exception #40

Open
ohager opened this issue Jan 4, 2023 · 0 comments
Open

Make the HTTP module throw a "Not connected" exception #40

ohager opened this issue Jan 4, 2023 · 0 comments
Assignees
@ohager
Labels
enhancement New feature or request http Http Package

Comments

@ohager
Copy link
Member

ohager commented Jan 4, 2023

Is your feature request related to a problem? Please describe.
Atm, when HTTP requests fail due to unreachable hosts, the following error is thrown:

TypeError: Cannot destructure property 'response' of '(intermediate value)' as it is undefined.
    at ChainService.<anonymous> (chainService.js:47:20)

This is an indirect indicator that a request did not succeed

Describe the solution you'd like
It would be easier for developers, if there is a specific "Not connected" exception. This can be used programmatically to take further actions, like fallback to other hosts, showing an UI message, or open host configuration page....
@ohager ohager added enhancement New feature or request http Http Package labels Jan 4, 2023
@ohager ohager self-assigned this Jan 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ohager ohager

Labels
enhancement New feature or request http Http Package
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ohager