Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reducing log noise and other current issues #152

Merged
merged 29 commits into from
Dec 19, 2024

Conversation

jonrandahl
Copy link
Contributor

@jonrandahl jonrandahl commented Dec 18, 2024

While containing the primary improvements for the #149 ticket itself, this also contains updating the error template path to use Rails.public_path as well as contain the html extension to ensure the correct template is rendered alongside a myriad of tweaks to ensure safe navigation to access variable value, assigning to a default value to prevent errors in the application. or simply improving the build process via the makefile.

This also includes multitude of Rubocop fixes or disabling of rules where applicable.

- ensures that logging always happens with the appropriate severity depending on the exception status
- reduces the types of errors that can trigger a an error metric and therefore a notification in slack (e.g. :bad_request is set at WARN level and therefore not alerted on, whereas :internal_server_error is set at ERROR level and does trigger alerts
also align the operands of a condition in an `unless` statement spanning multiple lines
prevents unknown method errors if nil
- `Metrics/MethodLength`
- `Metrics/CyclomaticComplexity`
now triggers `api_service_exception` metric on error
update to use `Rails.public_path` alongside ending the path with `.html` to be sure
Emits logs with specific message and error to help pinpoint
also restores block to resolve unexpected token tRPAREN error as well as sets Time.now.year to Time.current.year to ensure zone is incorporated in calculations
as otherwise ruby throws `unexpected token tRPAREN` error
@jonrandahl jonrandahl marked this pull request as ready for review December 18, 2024 13:35
@jonrandahl jonrandahl requested review from bogdanadrianmarc, DanielCouzens and joescottdave and removed request for DanielCouzens December 18, 2024 13:35
@jonrandahl jonrandahl self-assigned this Dec 18, 2024
Copy link
Contributor

@joescottdave joescottdave left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@bogdanadrianmarc bogdanadrianmarc merged commit 20b67c9 into dev Dec 19, 2024
2 checks passed
@bogdanadrianmarc bogdanadrianmarc deleted the spike/improve-metrics-logging branch December 19, 2024 15:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants