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

Improve reliability of Stitch webhook #53

Closed
6 tasks done
chadokruse opened this issue Jan 6, 2019 · 2 comments
Closed
6 tasks done

Improve reliability of Stitch webhook #53

chadokruse opened this issue Jan 6, 2019 · 2 comments
Labels
dynamic Tasks that require technologies loaded AFTER pageload (e.g. Vue, Stitch, Firebase)

Comments

@chadokruse
Copy link
Member

chadokruse commented Jan 6, 2019

  • Explore why adding retry-axios has not improved error rate of Webhooks
  • After clearing Cloudflare cache, errors level now at standard/acceptable levels

Related to Cloudflare somehow?
Would explain recent Enativ-based run, where 50% appears in Stitch logs as being captured, while 50% threw the generic Network Error.


  • Add single retry method
  • Only track errors on retry, not on first fail
  • Is Vue triggering a new Axios call after search refinements? A: No
  • Figure out root cause of silent Axios failures (see "troubleshooting" issue) A: pre-flight issue
  • Implement method to reduce silent Axios failures (see "troubleshooting" issue)

Re: root cause, question is same as this thread in SO

The request was made but no response was received

Retry mechanisms
Retry Axios npm package
https://www.npmjs.com/package/retry-axios

Homegrown example(s)
https://stackoverflow.com/a/49976113/1944104
axios/axios#164
axios/axios#934

@chadokruse
Copy link
Member Author

No longer needed as moving to Stitch auth. Using Stitch Webhook was an interim fix to the high error rate using Stitch auth, which was likely due to this Stitch bug.

@chadokruse
Copy link
Member Author

Re-opening as Stitch Webhooks will likely always play a role. They are perfect for pulling in fully public information, especially for non-logged in users as they do NOT require the Stitch SDK (e.g. fast page loads)

@chadokruse chadokruse reopened this Apr 22, 2019
@chadokruse chadokruse added the dynamic Tasks that require technologies loaded AFTER pageload (e.g. Vue, Stitch, Firebase) label Apr 22, 2019
@chadokruse chadokruse reopened this May 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dynamic Tasks that require technologies loaded AFTER pageload (e.g. Vue, Stitch, Firebase)
Projects
None yet
Development

No branches or pull requests

1 participant