[incubator-openwhisk-apigateway] branch master updated: remove redis.close causing intermittent failures (#211)

[cs...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

csantanapr pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-openwhisk-apigateway.git

The following commit(s) were added to refs/heads/master by this push:
       new  c3f46da   remove redis.close causing intermittent failures (#211)
c3f46da is described below

commit c3f46dae449758692f36e17b01d604824d00f8e8
Author: Taylor King <ta...@gmail.com>
AuthorDate: Wed May 24 16:07:20 2017 -0400

    remove redis.close causing intermittent failures (#211)
---
 scripts/lua/oauth/github.lua             | 1 -
 scripts/lua/oauth/google.lua             | 1 -
 scripts/lua/policies/security/oauth2.lua | 1 -
 3 files changed, 3 deletions(-)

diff --git a/scripts/lua/oauth/github.lua b/scripts/lua/oauth/github.lua
index cb20f44..9d2c208 100644
--- a/scripts/lua/oauth/github.lua
+++ b/scripts/lua/oauth/github.lua
@@ -30,7 +30,6 @@ function validateOAuthToken (red, token)
   local httpc = http.new()
   local key = utils.concatStrings({'oauth:provider:github:', token})
   if redis.exists(red, key) == 1 then
-    redis.close(red)
     return cjson.decode(redis.get(red, key))
   end
  
diff --git a/scripts/lua/oauth/google.lua b/scripts/lua/oauth/google.lua
index ebdd8fd..aba85ba 100644
--- a/scripts/lua/oauth/google.lua
+++ b/scripts/lua/oauth/google.lua
@@ -29,7 +29,6 @@ function validateOAuthToken (red, token)
   local httpc = http.new()
   local key = utils.concatStrings({'oauth:provider:google:', token})
   if redis.exists(red, key) == 1 then
-    redis.close(red)
     return cjson.decode(redis.get(red, key))
   end
 
diff --git a/scripts/lua/policies/security/oauth2.lua b/scripts/lua/policies/security/oauth2.lua
index e847fcb..b799d3b 100644
--- a/scripts/lua/policies/security/oauth2.lua
+++ b/scripts/lua/policies/security/oauth2.lua
@@ -74,7 +74,6 @@ end
 -- @return the json object recieved from exchanging tokens with the provider
   function exchangeWithRedis(red, token, provider)
     -- exchange tokens with the provider
-    print (provider)
     local loaded, provider = pcall(require, utils.concatStrings({'oauth/', provider}))
  
     if not loaded then

-- 
To stop receiving notification emails like this one, please contact
['"commits@openwhisk.apache.org" <co...@openwhisk.apache.org>'].