Update email_sender_api.py

email_sender_api.py

@@ -1,15 +1,35 @@
-# ... (keep existing imports and Flask app setup, SMTP config variables) ...
+from flask import Flask, request, jsonify
+from flask_cors import CORS
+import smtplib
+from email.mime.multipart import MIMEMultipart
+from email.mime.text import MIMEText
+# from email.mime.base import MIMEBase # Not used in simplified version
+# from email import encoders             # Not used in simplified version
+import base64
+import os
+import socket 
+import traceback
 
-SMTP_CONNECTION_TIMEOUT = 30  # Increased slightly for good measure
-SMTP_COMMAND_TIMEOUT = 30     # Increased slightly for good measure
+# <<<--- FLASK APP INITIALIZATION MUST BE HERE (TOP LEVEL) --->>>
+app = Flask(__name__)
+CORS(app)
+# <<<--------------------------------------------------------->>>
 
-@app.route('/send-report-via-email', methods=['POST']) # Keep the same endpoint
+# Environment variables will be set as Secrets in Hugging Face Space settings
+SMTP_SERVER_HOST = os.environ.get("SMTP_SERVER_HOST")
+SMTP_SERVER_PORT = os.environ.get("SMTP_SERVER_PORT")
+SMTP_SENDER_EMAIL = os.environ.get("SMTP_SENDER_EMAIL")
+SMTP_SENDER_PASSWORD = os.environ.get("SMTP_SENDER_PASSWORD")
+
+SMTP_CONNECTION_TIMEOUT = 30
+SMTP_COMMAND_TIMEOUT = 30
+
+@app.route('/send-report-via-email', methods=['POST']) # This decorator now correctly sees 'app'
 def handle_send_email():
     request_id = base64.b64encode(os.urandom(6)).decode('utf-8')
     print(f"EMAIL_SENDER_API (HF) [{request_id}]: SIMPLIFIED TEST - Received request at /send-report-via-email")
 
     if not all([SMTP_SERVER_HOST, SMTP_SERVER_PORT, SMTP_SENDER_EMAIL, SMTP_SENDER_PASSWORD]):
-        # ... (same error handling for missing SMTP config as before) ...
         error_msg = "EMAIL_SENDER_API (HF) [{request_id}]: ERROR - SMTP environment variables (Secrets) not fully set."
         print(error_msg)
         return jsonify({"status": "error", "message": "Email server configuration incomplete on the API."}), 500
@@ -17,29 +37,28 @@ def handle_send_email():
     try:
         SMTP_PORT_INT = int(SMTP_SERVER_PORT)
     except ValueError:
-        # ... (same error handling for invalid port) ...
         error_msg = f"EMAIL_SENDER_API (HF) [{request_id}]: ERROR - Invalid SMTP_SERVER_PORT: '{SMTP_SERVER_PORT}'."
         print(error_msg)
         return jsonify({"status": "error", "message": "Invalid email server port configured on the API."}), 500
 
     data = request.json
     if not data:
-        # ... (same error handling for no data) ...
         print(f"EMAIL_SENDER_API (HF) [{request_id}]: SIMPLIFIED TEST - No JSON data received.")
         return jsonify({"status": "error", "message": "No data received by email API."}), 400
 
     recipient_email = data.get('recipient_email')
-    # We are not using pdf_base64_data or pdf_filename in this simplified test
+    # pdf_base64_data = data.get('pdf_base64_data') # Not using these in simplified test
+    # pdf_filename = data.get('pdf_filename')       # Not using these in simplified test
 
     print(f"EMAIL_SENDER_API (HF) [{request_id}]: SIMPLIFIED TEST - Attempting to send to: {recipient_email}")
 
-    if not recipient_email: # Only recipient email is strictly needed for this test
+    if not recipient_email: 
         print(f"EMAIL_SENDER_API (HF) [{request_id}]: SIMPLIFIED TEST - Missing recipient_email in JSON payload.")
         return jsonify({"status": "error", "message": "Missing required data: recipient_email."}), 400
 
     try:
         # Create a simple plain text message
-        simple_msg = MIMEMultipart() # Still use MIMEMultipart for proper headers
+        simple_msg = MIMEMultipart() 
         simple_msg['From'] = SMTP_SENDER_EMAIL
         simple_msg['To'] = recipient_email
         simple_msg['Subject'] = "EPIC-AMP - Simple Connectivity Test"
@@ -86,12 +105,12 @@ def handle_send_email():
         print(f"EMAIL_SENDER_API (HF) [{request_id}]: {success_msg}")
         return jsonify({"status": "success", "message": success_msg}), 200
 
-    # Keep all the specific exception handling blocks as they were
+    # Keep all the specific exception handling blocks
     except smtplib.SMTPConnectError as e:
         error_msg = f"EMAIL_SENDER_API (HF) [{request_id}]: SIMPLIFIED TEST - SMTP Connect Error. {e}"
         print(error_msg); traceback.print_exc()
         return jsonify({"status": "error", "message": f"SIMPLIFIED TEST - Could not connect: {e.strerror if hasattr(e, 'strerror') else e}"}), 503
-    except smtplib.SMTPHeloError as e: # Keep other specific exceptions
+    except smtplib.SMTPHeloError as e: 
         error_msg = f"EMAIL_SENDER_API (HF) [{request_id}]: SIMPLIFIED TEST - SMTP Helo/Ehlo Error. {e}"
         print(error_msg); traceback.print_exc()
         return jsonify({"status": "error", "message": "SIMPLIFIED TEST - Server HELO/EHLO error."}), 502
@@ -99,7 +118,6 @@ def handle_send_email():
         error_msg = f"EMAIL_SENDER_API (HF) [{request_id}]: SIMPLIFIED TEST - SMTP Authentication Error. {e}"
         print(error_msg); traceback.print_exc()
         return jsonify({"status": "error", "message": "SIMPLIFIED TEST - Email server authentication failed."}), 500
-    # ... include ALL other smtplib exceptions and socket exceptions from the previous version of email_sender_api.py ...
     except socket.timeout as e:
         error_msg = f"EMAIL_SENDER_API (HF) [{request_id}]: SIMPLIFIED TEST - Socket Timeout. {e}"
         print(error_msg); traceback.print_exc()
@@ -108,8 +126,8 @@ def handle_send_email():
         error_msg = f"EMAIL_SENDER_API (HF) [{request_id}]: SIMPLIFIED TEST - Address-related error. {e}"
         print(error_msg); traceback.print_exc()
         return jsonify({"status": "error", "message": "SIMPLIFIED TEST - Could not resolve hostname."}), 503
-    except OSError as e: # Specifically catch OSError for "Network is unreachable"
-        if e.errno == 101: # Network is unreachable
+    except OSError as e: 
+        if e.errno == 101: 
             error_msg = f"EMAIL_SENDER_API (HF) [{request_id}]: SIMPLIFIED TEST - OSError [Errno 101] Network is unreachable. {e}"
             print(error_msg); traceback.print_exc()
             return jsonify({"status": "error", "message": "SIMPLIFIED TEST - Network is unreachable from API server."}), 503
@@ -123,6 +141,13 @@ def handle_send_email():
         traceback.print_exc()
         return jsonify({"status": "error", "message": f"SIMPLIFIED TEST - Unexpected API error: {type(e).__name__}"}), 500
 
-# The if __name__ == '__main__': block for local testing can remain commented out or as is
+# The if __name__ == '__main__': block for local testing:
+# On Hugging Face Spaces with Gunicorn, Gunicorn directly imports and runs the 'app' object,
+# so this block isn't strictly executed by Gunicorn.
+# It's useful if you want to run this script directly with `python email_sender_api.py` for local testing.
 # if __name__ == '__main__':
-#     # ... local testing startup ...
+#     # For local testing, set your ENV VARS in this terminal session before running
+#     api_port = int(os.environ.get("PORT", 5002)) # Default to 5002 for local test
+#     print(f"Starting Email Sender API Service locally on port {api_port}...")
+#     # ... (rest of the local startup print messages and app.run) ...
+#     app.run(host='0.0.0.0', port=api_port, debug=True)