1.0A should be recognized as a valid version

harrisiirak · harrisiirak · commit a847812b2577 · 2012-07-05T23:00:12.000+03:00
diff --git a/lib/passport-http-oauth/strategies/consumer.js b/lib/passport-http-oauth/strategies/consumer.js
@@ -188,7 +188,7 @@ ConsumerStrategy.prototype.authenticate = function(req) {
     , verifier = params['oauth_verifier']
     , version = params['oauth_version']
 
-  if (version && version !== '1.0') {
+  if (version && (version !== '1.0' && version.toUpperCase() !== '1.0A')) {
     return this.fail(this._challenge('version_rejected'), 400);
   }
 
diff --git a/lib/passport-http-oauth/strategies/token.js b/lib/passport-http-oauth/strategies/token.js
@@ -179,7 +179,7 @@ TokenStrategy.prototype.authenticate = function(req) {
     , nonce = params['oauth_nonce']
     , version = params['oauth_version']
 
-  if (version && version !== '1.0') {
+  if (version && (version !== '1.0' && version.toUpperCase() !== '1.0A')) {
     return this.fail(this._challenge('version_rejected'), 400);
   }
 

Original file line number	Diff line number	Diff line change
`@@ -188,7 +188,7 @@ ConsumerStrategy.prototype.authenticate = function(req) {`
`188`	`188`	`, verifier = params['oauth_verifier']`
`189`	`189`	`, version = params['oauth_version']`
`190`	`190`
`191`		`- if (version && version !== '1.0') {`
	`191`	`+ if (version && (version !== '1.0' && version.toUpperCase() !== '1.0A')) {`
`192`	`192`	`return this.fail(this._challenge('version_rejected'), 400);`
`193`	`193`	`}`
`194`	`194`
Original file line number	Diff line number	Diff line change
`@@ -179,7 +179,7 @@ TokenStrategy.prototype.authenticate = function(req) {`
`179`	`179`	`, nonce = params['oauth_nonce']`
`180`	`180`	`, version = params['oauth_version']`
`181`	`181`
`182`		`- if (version && version !== '1.0') {`
	`182`	`+ if (version && (version !== '1.0' && version.toUpperCase() !== '1.0A')) {`
`183`	`183`	`return this.fail(this._challenge('version_rejected'), 400);`
`184`	`184`	`}`
`185`	`185`