Merge pull request #10 from mattpass/master

michael · michael · commit a944dcd0d72f · 2012-07-19T10:47:41.000-07:00
Extra status code (304) and Content-Type header added
diff --git a/github.js b/github.js
@@ -19,14 +19,15 @@
       xhr.open(method, API_URL + path);
       xhr.onreadystatechange = function () {
         if (this.readyState == 4) {
-          if (this.status >= 200 && this.status < 300) {
+          if (this.status >= 200 && this.status < 300 || this.status === 304) {
             cb(null, raw ? this.responseText : JSON.parse(this.responseText));
           } else {
             cb(this.status);
           }
         }
       }
       xhr.setRequestHeader('Accept','application/vnd.github.raw');
+      xhr.setRequestHeader('Content-Type','application/x-www-form-urlencoded');
       if (
          (options.auth == 'oauth' && options.token) ||
          (options.auth == 'basic' && options.username && options.password)

Original file line number	Diff line number	Diff line change
`@@ -19,14 +19,15 @@`
`19`	`19`	`xhr.open(method, API_URL + path);`
`20`	`20`	`xhr.onreadystatechange = function () {`
`21`	`21`	`if (this.readyState == 4) {`
`22`		`- if (this.status >= 200 && this.status < 300) {`
	`22`	`+ if (this.status >= 200 && this.status < 300 \|\| this.status === 304) {`
`23`	`23`	`cb(null, raw ? this.responseText : JSON.parse(this.responseText));`
`24`	`24`	`} else {`
`25`	`25`	`cb(this.status);`
`26`	`26`	`}`
`27`	`27`	`}`
`28`	`28`	`}`
`29`	`29`	`xhr.setRequestHeader('Accept','application/vnd.github.raw');`
	`30`	`+ xhr.setRequestHeader('Content-Type','application/x-www-form-urlencoded');`
`30`	`31`	`if (`
`31`	`32`	`(options.auth == 'oauth' && options.token) \|\|`
`32`	`33`	`(options.auth == 'basic' && options.username && options.password)`