-
Notifications
You must be signed in to change notification settings - Fork 0
Conversation
df236aa
to
74d929c
Compare
74d929c
to
6568f80
Compare
@@ -0,0 +1,19 @@ | |||
<?php | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe you should add the copyright notice to this file.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is still WIP, this will be fixed in: #27, after the legal team is consulted.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
public function __invoke(Throwable $exception): Response | ||
{ | ||
return $this->render('default/error.html.twig', [ | ||
'message' => $exception->getMessage(), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this desirable? Showing the exception message without being certain what text may be on there? We may run a risk at disclosing sensitive data here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changed it to use static translations
6568f80
to
a2919d3
Compare
https://www.pivotaltracker.com/story/show/176891048