do not catch exception
This commit is contained in:
@@ -25,11 +25,9 @@ class Captain::Documents::PdfExtractionJob < ApplicationJob
|
||||
end
|
||||
|
||||
def process_extraction_result(document, result)
|
||||
if result[:success] && result[:content].present?
|
||||
process_pdf_content_chunks(document, result[:content])
|
||||
else
|
||||
handle_pdf_extraction_failure(document, result)
|
||||
end
|
||||
return unless result[:success] && result[:content].present?
|
||||
|
||||
process_pdf_content_chunks(document, result[:content])
|
||||
end
|
||||
|
||||
def process_pdf_content_chunks(document, content_chunks)
|
||||
@@ -74,9 +72,4 @@ class Captain::Documents::PdfExtractionJob < ApplicationJob
|
||||
Rails.logger.error "PDF extraction failed for document #{document.id}: #{error_message}"
|
||||
document.update(status: 'available')
|
||||
end
|
||||
|
||||
def handle_pdf_extraction_error(document, error)
|
||||
Rails.logger.error "PDF extraction failed for document #{document.id}: #{error.message}"
|
||||
document.update(status: 'available')
|
||||
end
|
||||
end
|
||||
@@ -41,9 +41,6 @@ class Captain::Tools::PdfExtractionService
|
||||
log_extraction_success(chunked_content)
|
||||
|
||||
success_response(chunked_content)
|
||||
rescue StandardError => e
|
||||
Rails.logger.error "PDF extraction failed: #{e.message}"
|
||||
failure_response([e.message])
|
||||
end
|
||||
|
||||
def log_extraction_success(chunked_content)
|
||||
|
||||
Reference in New Issue
Block a user