try: pdf = generate_pdf(data) return pdf except Exception as e: logger.error(f"PDF generation failed: str(e)") return jsonify( "success": False, "error": "code": "PDF_RENDER_ERROR", "message": "Report could not be assembled due to invalid data.", "recoverable": False, "userDataPreserved": True ), 200 # still 200 to avoid download interrupt If PDF fails, offer structured data export.
# DON'T DO THIS output = BytesIO() pdf = canvas.Canvas(output) pdf.drawString(100, 750, "Report") # crash here – user gets zero-byte or partial PDF gracefully broken pdf download
Return a clean error message before ever calling the PDF engine. Never send an error inside a PDF binary. Use structured responses. Success (200 OK with PDF) Content-Type: application/pdf Content-Disposition: attachment; filename="report.pdf" Failure (200 OK with JSON) Even for errors, use 200 OK to avoid browser download interruption, then handle on frontend. try: pdf = generate_pdf(data) return pdf except Exception
function logPDFFailure(error, context) navigator.sendBeacon('/api/log-pdf-error', JSON.stringify( errorCode: error.code, userAction: 'download_pdf', timestamp: Date.now(), page: window.location.pathname, dataSize: context.dataSize )); Use structured responses
Content-Type: application/json
"success": false, "error": "code": "PDF_GEN_FAILED", "message": "Could not render chart data: time series missing.", "recoverable": true, "suggestedAction": "retry", "userDataPreserved": true