fixing the uploading image error

Dockerfile

@@ -3,7 +3,8 @@ FROM python:3.11-slim
 ENV PYTHONDONTWRITEBYTECODE=1 \
     PYTHONUNBUFFERED=1 \
     PIP_NO_CACHE_DIR=1 \
-    PORT=7860
+    PORT=7860 \
+    MEDIA_ROOT=/data/media
 
 # Install required system packages for OpenCV
 RUN apt-get update && apt-get install -y --no-install-recommends \
@@ -33,9 +34,10 @@ RUN pip install --upgrade pip && \
 COPY --chown=user . .
 
 # Create necessary directories with proper user permissions
-RUN mkdir -p $HOME/app/staticfiles $HOME/app/media
+RUN mkdir -p $HOME/app/staticfiles $HOME/app/media /data/media
 
 EXPOSE 7860
 
 # Run migrations, collect static files, and start Gunicorn (all as user 1000)
-CMD ["sh", "-c", "python manage.py migrate && python manage.py collectstatic --noinput && python manage.py reset_catalog && python manage.py create_admin && gunicorn config.wsgi:application --bind 0.0.0.0:${PORT:-7860} --workers 2 --timeout 120"]
+# Catalog reset is optional to avoid deleting products/images on each restart.
+CMD ["sh", "-c", "python manage.py migrate && python manage.py collectstatic --noinput && if [ \"${RUN_RESET_CATALOG:-0}\" = \"1\" ]; then python manage.py reset_catalog; fi && python manage.py create_admin && gunicorn config.wsgi:application --bind 0.0.0.0:${PORT:-7860} --workers 2 --timeout 120"]

config/urls.py

@@ -15,10 +15,11 @@ Including another URLconf
     2. Add a URL to urlpatterns:  path('blog/', include('blog.urls'))
 """
 from django.contrib import admin
-from django.urls import path, include
+from django.urls import path, include, re_path
 from django.conf import settings
 from django.conf.urls.static import static
 from django.conf.urls.i18n import i18n_patterns
+from django.views.static import serve
 
 urlpatterns = [
     path('admin/', admin.site.urls),
@@ -32,7 +33,9 @@ urlpatterns += i18n_patterns(
 )
 
 # WhiteNoise serves STATIC_URL in production.
-# Keep media URL mapping available for container deployments where legacy media
-# files may still be referenced.
+# Media files need explicit routing under Gunicorn deployments.
 urlpatterns += static(settings.MEDIA_URL, document_root=settings.MEDIA_ROOT)
+urlpatterns += [
+    re_path(r"^media/(?P<path>.*)$", serve, {"document_root": settings.MEDIA_ROOT}),
+]
 

fitting_system/storage.py

@@ -6,8 +6,8 @@ from django.core.files.storage import FileSystemStorage
 
 class ProductImageStorage(FileSystemStorage):
     """
-    Save new product uploads into static/images/products while keeping legacy
-    media/products files readable.
+    Save product uploads into MEDIA_ROOT and keep older static/media paths
+    readable for backward compatibility.
     """
 
     def __init__(self, *args, **kwargs):
@@ -15,25 +15,37 @@ class ProductImageStorage(FileSystemStorage):
         media_base_url = settings.MEDIA_URL if settings.MEDIA_URL.endswith("/") else f"{settings.MEDIA_URL}/"
 
         super().__init__(
+            location=settings.MEDIA_ROOT,
+            base_url=media_base_url,
+        )
+        self.legacy_storage = FileSystemStorage(
             location=Path(settings.BASE_DIR) / "static",
             base_url=static_base_url,
         )
-        self.legacy_storage = FileSystemStorage(
+        self.legacy_media_storage = FileSystemStorage(
             location=Path(settings.BASE_DIR) / "media",
             base_url=media_base_url,
         )
 
     def exists(self, name):
-        return super().exists(name) or self.legacy_storage.exists(name)
+        return (
+            super().exists(name)
+            or self.legacy_storage.exists(name)
+            or self.legacy_media_storage.exists(name)
+        )
 
     def open(self, name, mode="rb"):
         if super().exists(name):
             return super().open(name, mode)
-        return self.legacy_storage.open(name, mode)
+        if self.legacy_storage.exists(name):
+            return self.legacy_storage.open(name, mode)
+        return self.legacy_media_storage.open(name, mode)
 
     def url(self, name):
         if super().exists(name):
             return super().url(name)
         if self.legacy_storage.exists(name):
             return self.legacy_storage.url(name)
+        if self.legacy_media_storage.exists(name):
+            return self.legacy_media_storage.url(name)
         return super().url(name)