Slide 1

Slide 1 text

How to get to zero unhandled exceptions in production Radoslav Stankov 07/09/2019

Slide 2

Slide 2 text

Radoslav Stankov @rstankov blog.rstankov.com github.com/rstankov
 twitter.com/rstankov

Slide 3

Slide 3 text

No content

Slide 4

Slide 4 text

No content

Slide 5

Slide 5 text

No content

Slide 6

Slide 6 text

No content

Slide 7

Slide 7 text

No content

Slide 8

Slide 8 text

No content

Slide 9

Slide 9 text


 ! Product Hunt Engineering Team " 


Slide 10

Slide 10 text

No content

Slide 11

Slide 11 text

No content

Slide 12

Slide 12 text

No content

Slide 13

Slide 13 text

No content

Slide 14

Slide 14 text

No content

Slide 15

Slide 15 text

No content

Slide 16

Slide 16 text

No content

Slide 17

Slide 17 text

Happy Friday # $ Fix bugs % Goodies features & Pay technical dept ' Catchup on projects ( Fix exceptions

Slide 18

Slide 18 text

Happy Friday # $ Fix bugs % Goodies features & Pay technical dept ' Catchup on projects ( Fix exceptions

Slide 19

Slide 19 text

No content

Slide 20

Slide 20 text

“Have process around exceptions.” 
 ) Tip 


Slide 21

Slide 21 text

Back to basics

Slide 22

Slide 22 text


 http://exceptionalruby.com/


Slide 23

Slide 23 text

def perform do_something rescue end

Slide 24

Slide 24 text

No content

Slide 25

Slide 25 text

def perform do_something rescue SpecificError end

Slide 26

Slide 26 text

def perform do_something rescue SpecificError # NOTE(rstankov): Reason to return nil nil end Your name, not mine *

Slide 27

Slide 27 text

def perform do_something rescue Timeout::Error # NOTE(rstankov): WiFi Sucks nil end

Slide 28

Slide 28 text

“Be explicit around the exceptions. Handle specific errors and have explanations of why they happen.” 
 ) Tip 


Slide 29

Slide 29 text

Monitoring

Slide 30

Slide 30 text

No content

Slide 31

Slide 31 text

No content

Slide 32

Slide 32 text

No content

Slide 33

Slide 33 text

No content

Slide 34

Slide 34 text

Raven.configure do |config| # Note(rstankov): Exclude unactionable errors config.excluded_exceptions = [ 'Rack::Timeout::RequestExpiryError', 'Rack::Timeout::RequestTimeoutException', 'ActionController::RoutingError', 'ActionController::InvalidAuthenticityToken', 'ActionDispatch::ParamsParser::ParseError', 'Sidekiq::Shutdown', ] end

Slide 35

Slide 35 text

No content

Slide 36

Slide 36 text

ArgumentError: invalid byte sequence in UTF-8 (

Slide 37

Slide 37 text

No content

Slide 38

Slide 38 text

No content

Slide 39

Slide 39 text

# NOTE(rstankov): Fix invalid byte sequence in UTF-8. More info: # - https://robots.thoughtbot.com/fight-back-utf-8-invalid-byte-sequences module Handle::InvalidByteSequence extend self def call(string) return if string.nil? string.encode('UTF-8', 'binary', invalid: :replace, undef: :replace, replace: '' ) end end

Slide 40

Slide 40 text

“Reduce noise. See only exceptional errors in your tracker.” 
 ) Tip 


Slide 41

Slide 41 text

No content

Slide 42

Slide 42 text

No content

Slide 43

Slide 43 text

account.subscription.status

Slide 44

Slide 44 text

account.subscription&.status

Slide 45

Slide 45 text

✅ Check for other accounts without a subscription ✅ Find out why some accounts don't have a subscription ✅ Fix the root problem ✅ Add missing subscriptions to accounts , Steps to fix -

Slide 46

Slide 46 text

No content

Slide 47

Slide 47 text

No content

Slide 48

Slide 48 text

if account.ship_subscription.blank? Raven.capture_message "Missing ship subscription", extra: { account_id: account.id } return :no_subscription end

