Fix Functions Call Timeouts

[allisonbm92]

Firebase Functions incorrectly implements call timeouts. This was originally raised in #212 but was never correctly fixed.

OkHttp provides four different kinds of timeouts: connect, read, write, and call. Functions only configures the call timeout. The default connect, read, and write timeouts are still enforced, meaning the server must respond within 10 seconds. The default Functions timeout of 70 seconds, or any user provided timeout, are essentially ignored.

This can be seen from this test run: https://storage.cloud.google.com/android-ci/pr-logs/pull/firebase_firebase-android-sdk/578/new-smoke-tests/1149370348362272769/artifacts/walleye-26-en-portrait/test_cases/0006_logcat

[google-oss-bot]

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

[mikemilla]

@allisonbm92 I may be overlooking a test somewhere, but maybe a new test or an update to the following test would be nice for this fix as well?

firebase-android-sdk/firebase-functions/src/androidTest/java/com/google/firebase/functions/CallTest.java

Line 218 in c4f0bb3

public void testTimeout() {

Thanks for the fix btw! Been needing this for a while 😊

[shc2018]

Thank you for the fix. Is it possible to know estimate date/days for this release?

[allisonbm92]

@shc2018 We're planning on releasing it by the end of next week!

[shc2018]

Great! Thank you.

[permadiberlianto]

Waiting for updates, I am getting this error too. Thank you.