docs(pre-auth-hook): clarify .then nesting with comment

#526 (comment)
hoodiehq · Jan 5, 2017 · a020b7a · a020b7a
1 parent 5899eda
commit a020b7a
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/lib/config/store/pre-auth-hook.js b/lib/config/store/pre-auth-hook.js
@@ -40,6 +40,10 @@ function onStorePreAuth (request, reply) {
 
     return server.plugins.account.api.sessions.find(sessionToken)
 
+    // we have to nest the .then callback here because of the `return`
+    // statements above. If we would chain the .then below outside of this
+    // `.then`, then it would be executed after the two `return` statements
+    // above and cause `reply` to be called twice
     .then(function (session) {
       delete request.headers.authorization
       request.headers.cookie = 'AuthSession=' + session.id