-
Type: Improvement
-
Resolution: Unresolved
-
Priority: Minor - P4
-
None
-
Affects Version/s: None
-
Component/s: Sync
-
?
-
6301
-
Not Needed
The sync client will ignore certain config options if they don't have values it deems "sensible". While this kind of makes sense to me, silently ignoring these values does not seem right. Ideally, we should accept whatever the SDK supplies and emit warning messages if the timeouts are too short. The reasoning here is that there are certain error conditions, which we can write integration tests for if we reduce the connect timeout significantly, but as it stands today, we can't.
Here's the code I think we should change to log warnings rather than ignore certain values: