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

Error on Timeout #101

Open
t0mstah opened this issue Apr 6, 2021 · 3 comments
Open

Error on Timeout #101

t0mstah opened this issue Apr 6, 2021 · 3 comments
Assignees
Labels
bug Something isn't working

Comments

@t0mstah
Copy link

t0mstah commented Apr 6, 2021

Expected Behavior

Unexpected error on Timeout when calling Amplitude logEvent.

Current Behavior

    at Timeout._onTimeout (/workspace/node_modules/@amplitude/utils/dist/src/queue.js:67:48)
    at ontimeout (timers.js:436:11)
    at tryOnTimeout (timers.js:300:5)
    at listOnTimeout (timers.js:263:5)
    at Timer.processTimers (timers.js:223:10) 

Possible Solution

Didn't happen in previous SDKs. Looks like there were some recent code changes to that queue.js file.

Steps to Reproduce

Happens occasionally on calls to logEvent.

Environment

  • SDK Version: 1.5.3
  • Node version: 10
@t0mstah t0mstah added the bug Something isn't working label Apr 6, 2021
@axelvaindal
Copy link

axelvaindal commented Apr 8, 2021

I have the same bug and it affects approximately 25% of my logs when we log approximately 300k events per day only on this call.

I'll roll back to a previous version to see if it helps.

@t0mstah
Copy link
Author

t0mstah commented Apr 15, 2021

Bug is only in new version. Temp fix is to roll back to older version.

@ajhorst
Copy link
Contributor

ajhorst commented Apr 15, 2021

@tfang17 we released v1.5.4 earlier this week with fixes to the timeout logic. When you have a chance, upgrade to that version and let me know if you're still seeing timeout issues

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants