Hi Oleg,

just a short glimpse...

ContentDecoder.isCompeted should be .isCompleted
ContentEncoder.isCompeted should be .isCompleted

NHttpClientConnection.getHttpResponse()
NHttpServerConnection.getHttpRequest()
We've avoided tying the request and response directly so far.
But it's probably not a big issue, so leave it that way.

NHttpServerConnection.submitContinue()
I don't like this at all. You are assuming that 100 continue
is the only preliminary response that will ever be sent.
That is true for the current HTTP specifications, but HTTP
explicitly allows for extensions. I don't see why we should
restrict our API in this case. Alternatives:

1. NHttpServerConnection.submitPreliminary(HttpResponse)
2. NHttpServerConnection.submitResponse(HttpResponse, boolean preliminary)

NHttpClientHandler.exception(...)
Will the client handler get a chance to handle other kinds
of exceptions, too? I usually prefer a generic error handling
method, where the handler decides what to do (if anything):

/** @return true if handled, false otherwise */
boolean NHttpClientHandler.exception(conn, Throwable)

NHttpServerHandler.exception(...)
See comments for client.

Your JavaDocing is much improved! I noticed a typo in
NHttpConnection.request*put: "oprtations".

cheers,
  Roland

---------------------------------------------------------------------
To unsubscribe, e-mail: httpclient-dev-unsubscribe@jaka...
For additional commands, e-mail: httpclient-dev-help@jaka...