Envoye de Fichier au client

Bonjour,

j'essaye d'envoyé un fichier a partir du serveur au client, j'ai donc suivi la FAQ et j'ai le code suivant dans une méthode execute d'une Action :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25

File f = new File(path); response.setContentType("multipart/zip"); response.setHeader("Content-Disposition", "attachment; filename=\"" + f.getName().trim().substring(1,f.getName().length()) + "\";"); response.setContentLength((int)f.length());   try { OutputStream os = response.getOutputStream(); FileInputStream stream = new FileInputStream(f); BufferedInputStream bis = new BufferedInputStream(stream); InputStream is = new BufferedInputStream(bis); int count; byte buf[] = new byte[4096]; while ((count = is.read(buf)) > -1) { os.write(buf, 0, count); } is.close(); os.close(); } catch (Exception ex) { ex.printStackTrace(); }

---

(C'est le code de la FAQ)

Mais lorsque je l'exécute, j'obtiens cette erreure :

Code:

---

java.lang.IllegalStateException: "getWriter()" a déjà été appelé pour cette réponse

---

J'avoue être un peu perdu et ne pas savoir quoi faire .. je vous appelle donc a l'aide ! :)

Merci d'avance !

Tu peux donner la stack complète ? Notamment à quelle ligne ça se produit ...

Ca se produit a la ligne suivante :

Code:

---

OutputStream os = response.getOutputStream();

---

la stack entière donne ca :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32

java.lang.IllegalStateException: "getWriter()" a déjà été appelé pour cette réponse at org.apache.catalina.connector.Response.getOutputStream(Response.java:568) at org.apache.catalina.connector.ResponseFacade.getOutputStream(ResponseFacade.java:180) at javax.servlet.ServletResponseWrapper.getOutputStream(ServletResponseWrapper.java:101) at planning.web.action.FileUpAction.executeOk(FileUpAction.java:57) at planning.web.action.MyAction.execute(MyAction.java:66) at org.apache.struts.action.RequestProcessor.processActionPerform(RequestProcessor.java:431) at org.apache.struts.action.RequestProcessor.process(RequestProcessor.java:236) at org.apache.struts.action.ActionServlet.process(ActionServlet.java:1196) at org.apache.struts.action.ActionServlet.doGet(ActionServlet.java:414) at javax.servlet.http.HttpServlet.service(HttpServlet.java:689) at javax.servlet.http.HttpServlet.service(HttpServlet.java:802) at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:252) at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:173) at planning.web.filter.FilterSpace.doFilter(FilterSpace.java:27) at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:202) at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:173) at org.displaytag.filter.ResponseOverrideFilter.doFilter(ResponseOverrideFilter.java:125) at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:202) at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:173) at org.apache.catalina.core.StandardWrapperValve.invoke(StandardWrapperValve.java:213) at org.apache.catalina.core.StandardContextValve.invoke(StandardContextValve.java:178) at org.apache.catalina.core.StandardHostValve.invoke(StandardHostValve.java:126) at org.apache.catalina.valves.ErrorReportValve.invoke(ErrorReportValve.java:105) at org.apache.catalina.core.StandardEngineValve.invoke(StandardEngineValve.java:107) at org.apache.catalina.connector.CoyoteAdapter.service(CoyoteAdapter.java:148) at org.apache.coyote.http11.Http11Processor.process(Http11Processor.java:869) at org.apache.coyote.http11.Http11BaseProtocol$Http11ConnectionHandler.processConnection(Http11BaseProtocol.java:664) at org.apache.tomcat.util.net.PoolTcpEndpoint.processSocket(PoolTcpEndpoint.java:527) at org.apache.tomcat.util.net.LeaderFollowerWorkerThread.runIt(LeaderFollowerWorkerThread.java:80) at org.apache.tomcat.util.threads.ThreadPool$ControlRunnable.run(ThreadPool.java:684) at java.lang.Thread.run(Unknown Source)

---

Tu peux nous poster l'intégralité de ta méthode exécute ?

Désolé, j'étais pas la hier soir, donc ma méthode execute :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40