Slide 49

Slide 49 text

“Don't hide exceptions. Fix root causes.” 
 ) Tip 


Slide 50

Slide 50 text


 https://graphql.org/


Slide 51

Slide 51 text

No content

Slide 52

Slide 52 text

No content

Slide 53

Slide 53 text

Which query causes this issue? .

Slide 54

Slide 54 text

class Frontend::GraphqlController < Frontend::BaseController before_action :ensure_query def index render json: Graph::Schema.execute(query, variables: variables, context: context) rescue => e handle_error e end private # ... def handle_error(error) if Rails.env.development? logger.error error.message logger.error error.backtrace.join("\n") render json: { error: { message: error.message, backtrace: error.backtrace } }, status: 500 elsif Rails.env.test? p error.message p error.backtrace render json: { error: { message: error.message, backtrace: error.backtrace } }, status: 500 else Raven.capture_exception(e, extra: { query: query }) render json: { error: { message: 'SERVER_ERROR' }, data: {} }, status: 500 end end end

Slide 55

Slide 55 text

class Frontend::GraphqlController < Frontend::BaseController before_action :ensure_query def index render json: Graph::Schema.execute(query, variables: variables, context: context) rescue => e handle_error e end private # ... def handle_error(error) if Rails.env.development? logger.error error.message logger.error error.backtrace.join("\n") render json: { error: { message: error.message, backtrace: error.backtrace } }, status: 500 elsif Rails.env.test? p error.message p error.backtrace render json: { error: { message: error.message, backtrace: error.backtrace } }, status: 500 else Raven.capture_exception(e, extra: { query: query }) render json: { error: { message: 'SERVER_ERROR' }, data: {} }, status: 500 end end end

Slide 56

Slide 56 text

class Frontend::GraphqlController < Frontend::BaseController before_action :ensure_query def index render json: Graph::Schema.execute(query, variables: variables, context: context) rescue => e handle_error e end private # ... def handle_error(error) if Rails.env.development? logger.error error.message logger.error error.backtrace.join("\n") render json: { error: { message: error.message, backtrace: error.backtrace } }, status: 500 elsif Rails.env.test? p error.message p error.backtrace render json: { error: { message: error.message, backtrace: error.backtrace } }, status: 500 else Raven.capture_exception(e, extra: { query: query }) render json: { error: { message: 'SERVER_ERROR' }, data: {} }, status: 500 end end end

Slide 57

Slide 57 text

No content

Slide 58

Slide 58 text

“Invest in your monitoring and exception traceability. When you have a hard time racing an exception. Ask yourself - what more information I need? . Then add it.” 
 ) Tip 


Slide 59

Slide 59 text


 https://sidekiq.org/


Slide 60

Slide 60 text

No content

Slide 61

Slide 61 text

Use action Achievements::Job Achievement
 (unique)

Slide 62

Slide 62 text

class Achievements::Job < ApplicationJob def perform(achievement, user) Achievements::Create.call achievement, user end end

Slide 63

Slide 63 text

Use action Achievements::Job Achievement
 (unique) Use action Achievements::Job Achievement
 (unique)

Slide 64

Slide 64 text

