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.

Sign up for GitHub

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

Jump to bottom

[Workers] Add await writer.close(), otherwise it takes 4m30s to get the response from TCP sockets API #15817

Closed
wants to merge 1 commit into from
Closed

[Workers] Add await writer.close(), otherwise it takes 4m30s to get the response from TCP sockets API #15817

wants to merge 1 commit into from

Conversation

kyouheicf
Copy link
Contributor

Summary

Add await writer.close(), otherwise it takes 4m30s to get the response from TCP sockets

% date && curl https://abc.xyz.workers.dev && date
Fri Jul 26 14:33:23 JST 2024
...
Fri Jul 26 14:37:53 JST 2024

@kyouheicf
Add await writer.close(), otherwise it takes 4m30s to get the response
c4eaa9b 
Add await writer.close(), otherwise it takes 4m30s to get the response
@github-actions github-actions bot added the product:workers Related to Workers product label Jul 26, 2024
@kyouheicf
Copy link
Contributor Author

PCX-12482

@KimJ15
Copy link
Collaborator

KimJ15 commented Sep 27, 2024

👋Hi there, we’re going to close this PR. All PRs prior to August 12 need to be reworked due to the platform migration to Astro.

@KimJ15 KimJ15 closed this Sep 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@irvinebroque irvinebroque Awaiting requested review from irvinebroque

@jasnell jasnell Awaiting requested review from jasnell

@mikenomitch mikenomitch Awaiting requested review from mikenomitch

@kodster28 kodster28 Awaiting requested review from kodster28

@GregBrimble GregBrimble Awaiting requested review from GregBrimble

Assignees

@jasnell jasnell

@irvinebroque irvinebroque

@mikenomitch mikenomitch

@GregBrimble GregBrimble

@kodster28 kodster28

Labels
product:workers Related to Workers product size/xs
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@kyouheicf @KimJ15 @jasnell @irvinebroque @mikenomitch @GregBrimble @kodster28