Skip to content

Commit d93f3ec

Browse files
committed
minor #117 Increase logging (Nyholm)
This PR was squashed before being merged into the master branch. Discussion ---------- Increase logging There is some behaviour I still cannot figure out. I need more information. Commits ------- af60b82 Increase logging
2 parents 77db61a + af60b82 commit d93f3ec

File tree

3 files changed

+9
-1
lines changed

3 files changed

+9
-1
lines changed

composer.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44
"license": "MIT",
55
"require": {
66
"php": ">=7.2",
7+
"ext-json": "*",
78
"composer/package-versions-deprecated": "^1.11",
89
"doctrine/doctrine-bundle": "^2.1",
910
"doctrine/doctrine-migrations-bundle": "^3.0",

config/packages/prod/monolog.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ monolog:
22
handlers:
33
main:
44
type: fingers_crossed
5-
action_level: error
5+
action_level: info
66
handler: nested
77
excluded_http_codes: [404, 405]
88
buffer_size: 50 # How many messages should be saved? Prevent memory leaks

src/Service/GitHubRequestHandler.php

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -78,6 +78,13 @@ public function handle(Request $request)
7878
$responseData['unsupported_action'] = $eventName;
7979
}
8080

81+
$this->logger->info('Done handling request', [
82+
'event' => $eventName,
83+
'response-data' => json_encode($responseData),
84+
'repository' => $repositoryFullName,
85+
'issue-number' => $data['number'] ?? $data['issue']['number'] ?? $data['pull_request']['number'] ?? 'unknown',
86+
]);
87+
8188
return $responseData;
8289
}
8390

0 commit comments

Comments
 (0)