feat: improve avatar filename generation with content type handling

- Refactored avatar download logic to determine file extension based on the content type from the response headers.
- Removed redundant code for extension mapping, ensuring a cleaner implementation.
- Enhanced error handling by returning None for unsupported content types.

Author: pablohashescobar

apps/api/plane/authentication/adapter/base.py

@@ -130,6 +130,20 @@ def download_and_upload_avatar(self, avatar_url, user):
             if content_length and int(content_length) > max_size:
                 return None
 
+            # Get content type and determine file extension
+            content_type = response.headers.get("Content-Type", "image/jpeg")
+            extension_map = {
+                "image/jpeg": "jpg",
+                "image/jpg": "jpg",
+                "image/png": "png",
+                "image/gif": "gif",
+                "image/webp": "webp",
+            }
+            extension = extension_map.get(content_type)
+
+            if not extension:
+                return None
+
             # Download with size limit
             chunks = []
             total_size = 0
@@ -141,17 +155,6 @@ def download_and_upload_avatar(self, avatar_url, user):
             content = b"".join(chunks)
             file_size = len(content)
 
-            # Get content type and determine file extension
-            content_type = response.headers.get("Content-Type", "image/jpeg")
-            extension_map = {
-                "image/jpeg": "jpg",
-                "image/jpg": "jpg",
-                "image/png": "png",
-                "image/gif": "gif",
-                "image/webp": "webp",
-            }
-            extension = extension_map.get(content_type, "jpg")
-
             # Generate unique filename
             filename = f"{uuid.uuid4().hex}-user-avatar.{extension}"