apiengine/seoserver — pull request #11 by richard-flosi (2013-09-03 23:19:52+00:00)
Title: Removed minResponses and minContentLength from config options and determine them dynamically based on the first and second responses.

I think having the minResponses and minContentLength in the config would be annoying and problematic to users over time as well as a barrier to entry. By changing a few lines here we can make some reasonable guesses about what these values should be.

When we get the first response we use the bodySize to set the minContentLength.
When we get the second response we use the current requestCount to set the minResponses.

If there is a desire for configuration here in the future, then we can set it up so that the config overrides the defaults, but I suspect that will be undesirable at the moment.

Let me know if you have any comments, questions, or suggestions.

Thanks!