protected void executeOk(ActionMapping mapping, ActionForm form, HttpServletRequest request, HttpServletResponse response) throws SQLException { target = "up";   if(request.getServletPath().startsWith("/resetFileUp")){ target = "reset"; return; } if(request.getServletPath().startsWith("/sendFichier")){ target = "ok"; String path = request.getParameter("path"); System.out.println(path);   File f = new File(path); response.setContentType("multipart/zip"); response.setHeader("Content-Disposition", "attachment; filename=\"" + f.getName().trim().substring(1,f.getName().length()) + "\";"); response.setContentLength((int)f.length());   try { OutputStream os = response.getOutputStream(); FileInputStream stream = new FileInputStream(f); BufferedInputStream bis = new BufferedInputStream(stream); InputStream is = new BufferedInputStream(bis); int count; byte buf[] = new byte[4096]; while ((count = is.read(buf)) > -1) { os.write(buf, 0, count); } is.close(); os.close(); } catch (Exception ex) { ex.printStackTrace(); } return; }

---

qui extends d'une classe MyAction avec pour vrai méthode execute ceci :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41

public ActionForward execute(ActionMapping mapping, ActionForm form, HttpServletRequest request, HttpServletResponse response) {   if (logger == null) { initLogger(); }   logger.info(" Enter in controller "); time = System.currentTimeMillis(); msg = new ActionMessages(); target = "erreur";     if (!baseConnect.isConnected()) { target = "erreurDB"; } else if (verifierSession(request.getSession()) && isErrEmpty()) { try { readDroit(); executeOk(mapping, form, request, response); } catch (SQLException e) { target = "erreur"; addErr("erreur", "errors.bd"); logger.error("SQLException", e); } }   saveErr(request);       ActionForward af = (mapping.findForward(target)); if (af != null) { logger.info("Exit controller | Time: " + (System.currentTimeMillis() - time) / 1000.0 + " | Target: " + target + " | Path: " + af.getPath()); } else { logger.info("Exit controller | Time: " + (System.currentTimeMillis() - time) / 1000.0 + " | Target: " + target); } return af;   }

---

J'ai fait un traitement similaire (qui fonctionne). J'ai juste une petite différence...

Code:

---

1 2 3 4 5 6 7

... Tu utilises ceci OutputStream os = response.getOutputStream();   Il faudrait utiliser ceci   ServletOutputStream out = response.getOutputStream();

---

Le reste reste inchangé...

( J'utilise également setContentType("unknown/unknown"); )

A+

J'ai toujours la même erreur :?

j'ai modifié mon code comme suit :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24

File f = new File(path); response.setContentType("unknown/unknown"); response.setHeader("Content-Disposition", "attachment; filename=\"" + f.getName().trim().substring(0,f.getName().length()) + "\";"); response.setContentLength((int)f.length()); try { ServletOutputStream os = response.getOutputStream(); FileInputStream stream = new FileInputStream(f); BufferedInputStream bis = new BufferedInputStream(stream); InputStream is = new BufferedInputStream(bis); int count; byte buf[] = new byte[4096]; while ((count = is.read(buf)) > -1) { os.write(buf, 0, count); } is.close(); os.close(); } catch (Exception ex) { ex.printStackTrace(); } return;

---

j'ai mis "0" comme 1er attribut du substring car il m'enlevait a chaque fois la 1ere lettre de mon fichier ... Est-ce que l'erreur ne pourrait pas venir du path qui est mauvais ? Pourtant le fichier existe bien..

slt,

comme énoncé dans ton exception, le probleme vient du fait qu'un getWriter a déjà été effectué sur ton objet response au moment ou tu fais un getOutputStream, et tu peux faire soit l'un soit l'autre mais pas les 2!

A priori, j'imagine que c'est l'une de ces lignes :

Code:

---

1 2 3 4

response.setContentType("unknown/unknown"); response.setHeader("Content-Disposition", "attachment; filename=\"" + f.getName().trim().substring(0,f.getName().length()) + "\";"); response.setContentLength((int)f.length());

---

qui pose probleme en appelant getWriter avant response.getOutputStream().

Tu devrais donc tenter a mon avis de faire un response.getWriter à la place de response.getOutputStream, puis de travailler sur le PrintWriter récupéré à la pace de ton ServletOutputStream (tu trouveras les mêmes méthodes write ...)

a+

Il faudrait que ton contrôleur étende Action ou DispatchAction mais pas MyAction
(je pense que le problème vient de saveErrors)

A+

Citation:

---

Envoyé par OButterlin

Il faudrait que ton contrôleur étende Action ou DispatchAction mais pas MyAction
(je pense que le problème vient de saveErrors)

A+

---

Avec ceci j'ai tjs la meme erreure :?


Par contre, avec l'idée de Zag, j'arrive a récupérer le PrintWriter, mais le problème est que ca me génere un fichier corrompu, apparemment car il écrit tout le buffer (même s'il y en a trop a la fin) dans le fichier, et j'avoue que je ne sais plus comment on fait pour passé outre ca.

Mon nouveau code :

Code:

---

1 2 3 4 5 6 7 8 9 10 11

PrintWriter os = response.getWriter(); FileReader fr = new FileReader(f); int count; char buf[] = new char[4096]; while((count = fr.read(buf)) > -1){   os.write(buf, 0, count);   } fr.close(); os.close();

---

C'est vrai qu'en regardant de plus près ce que j'avais codé, j'étends HttpServlet et pas Action (ou DispatchAction etc...)
Je crois que Struts fait des traitements sur request (ça c'est certain) et response (peut-être)
Le premier, j'en suis sûr, j'ai du passer par un Filtre pour traiter correctement l'encodate UTF-8...
Ca t'obligerait à definir et mapper la servlet dans le fichier web.xml, c'est tout...

A+

Ahem ..

Ca ne fait pas longtemps que je développe avec Struts, et j'ai du mal a comprendre ce que tu essayes de me dire ^^"

Alors en résumé, fait étendre ton contrôleur de HttpServlet, renomme la méthode executeOk en service avec cette signature

Code:

---

1 2 3 4 5

public void service(HttpServletRequest request, HttpServletResponse response) { ... le code de executeOk }

---

et rajoute la servlet dans ton fichier web.xml

Code:

---

1 2 3 4 5 6 7 8 9 10 11

... <servlet> <servlet-name>leNomVoulu</servlet-name> <servlet-class>package.nomClasse.servlet</servlet-class> </servlet> ... <servlet-mapping> <servlet-name>leNomVoulu</servlet-name> <url-pattern>/leNomMappeDansLaPage</url-pattern> </servlet-mapping>

---

re,


je pense que ca doit venir du fait que tu ne fais pas de flush et qu'en plus tu fermes ton writer à la fin.
Concrètement, tu peux essayer comme ca :

Code:

---

1 2 3 4 5 6 7 8 9 10 11

PrintWriter os = response.getWriter(); FileReader fr = new FileReader(f); int count; char buf[] = new char[4096]; while((count = fr.read(buf)) > -1){   os.write(buf, 0, count); os.flush(); } fr.close();

---

à vérifier ...

Le problème a en effet l'air de venir de l'encodage qui est mal géré (et non pas du PrintWriter), je vais donc essayer de mettre en place la servlet comme le suggère OButterlin.

En tout cas merci beaucoup a vous deux ! ^^