discourse/discourse
1
0
Fork 0
mirror of https://github.com/discourse/discourse.git synced 2025-03-09 14:34:35 +00:00
Code Issues Packages Projects Releases Wiki Activity

DEV: Include upload_id on image optimization errors to have more context (#19715)

Browse Source
This commit is contained in:
Roman Rizzi 2023-01-26 14:57:40 -03:00 committed by GitHub
parent 13e9f5054c
commit d96b4bbb46
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
app/models/optimized_image.rb
View File

@ -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