Refactor PrestashopProxyController to simplify image upload API path

api/src/main/java/fr/gameovergne/api/service/prestashop/PrestashopClient.java

@@ -2,10 +2,12 @@ package fr.gameovergne.api.service.prestashop;
 
 import lombok.extern.slf4j.Slf4j;
 import org.springframework.beans.factory.annotation.Value;
+import org.springframework.core.io.ByteArrayResource;
 import org.springframework.http.HttpHeaders;
 import org.springframework.http.MediaType;
 import org.springframework.http.ResponseEntity;
 import org.springframework.stereotype.Service;
+import org.springframework.util.LinkedMultiValueMap;
 import org.springframework.util.MultiValueMap;
 import org.springframework.web.client.RestClient;
 import org.springframework.web.client.RestClientException;
@@ -229,56 +231,50 @@ public class PrestashopClient {
     /**
      * Upload d'une image produit vers PrestaShop :
      * POST /api/images/products/{productId}
-     *
+     * <p>
      * On envoie directement les bytes du fichier avec le bon content-type (image/jpeg, image/png, ...).
      */
-    public ResponseEntity<String> uploadProductImage(String productId, String rawQuery, MultipartFile imageFile) {
+
-        String uri = baseUrl + "/images/products/" + productId;
+    public ResponseEntity<String> uploadProductImage(String productId,
+                                                     String rawQuery,
+                                                     MultipartFile imageFile) {
+        try {
+            StringBuilder url = new StringBuilder(baseUrl)
+                    .append("/images/products/")
+                    .append(productId);
+
             if (rawQuery != null && !rawQuery.isBlank()) {
-            uri = uri + "?" + rawQuery;
+                url.append("?").append(rawQuery);
             }
 
-        // Content-Type de l'image : on essaie de respecter celui fourni par le client,
+            // Corps multipart avec un champ "image" comme demandé par PrestaShop
-        // sinon on force sur du JPEG par défaut.
+            MultiValueMap<String, Object> body = new LinkedMultiValueMap<>();
-        String ct = imageFile.getContentType();
-        MediaType mediaType = (ct != null && !ct.isBlank())
-                ? MediaType.parseMediaType(ct)
-                : MediaType.IMAGE_JPEG;
 
-        log.info("[PrestaShop] POST (image) {} (size={} bytes, contentType={})",
+            ByteArrayResource imageResource = new ByteArrayResource(imageFile.getBytes()) {
-                uri, imageFile.getSize(), mediaType);
+                @Override
+                public String getFilename() {
+                    // PrestaShop veut un filename, même si ce n'est pas critique
+                    return imageFile.getOriginalFilename() != null
+                            ? imageFile.getOriginalFilename()
+                            : "image.jpg";
+                }
+            };
 
-        try {
+            body.add("image", imageResource); // 👈 IMPORTANT : le nom du champ = "image"
-            byte[] bytes = imageFile.getBytes();
 
-            return client.post()
+            log.info("[PrestaShop] POST (image multipart) {} (size={} bytes, contentType={})",
-                    .uri(uri)
+                    url, imageFile.getSize(), imageFile.getContentType());
-                    .contentType(mediaType)
+
-                    .body(bytes)
+            return client
+                    .post()
+                    .uri(url.toString())
+                    .contentType(MediaType.MULTIPART_FORM_DATA)
+                    .body(body)
                     .retrieve()
                     .toEntity(String.class);
 
-        } catch (RestClientResponseException ex) {
+        } catch (IOException e) {
-            // Erreur fonctionnelle renvoyée par Presta (taille, format, etc.)
+            throw new RuntimeException("Erreur lors de la lecture du fichier image", e);
-            log.error("[PrestaShop] POST image error {} : {}", ex.getRawStatusCode(), ex.getResponseBodyAsString());
-            return ResponseEntity
-                    .status(ex.getRawStatusCode())
-                    .contentType(MediaType.APPLICATION_XML)
-                    .body(ex.getResponseBodyAsString());
-        } catch (IOException ex) {
-            // Problème de lecture du fichier envoyé depuis le client
-            log.error("[PrestaShop] POST image, erreur lecture fichier", ex);
-            return ResponseEntity
-                    .status(400)
-                    .contentType(MediaType.TEXT_PLAIN)
-                    .body("Invalid image file");
-        } catch (RestClientException ex) {
-            // Cas réseau / technique
-            log.error("[PrestaShop] POST image technical error", ex);
-            return ResponseEntity
-                    .status(502)
-                    .contentType(MediaType.TEXT_PLAIN)
-                    .body("Error while calling PrestaShop WebService");
         }
     }
 }