Merge pull request #8 from joshdrake/master

jmdobry · jmdobry · commit fa6c996decbd · 2014-12-01T11:03:31.000-07:00
Fixed handling of 'forceTrailingSlash' option.
diff --git a/src/index.js b/src/index.js
@@ -73,7 +73,7 @@ dsHttpAdapterPrototype.HTTP = function (config) {
   var _this = this;
   var start = new Date();
   config = deepMixIn(config, _this.defaults.httpConfig);
-  if (_this.defaults.forceTrailingSlash && config.url[config.url.length] !== '/') {
+  if (_this.defaults.forceTrailingSlash && config.url[config.url.length - 1] !== '/') {
     config.url += '/';
   }
 

Original file line number	Diff line number	Diff line change
`@@ -73,7 +73,7 @@ dsHttpAdapterPrototype.HTTP = function (config) {`
`73`	`73`	`var _this = this;`
`74`	`74`	`var start = new Date();`
`75`	`75`	`config = deepMixIn(config, _this.defaults.httpConfig);`
`76`		`- if (_this.defaults.forceTrailingSlash && config.url[config.url.length] !== '/') {`
	`76`	`+ if (_this.defaults.forceTrailingSlash && config.url[config.url.length - 1] !== '/') {`
`77`	`77`	`config.url += '/';`
`78`	`78`	`}`
`79`	`79`