lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Uwe Schindler (JIRA)" <>
Subject [jira] [Commented] (SOLR-5517) Return HTTP error on POST requests with no Content-Type
Date Sun, 01 Dec 2013 13:50:35 GMT


Uwe Schindler commented on SOLR-5517:

I looked around in the RequestHandlers:
- SearchRequestHandler is the bad guy, which does not throw SolrException, if the request
does not fit a "search". The SearchHandler should throw SolrException, if request.streams.length>0.
For a search request, there should be not passed any streams. If you then have a typo in your
content-type, the request handler would throw exception (because the FormData handler is not
invoked and the RawRequestParser then sets a single ContentStream).
- UpdateRequestHandler should use the new HTTP status constant here to be consistent:
      if( type == null ) { // Normal requests will not get here.
        throw new SolrException(ErrorCode.BAD_REQUEST, "Missing ContentType");
      int idx = type.indexOf(';');
      if(idx>0) {
        type = type.substring(0,idx);
      ContentStreamLoader loader = loaders.get(type);
      if(loader==null) {
        throw new SolrException(ErrorCode.BAD_REQUEST, "Unsupported ContentType: "
            +type+ "  Not in: "+loaders.keySet());
      if(loader.getDefaultWT()!=null) {
      loader.load(req, rsp, stream, processor);

> Return HTTP error on POST requests with no Content-Type
> -------------------------------------------------------
>                 Key: SOLR-5517
>                 URL:
>             Project: Solr
>          Issue Type: Improvement
>            Reporter: Ryan Ernst
>         Attachments: SOLR-5517.patch, SOLR-5517.patch
> While the http spec states requests without a content-type should be treated as application/octet-stream,
the html spec says instead that post requests without a content-type should be treated as
a form (  It would be nice to
allow large search requests from html forms, and not have to rely on the browser to set the
content type (since the spec says it doesn't have to).

This message was sent by Atlassian JIRA

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message