fix: allow 50 MP smartphone photos in bulk image upload

NetBox sets Pillow's MAX_IMAGE_PIXELS to 25 MP, so Pillow rejects images
above 50 MP as decompression bombs. Raise the limit to 100 MP while the
bulk upload is processed and surface the underlying Pillow error.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This commit is contained in:
2026-10-02 10:14:13 +02:00
co-authored by Claude Opus 5.5
parent c7b65f123b
commit a9fd36f31c
4 changed files with 83 additions and 26 deletions
+6
View File
@@ -541,6 +541,12 @@ Anzeigename bleibt der jeweilige ursprüngliche Dateiname erhalten. Benötigt
werden weiterhin die NetBox-Berechtigung zum Hinzufügen von Bildanhängen und
eine Leseberechtigung für das Zielobjekt.
NetBox begrenzt Pillow global auf 25 Megapixel und lehnt Bilder ab 50
Megapixeln ab. Damit aktuelle 50-MP-Smartphone-Fotos (z. B. Google Pixel,
8160×6144) hochgeladen werden können, hebt der Mehrfach-Upload dieses Limit
während der Verarbeitung auf 100 Megapixel an. Wird ein Bild abgelehnt, zeigt
das Formular zusätzlich die genaue Fehlermeldung von Pillow an.
### Module mehrfach einbauen
Unter **Plugins > NetBox Utilities > Module mehrfach einbauen** kann ein
+1 -1
View File
@@ -1,6 +1,6 @@
from netbox.plugins import PluginConfig, get_plugin_config
__version__ = "0.14.0"
__version__ = "0.14.1"
class NetBoxUtilitiesConfig(PluginConfig):
+51
View File
@@ -1,3 +1,6 @@
from contextlib import contextmanager
from io import BytesIO
from dcim.models import Device, Module, Rack
from django.contrib import messages
from django.contrib.auth.mixins import LoginRequiredMixin, UserPassesTestMixin
@@ -50,6 +53,24 @@ def _safe_return_url(request, default_name="home"):
return reverse(default_name)
# NetBox limits Pillow to 25 MP (bomb error above 50 MP), which rejects
# current 50 MP smartphone photos such as Google Pixel images (8160x6144).
BULK_IMAGE_MAX_PIXELS = 100_000_000
@contextmanager
def raised_image_pixel_limit():
from PIL import Image
original_limit = Image.MAX_IMAGE_PIXELS
if original_limit is not None and original_limit < BULK_IMAGE_MAX_PIXELS:
Image.MAX_IMAGE_PIXELS = BULK_IMAGE_MAX_PIXELS
try:
yield
finally:
Image.MAX_IMAGE_PIXELS = original_limit
class BulkImageUploadView(ContentTypePermissionRequiredMixin, View):
template_name = "netbox_utilities/bulk_image_upload.html"
@@ -65,9 +86,19 @@ class BulkImageUploadView(ContentTypePermissionRequiredMixin, View):
parent = self._get_parent(request, request.POST)
form = BulkImageUploadForm(request.POST, request.FILES)
if form.is_valid():
with raised_image_pixel_limit():
return self._process_upload(request, form, parent)
return self._render(request, form, parent)
def _process_upload(self, request, form, parent):
image_forms = self._build_image_forms(parent, form.cleaned_data)
invalid = [image_form for image_form in image_forms if not image_form.is_valid()]
if invalid:
for image_form in invalid:
image = image_form.files.get("image")
reason = self._image_error_reason(image)
if reason:
form.add_error("images", f"{getattr(image, 'name', 'Bild')}: Pillow meldet: {reason}")
for image_form in invalid:
filename = getattr(image_form.files.get("image"), "name", "Bild")
for errors in image_form.errors.values():
@@ -109,6 +140,26 @@ class BulkImageUploadView(ContentTypePermissionRequiredMixin, View):
queryset = queryset.restrict(request.user, "view")
return get_object_or_404(queryset, pk=object_id)
@staticmethod
def _image_error_reason(image):
"""Return the underlying Pillow error, which Django hides behind a generic message."""
from PIL import Image
try:
if hasattr(image, "temporary_file_path"):
source = image.temporary_file_path()
else:
image.seek(0)
source = BytesIO(image.read())
with Image.open(source) as opened:
opened.verify()
except Exception as error: # noqa: BLE001 - surface any Pillow failure
return f"{type(error).__name__}: {error}"
finally:
if hasattr(image, "seek"):
image.seek(0)
return None
@staticmethod
def _build_image_forms(parent, cleaned_data):
image_forms = []
+1 -1
View File
@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
[project]
name = "netbox-utilities"
version = "0.14.0"
version = "0.14.1"
description = "Navigation, tenant utilities, connection VLANs, partial-width racks, and bulk operations for NetBox 4.6/4.7"
readme = "README.md"
requires-python = ">=3.12"