From d45e50878c25e17f552c8a8a1e0f2973bd841351 Mon Sep 17 00:00:00 2001 From: nimrodshn Date: Wed, 29 Nov 2017 14:47:18 +0200 Subject: [PATCH] allow full error log adding turncate in different places further truncate minor refactoring --- app/models/authentication.rb | 2 +- app/models/mixins/authentication_mixin.rb | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/app/models/authentication.rb b/app/models/authentication.rb index 096f40a45b5..36604c161b2 100644 --- a/app/models/authentication.rb +++ b/app/models/authentication.rb @@ -92,7 +92,7 @@ def validation_successful def validation_failed(status = :unreachable, message = nil) message ||= ERRORS[status] _log.warn("[#{resource_type}] [#{resource_id}], previously valid on: #{last_valid_on}, previous status: [#{self.status}]") - update_attributes(:status => status.to_s.capitalize, :status_details => message.to_s, :last_invalid_on => Time.now.utc) + update_attributes(:status => status.to_s.capitalize, :status_details => message.to_s.truncate(200), :last_invalid_on => Time.now.utc) raise_event(status, message) end diff --git a/app/models/mixins/authentication_mixin.rb b/app/models/mixins/authentication_mixin.rb index 15a2594ec9a..d68a60b23a9 100644 --- a/app/models/mixins/authentication_mixin.rb +++ b/app/models/mixins/authentication_mixin.rb @@ -334,7 +334,7 @@ def authentication_check(*args) status == :valid ? auth.validation_successful : auth.validation_failed(status, details) end - return status == :valid, details + return status == :valid, details.truncate(20_000) end def default_authentication @@ -360,9 +360,9 @@ def authentication_check_no_validation(type, options) end end - details &&= details.to_s.truncate(200) + details &&= details.to_s - _log.warn("#{header} Validation failed: #{status}, #{details}") unless status == :valid + _log.warn("#{header} Validation failed: #{status}, #{details.truncate(200)}") unless status == :valid return status, details end