Skip to content

HttpRequest with 0 write timeout may make program hangs #850

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

Closed
meor9zcu12 opened this issue Aug 7, 2023 · 4 comments · Fixed by #900
Closed

HttpRequest with 0 write timeout may make program hangs #850

meor9zcu12 opened this issue Aug 7, 2023 · 4 comments · Fixed by #900
Assignees

Comments

@meor9zcu12
Copy link

Firebase Admin Java SDK: v9.2.0

Problem encountered:

When under high traffic, FCM admin SDK will throw some errors like:

Error 1. com.google.firebase.messaging.FirebaseMessagingException: Unexpected HTTP response with status: 408

Error 2. com.google.firebase.messaging.FirebaseMessagingException: Unknown error while making a remote service call: Unexpected end of file from server

Error 3. com.google.firebase.messaging.FirebaseMessagingException: Unknown error while making a remote service call: Connection timed out (Connection timed out)

At the same time, after throwing these errors, FCM admin SDK may still make java program never stop because of FCM admin SDK still waiting response from fcm.googleapis.com

Findings:

FCM admin uses com.google.http-client, which writeTimeout = 0 by default, link

I can't find how ErrorHandlingHttpClient to set writeTimeout, it may be one of result that make program keep waiting response from fcm.googleapis.com under high traffic.

@google-oss-bot
Copy link

I couldn't figure out how to label this issue, so I've labeled it for a human to triage. Hang tight.

@TheRittler
Copy link
Contributor

I think we also ran into this Issue. Are there any news on this topic?

@chong-shao
Copy link
Contributor

Sorry for the delay in reply. We are currently investigating this issue. Will update here later.

@GarreltMock
Copy link

Any updates here?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants