Merge pull request #26 from qwertyllo/patch-1

Fix unmanaged exception when Azure response contents are missing

Author: danieltjewett

src/Azure.php

@@ -61,6 +61,10 @@ public function handle($request, Closure $next)
             $this->fail($request, $e);
         }
 
+        if (empty($contents->access_token) || empty($contents->refresh_token)) {
+            $this->fail($request, \Exception('Missing tokens in response contents'));
+        }
+        
         $request->session()->put('_rootinc_azure_access_token', $contents->access_token);
         $request->session()->put('_rootinc_azure_refresh_token', $contents->refresh_token);
 
@@ -177,10 +181,10 @@ protected function success(Request $request, $access_token, $refresh_token, $pro
      * Handler that is called when a failed handshake has taken place
      *
      * @param \Illuminate\Http\Request $request
-     * @param \GuzzleHttp\Exception\RequestException $e
+     * @param \Exception $e
      * @return string
      */
-    protected function fail(Request $request, RequestException $e)
+    protected function fail(Request $request, \Exception $e)
     {
         // Added by smitthhyy 18Dec2019 - Return 403 if user authenticates in AD but is not assigned to this application
         if ($request->isMethod('get')) {