From c52d985d45453e7376b106dba4c9de5a99fa07c4 Mon Sep 17 00:00:00 2001
From: Johannes Zellner <johannes@cloudron.io>
Date: Wed, 14 Oct 2015 16:16:37 +0200
Subject: [PATCH] Properly skip decision dialog

---
 src/routes/oauth2.js | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/src/routes/oauth2.js b/src/routes/oauth2.js
index 16cce0fbd..2d4f7c2e7 100644
--- a/src/routes/oauth2.js
+++ b/src/routes/oauth2.js
@@ -375,7 +375,7 @@ var authorization = [
 
         session.ensureLoggedIn('/api/v1/session/login?returnTo=' + req.query.redirect_uri)(req, res, next);
     },
-    gServer.authorization(function (clientId, redirectURI, callback) {
+    gServer.authorization({}, function (clientId, redirectURI, callback) {
         debug('authorization: client %s with callback to %s.', clientId, redirectURI);
 
         clientdb.get(clientId, function (error, client) {
@@ -388,11 +388,9 @@ var authorization = [
 
             callback(null, client, '/api/v1/session/callback?redirectURI=' + url.resolve(redirectOrigin, redirectPath));
         });
-    }),
-    // we do not have a decision dialog, no need to load the transaction
-    gServer.decision({ loadTransaction: false }, function (req, done) {
-        debug('decision: with scope', req.oauth2.client.scope);
-        return done(null, { scope: req.oauth2.client.scope });
+    }, function (client, user, done) {
+        // This allows us to skip decision dialog
+        return done (null, true);
     })
 ];