search

zapier / zapier-platform

The SDK for you to build an integration on Zapier
https://docs.zapier.com/platform
Other
348 stars 188 forks source link

chore(legacy-scripting-runner) Release legacy-scripting-runner v3.8.13 #915

Closed rnegron closed 6 days ago

rnegron commented 1 week ago

3.8.13

rnegron commented 1 week ago

Hmm, seeing some tests fail around HTTP requests. Flaky? Although I can recreate them locally

1) request client
       should allow overriding the user-agent header:
     request to https://httpbin.zapier-tooling.com/get failed, reason: read ECONNRESET
What happened:
  Starting GET request to https://httpbin.zapier-tooling.com/get
  request to https://httpbin.zapier-tooling.com/get failed, reason: read ECONNRESET

First thought is this may be related to http-patching (like in https://github.com/zapier/zapier-platform/pull/905/files) but haven't dug into it...

eliangcs commented 1 week ago

I reran the failing tests and they passed. They look flaky tests to me. @rnegron are you still able to recreate them locally? If yes, we should look into it.

rnegron commented 1 week ago

@eliangcs Seems like it might have been https://httpbin.zapier-tooling.com that was flaky 😮 Thanks for checking!