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

Internal errors are not visible in my code. C# #22297

Open
skuvv opened this issue Apr 26, 2024 · 5 comments
Open

Internal errors are not visible in my code. C# #22297

skuvv opened this issue Apr 26, 2024 · 5 comments
Assignees

Comments

@skuvv
Copy link

skuvv commented Apr 26, 2024

Operating System

Windows 10

Programming Languages

No response

CCXT Version

4.3.6

Description

When something happens inside ccxt threads I never see it.
I can only catch by using the debug version of ccxt from the source code.
I think I should be able to be notified about internal errors.

Public error delegate is good thing for pushing such messages.

For example this issue #22037 show nothing, but has important information inside.

Code

  

@carlosmiei carlosmiei self-assigned this Apr 27, 2024
@carlosmiei
Copy link
Collaborator

Hello @skuvv, thanks once again for bringing our attention to this.

Our current approach is to bubble up internal exceptions to the user (that's how we do in the other languages) but since c# is a bit different we might still have some unexpected behaviors. I will let you know if I find something.

Just to make it clear, basically, you're saying that on the WS side, when there's an issue, you never see it explicitly, right?

@skuvv
Copy link
Author

skuvv commented Apr 27, 2024

@carlosmiei yes, correct.
But this also actual for all internal errors not only for WS

@ttodua
Copy link
Member

ttodua commented Apr 28, 2024

late reply here, I'll post an update within 24 hours

@ttodua
Copy link
Member

ttodua commented Apr 30, 2024

@skuvv I had a delay, I am still looking into this problem, hard to intercept but i would have some thoughts on this

@skuvv
Copy link
Author

skuvv commented May 2, 2024

@ttodua thanks

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

3 participants