Merge pull request #1110 from j0k3r/fix/github-scope

Add `user:email` scope
j0k3r · Jul 27, 2023 · 7a1ab5f · 7a1ab5f
2 parents 8f5a1a0 + aa65101
commit 7a1ab5f
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 5 deletions.
diff --git a/config/packages/monolog.yaml b/config/packages/monolog.yaml
@@ -47,15 +47,15 @@ when@prod:
                 excluded_http_codes: [404, 405]
                 buffer_size: 50 # How many messages should be saved? Prevent memory leaks
             nested:
-                type: stream
-                path: php://stderr
+                type: rotating_file
+                path: "%kernel.logs_dir%/%kernel.environment%.log"
                 level: debug
-                formatter: monolog.formatter.json
+                max_files: 10
             console:
                 type: console
                 process_psr_3_messages: false
                 channels: ["!event", "!doctrine"]
             deprecation:
                 type: stream
                 channels: [deprecation]
-                path: php://stderr
+                path: "%kernel.logs_dir%/deprecation.log"
diff --git a/config/packages/sentry.yaml b/config/packages/sentry.yaml
@@ -13,6 +13,7 @@ when@prod:
         handlers:
             sentry:
                 type: service
+                id: Sentry\Monolog\Handler
                 level: !php/const Monolog\Logger::ERROR
                 hub_id: Sentry\State\HubInterface
 

diff --git a/src/Controller/DefaultController.php b/src/Controller/DefaultController.php
@@ -134,7 +134,7 @@ public function connectAction(ClientRegistry $oauth): RedirectResponse
 
         return $oauth
             ->getClient('github')
-            ->redirect([], []);
+            ->redirect(['user:email'], []);
     }
 
     /**