Téléchargement de gros fichiers depuis un Servlet

Bonjour,

J'essaie de faire un Servlet dont la seule fonction est de lancer le téléchargement d'un fichier (depuis le serveur, vers le client). Le téléchargement se passe bien pour les petits fichiers, mais le serveur plante lamentablement dès que j'essaie avec des plus gros fichiers (>50mo) :

Code:

---

1 2 3 4 5 6 7 8 9 10

java.lang.OutOfMemoryError:*Java*heap*space org.apache.commons.io.output.ByteArrayOutputStream.needNewBuffer(ByteArrayOutputStream.java:124) org.apache.commons.io.output.ByteArrayOutputStream.write(ByteArrayOutputStream.java:155) org.apache.commons.io.IOUtils.copyLarge(IOUtils.java:1263) org.apache.commons.io.IOUtils.copy(IOUtils.java:1236) org.apache.commons.io.IOUtils.toByteArray(IOUtils.java:360) org.apache.commons.io.FileUtils.readFileToByteArray(FileUtils.java:1360) com.becatalog.servlet.MediaDownload.doGet(MediaDownload.java:51) javax.servlet.http.HttpServlet.service(HttpServlet.java:617) javax.servlet.http.HttpServlet.service(HttpServlet.java:717)

---

Voici mon code :

Code:

---

1 2 3 4 5 6 7 8 9 10

response.setContentType(type); // type du fichier response.setHeader("Content-Disposition", "attachment; filename="+name);   File file = new File(DBInterface.getFinalDir(), name); // chemin vers le fichier   ServletOutputStream out = response.getOutputStream();   out.write(FileUtils.readFileToByteArray(file)); out.flush(); out.close();

---

Bon, comme ça, ma première idée c'est de découper le fichier comme un bourin et de l'envoyer en pièces détachées, mais ça me parait long et pas très propre.
Est ce que quelqu'un sait comment je dois m'y prendre pour envoyer un fichier de cette façon?

Merci d'avance : )

On pourrait voir le code de ça FileUtils.readFileToByteArray(file)
Je soupçonne que tu charges tout le contenu du fichier.
Sinon c'est ton conteneur web qui bufferise trop et dans ce cas il faut "flusher" périodiquement.

Attention il convient également de setter le content-length.

Bonjour et merci pour votre réponse!

Citation:

---

Envoyé par Nemek

On pourrait voir le code de ça FileUtils.readFileToByteArray(file)

---

FileUtils fait parti d'un paquet que j'ai téléchargé et ajouté à mon projet :
http://commons.apache.org/io/apidocs/org/apache/commons/io/FileUtils.html
http://commons.apache.org/io/apidocs/org/apache/commons/io/FileUtils.html#readFileToByteArray(java.io.File)

Citation:

---

Envoyé par Nemek

Je soupçonne que tu charges tout le contenu du fichier.

---

Oui je pense aussi que ça vient de la, le fichier remplis toute la mémoire allouée à java et lance une exception.

Citation:

---

Envoyé par Nemek

Sinon c'est ton conteneur web qui bufferise trop et dans ce cas il faut "flusher" périodiquement.

---

Donc en gros, je charge le premier méga, je fais un flush, puis je charge le second... etc...?

C'était une idée mais normalement le conteneur devrait le gérer tout seul. Dès que son buffer est plein il envoie les données.

Pour copier un flux c'est pas très compliqué :

Code:

---

1 2 3 4 5 6 7 8 9 10 11

public static void copy(InputStream in, OutputStream out, int bufferSize) { byte[] buffer = new byte[bufferSize]; int n; while ( (n=in.read(buffer, 0, bufferSize) >= 0) { out.write(buffer, 0, n); } }   public static void copy(InputStream in, OutputStream out) { copy(in, out, 1024*8); }

---

Tu peux utiliser la première méthode avec la taille du buffer de la réponse.

Merci à vous, ça marche parfaitement maintenant.

Juste une dernière question, j'ai un problème à cette ligne :

Code:

---

response.setContentLength(file.length());

---

setContentLength requière un entier, file.length() retourne un long. Si je cast ça ne posera pas de problème?

Potentiellement si car un fichier (en fonction du système de fichier et de sa configuration) un fichier peut avoir plus de 2^31-1 octets (soit environ 2Go)

Cependant il n'est pas commun de transférer autant de données par HTTP.

Okay, et bien encore une fois merci,

Bonne journée à vous.
Cordialement.