From d96b4bbb46317a6d37088732554ce49bf4d37a25 Mon Sep 17 00:00:00 2001 From: Roman Rizzi Date: Thu, 26 Jan 2023 14:57:40 -0300 Subject: [PATCH] DEV: Include upload_id on image optimization errors to have more context (#19715) --- app/models/optimized_image.rb | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/app/models/optimized_image.rb b/app/models/optimized_image.rb index 954686ca086..e9721bf9e84 100644 --- a/app/models/optimized_image.rb +++ b/app/models/optimized_image.rb @@ -85,6 +85,7 @@ class OptimizedImage < ActiveRecord::Base target_quality = upload.target_image_quality(original_path, SiteSetting.image_preview_jpg_quality) opts = opts.merge(quality: target_quality) if target_quality + opts = opts.merge(upload_id: upload.id) if upload.extension == "svg" FileUtils.cp(original_path, temp_path) @@ -343,7 +344,7 @@ class OptimizedImage < ActiveRecord::Base error << " unknown reason" end - Discourse.warn(error, location: to, error_message: e.message, instructions: instructions) + Discourse.warn(error, upload_id: opts[:upload_id], location: to, error_message: e.message, instructions: instructions) false end end