A few questions aboutthis...
: SOLR-214 -- override getReader() explicitly. subclass called instance
: variables, not getContentType()
1) wouldn't it make sense to make ContentStreamBase use getContentType()
instead of contentType so that it can be subclassed in this case, instead
of duplicating hte getReader in every subclass?
2) ...
: + }
: + public Reader getReader() throws IOException {
: + String charset = ContentStreamBase.getCharsetFromContentType(
req.getContentType() );
: + return charset == null
: + ? new InputStreamReader( getStream() )
: + : new InputStreamReader( getStream(), charset );
...do we really want to use the single arg constructure if the request has
no charset, or should we assume UTF-8 ? ... i seem to i recall we talked
about this once before and i argued in favor of using the defualt and letting
the servlet container "do the right thing", but i thought the consesus was
to assume UTF-8 if the request itself didn't contain any explicit charset?
these are the two (semi-jumbled) threads i'm thinking of...
http://www.nabble.com/resin-and-UTF-8-in-URLs-tf3152910.html
http://www.nabble.com/charset-in-POST-from-browser-tf3153057.html
-Hoss
