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

JM/KG - (Production Bug): An Icalendar::Values::Date::FormatError occurr… #1024

Merged
merged 1 commit into from Jul 26, 2017

Conversation

jwiel86
Copy link
Contributor

@jwiel86 jwiel86 commented Jul 25, 2017

…ed in service_requests#catalog [#148701413] https://www.pivotaltracker.com/story/show/148701413

Copy link
Contributor

@kayla-glick kayla-glick left a comment

Choose a reason for hiding this comment

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

ArgumentError should probably be included in the first `resc

@@ -426,6 +426,7 @@ def setup_catalog_calendar
active_alert.save
ExceptionNotifier::Notifier.exception_notification(request.env, e).deliver unless request.remote_ip == '128.23.150.107' # this is an ignored IP address, MUSC security causes issues when they pressure test, this should be extracted/configurable
end
rescue ArgumentError
Copy link
Contributor

Choose a reason for hiding this comment

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

ArgumentError should probably be handled by the previous rescue block so that we will receive notifications for it.
rescue Exception, ArgumentError => e

Source: http://mikeferrier.com/2012/05/19/rescuing-multiple-exception-types-in-ruby-and-binding-to-local-variable/

@jwiel86 jwiel86 force-pushed the jm-production-format-error branch from b45fcca to 3aca4d2 Compare July 25, 2017 18:05
@Stuart-Johnson Stuart-Johnson merged commit be93dbb into master Jul 26, 2017
@Stuart-Johnson Stuart-Johnson deleted the jm-production-format-error branch July 26, 2017 15:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants