Upload 9 files

Dockerfile

@@ -18,8 +18,16 @@ COPY web_api.py .
 COPY converter.py .
 COPY preserve_linebreaks.lua .
 
-# Create necessary directories
-RUN mkdir -p temp/uploads temp/outputs
+# Create a user for the application (better security)
+RUN useradd -m -u 1000 appuser
+
+# Create necessary directories with proper permissions
+RUN mkdir -p /tmp/docx_converter && \
+    chmod 777 /tmp/docx_converter && \
+    chown -R appuser:appuser /app /tmp/docx_converter
+
+# Switch to the application user
+USER appuser
 
 # Expose port
 EXPOSE 7860
@@ -27,6 +35,7 @@ EXPOSE 7860
 # Set environment variables
 ENV PYTHONPATH=/app
 ENV PORT=7860
+ENV TMPDIR=/tmp/docx_converter
 
 # Run the application
 CMD ["python", "app.py"] 

app.py

@@ -1,17 +1,32 @@
 #!/usr/bin/env python3
 """
 DOCX to LaTeX Converter API
-Main entry point for Hugging Face Spaces deployment
+Main entry point for Hugging Face Spaces deployment with improved file handling
 """
 
 import os
 import sys
+import tempfile
 
 # Set up environment for Hugging Face Spaces
 if 'SPACE_ID' in os.environ:
     # Running on Hugging Face Spaces
     PORT = int(os.environ.get('PORT', 7860))
     HOST = '0.0.0.0'
+    
+    # Ensure we have a writable temp directory
+    temp_dir = os.environ.get('TMPDIR', tempfile.gettempdir())
+    print(f"📁 Using temp directory: {temp_dir}")
+    
+    # Test write permissions
+    try:
+        test_file = os.path.join(temp_dir, 'test_write.txt')
+        with open(test_file, 'w') as f:
+            f.write('test')
+        os.unlink(test_file)
+        print("✅ Write permissions confirmed")
+    except Exception as e:
+        print(f"⚠️ Write permission test failed: {e}")
 else:
     # Running locally
     PORT = 5001
@@ -24,10 +39,15 @@ if __name__ == "__main__":
     print(f"🚀 Starting DOCX to LaTeX Converter API")
     print(f"🌐 Server running on http://{HOST}:{PORT}")
     print(f"📖 Health check: http://{HOST}:{PORT}/api/health")
-    print(f"📚 API Documentation: https://huggingface.co/spaces/YOUR_USERNAME/docx-to-latex")
+    print(f"📚 API Documentation: https://huggingface.co/spaces/shayan5422/Docx_to_latex")
     
-    app.run(
-        host=HOST,
-        port=PORT,
-        debug=False  # Disable debug in production
-    ) 
+    # Enable proper error handling for file operations
+    try:
+        app.run(
+            host=HOST,
+            port=PORT,
+            debug=False  # Disable debug in production
+        )
+    except Exception as e:
+        print(f"❌ Failed to start server: {e}")
+        sys.exit(1) 

web_api.py

@@ -6,18 +6,33 @@ import uuid
 from werkzeug.utils import secure_filename
 from converter import convert_docx_to_latex
 import shutil
+import stat
 
 app = Flask(__name__)
 CORS(app)  # Enable CORS for all routes
 
 # Configuration
 app.config['MAX_CONTENT_LENGTH'] = 16 * 1024 * 1024  # 16MB max file size
-UPLOAD_FOLDER = 'temp/uploads'
-OUTPUT_FOLDER = 'temp/outputs'
 
-# Ensure directories exist
-os.makedirs(UPLOAD_FOLDER, exist_ok=True)
-os.makedirs(OUTPUT_FOLDER, exist_ok=True)
+# Use system temp directory for better compatibility with Hugging Face Spaces
+TEMP_BASE_DIR = tempfile.mkdtemp(prefix='docx_converter_')
+UPLOAD_FOLDER = os.path.join(TEMP_BASE_DIR, 'uploads')
+OUTPUT_FOLDER = os.path.join(TEMP_BASE_DIR, 'outputs')
+
+# Ensure directories exist with proper permissions
+def create_temp_dirs():
+    """Create temporary directories with proper permissions"""
+    for directory in [UPLOAD_FOLDER, OUTPUT_FOLDER]:
+        os.makedirs(directory, exist_ok=True)
+        # Set full permissions for the directory
+        try:
+            os.chmod(directory, stat.S_IRWXU | stat.S_IRWXG | stat.S_IRWXO)
+        except OSError:
+            # If chmod fails, continue anyway (some systems don't allow it)
+            pass
+
+# Create directories on startup
+create_temp_dirs()
 
 # Store conversion tasks
 conversion_tasks = {}
@@ -25,7 +40,13 @@ conversion_tasks = {}
 @app.route('/api/health', methods=['GET'])
 def health_check():
     """Health check endpoint"""
-    return jsonify({'status': 'healthy', 'message': 'DOCX to LaTeX API is running'})
+    return jsonify({
+        'status': 'healthy', 
+        'message': 'DOCX to LaTeX API is running',
+        'temp_dir': TEMP_BASE_DIR,
+        'upload_dir': UPLOAD_FOLDER,
+        'output_dir': OUTPUT_FOLDER
+    })
 
 @app.route('/api/upload', methods=['POST'])
 def upload_file():
@@ -44,26 +65,51 @@ def upload_file():
         # Generate unique task ID
         task_id = str(uuid.uuid4())
         
-        # Save uploaded file
+        # Save uploaded file using tempfile for better compatibility
         filename = secure_filename(file.filename)
-        file_path = os.path.join(UPLOAD_FOLDER, f"{task_id}_{filename}")
-        file.save(file_path)
-        
-        # Store task info
-        conversion_tasks[task_id] = {
-            'status': 'uploaded',
-            'original_filename': filename,
-            'file_path': file_path,
-            'output_filename': filename.replace('.docx', '.tex'),
-            'created_at': os.path.getctime(file_path)
-        }
         
-        return jsonify({
-            'task_id': task_id,
-            'filename': filename,
-            'status': 'uploaded',
-            'message': 'File uploaded successfully'
-        })
+        # Create a temporary file instead of using a fixed path
+        temp_fd, temp_path = tempfile.mkstemp(
+            suffix=f'_{filename}', 
+            prefix=f'{task_id}_',
+            dir=UPLOAD_FOLDER
+        )
+        
+        try:
+            # Close the file descriptor and save the file
+            os.close(temp_fd)
+            file.save(temp_path)
+            
+            # Set proper permissions on the file
+            try:
+                os.chmod(temp_path, stat.S_IRUSR | stat.S_IWUSR | stat.S_IRGRP | stat.S_IWGRP)
+            except OSError:
+                # If chmod fails, continue anyway
+                pass
+            
+            # Store task info
+            conversion_tasks[task_id] = {
+                'status': 'uploaded',
+                'original_filename': filename,
+                'file_path': temp_path,
+                'output_filename': filename.replace('.docx', '.tex'),
+                'created_at': os.path.getctime(temp_path)
+            }
+            
+            return jsonify({
+                'task_id': task_id,
+                'filename': filename,
+                'status': 'uploaded',
+                'message': 'File uploaded successfully'
+            })
+            
+        except Exception as e:
+            # Clean up the temp file if something goes wrong
+            try:
+                os.unlink(temp_path)
+            except:
+                pass
+            raise e
         
     except Exception as e:
         return jsonify({'error': f'Upload failed: {str(e)}'}), 500
@@ -95,9 +141,18 @@ def convert_document():
         task['status'] = 'converting'
         task['output_filename'] = output_filename
         
-        # Prepare output paths
-        output_path = os.path.join(OUTPUT_FOLDER, f"{task_id}_{output_filename}")
-        media_path = os.path.join(OUTPUT_FOLDER, f"{task_id}_media")
+        # Prepare output paths using tempfile for better compatibility
+        output_fd, output_path = tempfile.mkstemp(
+            suffix=f'_{output_filename}',
+            prefix=f'{task_id}_',
+            dir=OUTPUT_FOLDER
+        )
+        os.close(output_fd)  # Close file descriptor, we'll write to the path directly
+        
+        media_path = tempfile.mkdtemp(
+            prefix=f'{task_id}_media_',
+            dir=OUTPUT_FOLDER
+        )
         
         # Perform conversion
         success, message = convert_docx_to_latex(
@@ -425,12 +480,26 @@ def cleanup_old_files():
     except Exception as e:
         print(f"Warning: Failed to cleanup old files: {e}")
 
+# Add cleanup on application exit
+import atexit
+
+def cleanup_on_exit():
+    """Clean up temporary directory on exit"""
+    try:
+        shutil.rmtree(TEMP_BASE_DIR)
+        print(f"Cleaned up temporary directory: {TEMP_BASE_DIR}")
+    except OSError:
+        pass
+
+atexit.register(cleanup_on_exit)
+
 if __name__ == '__main__':
     # Cleanup old files on startup
     cleanup_old_files()
     
     # Run the Flask app
     print("Starting DOCX to LaTeX API server...")
+    print(f"Using temporary directory: {TEMP_BASE_DIR}")
     print("API endpoints:")
     print("  POST /api/upload - Upload DOCX file")
     print("  POST /api/convert - Convert to LaTeX")