Skip to content

Commit

Permalink
Config Cleanup + Feedback Integration
Browse files Browse the repository at this point in the history
 * `get_max` => `get_max_request_entity_size`
 * Removed local test core from the config
  • Loading branch information
kfitzgerald committed Aug 26, 2015
1 parent 06e1838 commit ba9fa70
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions lib/solr.js
Original file line number Diff line number Diff line change
Expand Up @@ -83,7 +83,7 @@ function Client(options){
agent : options.agent,
secure : options.secure || false,
bigint : options.bigint || false,
get_max: options.get_max || 2048
get_max_request_entity_size: options.get_max_request_entity_size || 2048
};

// Default paths of all request handlers
Expand Down Expand Up @@ -577,7 +577,7 @@ Client.prototype.get = function(handler,query,callback){
.join('/'),
approxUrlLength = 10 + this.options.host.length + (this.options.port+"").length + fullPath.length; // Buffer (10) accounts for protocol and special characters like ://, port colon, and initial slash etc

if (approxUrlLength <= this.options.get_max) {
if (approxUrlLength <= this.options.get_max_request_entity_size) {
var params = {
host: this.options.host,
port: this.options.port,
Expand Down
2 changes: 1 addition & 1 deletion test/config.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,6 @@
"host": "127.0.0.1",
"port": 8983,
"path": "/solr",
"core": "solr_client_test"
"core": null
}
}

0 comments on commit ba9fa70

Please sign in to comment.