Merge pull request #8 from fedwiki/fix

Fix empty list of providers when adding alternative identity. Yes, this fix previously deployed into production with vi. DevOps indeed
fedwiki · Nov 25, 2016 · 9f9a3c7 · 9f9a3c7
2 parents 84d6f25 + 30b5f99
commit 9f9a3c7
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/server/social.coffee b/server/social.coffee
@@ -403,7 +403,8 @@ module.exports = exports = (log, loga, argv) ->
     app.get '/auth/addAuthDialog', (req, res) ->
       # only makes sense to add alternative authentication scheme if
       # this the user is authenticated
-      if getUser(req)
+      user = getUser(req)
+      if user
         referer = req.headers.referer
 
         currentSchemes = _.keys(user)