Use action Achievements::Job Achievement
 (unique) Use action Achievements::Job Achievement
 (unique) (

Slide 65

Slide 65 text

Use action Achievements::Job Achievement
 (unique) Use action Achievements::Job PG::UniqueViolation (

Slide 66

Slide 66 text

class Achievements::Job < ApplicationJob def perform(achievement, user) Achievements::Create.call achievement, user end end

Slide 67

Slide 67 text

class Achievements::Job < ApplicationJob def perform(achievement, user) Handle::RaceCondition.call do Achievements::Create.call achievement, user end end end

Slide 68

Slide 68 text

module Handle::RaceCondition extend self UNIQUE_ACTIVE_RECORD_ERROR = 'has already been taken'.freeze def call retries ||= 2 yield rescue ActiveRecord::RecordNotUnique, PG::UniqueViolation retries -= 1 raise unless retries.nonzero? retry rescue ActiveRecord::RecordInvalid => e raise unless e.message.include? UNIQUE_ACTIVE_RECORD_ERROR retries -= 1 raise unless retries.nonzero? retry end end

Slide 69

Slide 69 text

class Notifications::DeliverJob < ApplicationJob queue_as :notifications def perform(event) Notifications::Deliver.call(event) end end

Slide 70

Slide 70 text

Errno::ECONNRESET (

Slide 71

Slide 71 text

No content

Slide 72

Slide 72 text

class Notifications::DeliverJob < ApplicationJob queue_as :notifications def perform(event) Notifications::Deliver.call(event) end end

Slide 73

Slide 73 text

class Notifications::DeliverJob < ApplicationJob queue_as :notifications def perform(event) Notifications::Deliver.call(event)
 rescue Errno::ECONNRESET
 retry_job end end

Slide 74

Slide 74 text

class Notifications::DeliverJob < ApplicationJob queue_as :notifications def perform(event) Notifications::Deliver.call(event)
 rescue Errno::ECONNRESET, EOFError
 retry_job end end

Slide 75

Slide 75 text

class Notifications::DeliverJob < ApplicationJob queue_as :notifications def perform(event) Notifications::Deliver.call(event)
 rescue Errno::ECONNRESET, EOFError, Timeout::Error
 retry_job end end

Slide 76

Slide 76 text

class Notifications::DeliverJob < ApplicationJob queue_as :notifications def perform(event) Notifications::Deliver.call(event)
 rescue Errno::ECONNRESET, EOFError, Timeout::Error, ... /
 retry_job end end

Slide 77

Slide 77 text

module Handle::NetworkErrors extend self ERRORS = [ EOFError, Errno::ECONNRESET, Errno::EINVAL, Net::HTTPBadResponse, Net::HTTPHeaderSyntaxError, Net::ProtocolError, Timeout::Error, # ... ] def ===(error) ERRORS.any? { |error_class| error_class === error } end end

Slide 78

Slide 78 text

class Notifications::Deliver < ApplicationJob queue_as :notifications def perform(event) Notifications::Deliver.call(event)
 rescue Handle::NetworkErrors retry_job end end

Slide 79

Slide 79 text

class Notifications::Deliver::Job < ApplicationJob retry_on(*::Handle::NetworkErrors::ERRORS, wait: 2.minutes, attempts: 5) queue_as :notifications def perform(event) Notifications::Deliver.call(event) end end

Slide 80

Slide 80 text

class Notifications::Deliver::Job < ApplicationJob include Handle::Job::NetworkErrors queue_as :notifications def perform(event) Notifications::Deliver.call(event) end end

Slide 81

Slide 81 text

module Handle::Job::NetworkErrors def self.included(job) job.retry_on(*::Handle::NetworkErrors::ERRORS, wait: 2.minutes, attempts: 5) end end

Slide 82

Slide 82 text

No content

Slide 83

Slide 83 text

0

Slide 84

Slide 84 text

module Handle::NetworkErrors extend self ERRORS = [ # ... Faraday::ConnectionFailed, Faraday::TimeoutError, RestClient::BadGateway, RestClient::BadRequest, # ... ] def ===(error) ERRORS.any? { |error_class| error_class === error } end end

Slide 85

Slide 85 text

“Have tooling around handling common exceptions. Make it a no-brainer to process everyday errors.” 
 ) Tip 


Slide 86

Slide 86 text

No content

Slide 87

Slide 87 text

No content

Slide 88

Slide 88 text

Recap

Slide 89

Slide 89 text

No content

Slide 90

Slide 90 text

) Have process around exceptions.
 ) Be explicit around the exceptions
 ) Reduce noise
 ) Don't hide exceptions
 ) Invest in your monitoring
 ) Have tooling around handling common exceptions 
 1 Recap 


Slide 91

Slide 91 text

No content

Slide 92

Slide 92 text

No content

Slide 93

Slide 93 text

Thanks 2

Slide 94

Slide 94 text

https://speakerdeck.com/rstankov