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

🐛 BUG: Handle Gateway Errors #3847

Closed
PBhadoo opened this issue Aug 27, 2023 · 3 comments
Closed

🐛 BUG: Handle Gateway Errors #3847

PBhadoo opened this issue Aug 27, 2023 · 3 comments
Labels
bug Something that isn't working

Comments

@PBhadoo
Copy link

PBhadoo commented Aug 27, 2023

Which Cloudflare product(s) does this pertain to?

Wrangler core

What version(s) of the tool(s) are you using?

3.5.1

What version of Node are you using?

No response

What operating system are you using?

Windows

Describe the Bug

Sometimes Cloudflare API return HTML instead of JSON in response.

Happens when errors like Gateway Timeout or Error 500 Happens.

The Workers should handle this with try except or something else.

Gateway Timout is rare but happens and stops the entire script. instead it should retry atleast few times before stopping the entire script.

Please provide a link to a minimal reproduction

No response

Please provide any relevant error logs

Don't have that for JS as it happens so rare. And I forgot to copy it.

@PBhadoo PBhadoo added the bug Something that isn't working label Aug 27, 2023
@1000hz 1000hz added the awaiting reporter response Needs clarification or followup from OP label Aug 28, 2023
@1000hz
Copy link
Contributor

1000hz commented Aug 28, 2023

Hey @PBhadoo. Do you happen to know what command you were running when you encountered this unhandled error response?

@PBhadoo
Copy link
Author

PBhadoo commented Aug 29, 2023

Hey @PBhadoo. Do you happen to know what command you were running when you encountered this unhandled error response?

while uploading 18000 files project using

wrangler deploy pages PATH

@1000hz
Copy link
Contributor

1000hz commented Sep 6, 2023

This was fixed by #3758 and released in wrangler@3.6.0.

@1000hz 1000hz closed this as completed Sep 6, 2023
@1000hz 1000hz removed the awaiting reporter response Needs clarification or followup from OP label Sep 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something that isn't working
Projects
None yet
Development

No branches or pull requests

2 participants