You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I think the root cause of the issue is that currently we don't pass ctx context.Context as the first argument to all functions that do i/o / networking / etc. I'll take care of this
I think the root cause of the issue is that currently we don't pass ctx context.Context as the first argument to all functions that do i/o / networking / etc. I'll take care of this
We use
context.Background
andcontext.TODO
throughout the code.This might become a problem if a Go routine need to be canceled when another is canceled, or if ZetaClient is incorporated in another package.
It is also in general proper to determine determine a unified context that is used in the package.
Solution:
Ctx
as a field of the different object:CoreObserver
,ZetaCoreBridge
,Signer
, etc...context.Background
with these valuescontext.Background
the root contextThe text was updated successfully, but these errors were encountered: