Problem in Java Mail using Servlet

Hi,

I am using an html form to be filled in and a servlet to get parameters from the html form. The servlet then uses the Java Mail API to attach the file and send the mail.

The problem I am facing is that when I deploy the application on server on my local machine, the servlet gets the file location from html form, searches it on my local machine, attaches it with mail and sends it. But when I use a server installed and running on different machine on network, the servlet gets the file name from the html form, searches that file on the machine on which server is running (and not on my local machine), and fails to attach the file, and mail sending gets failed. Is there any solution to my problem? The servlet code that I am using is as follows:

import java.io.*;
import java.util.*;
import javax.mail.*;
import javax.mail.internet.*;
import javax.servlet.*;
import javax.servlet.http.*;
import javax.activation.*;
 
 
public class AttachServ extends HttpServlet implements Servlet {
protected void doPost(HttpServletRequest req, HttpServletResponse res) throws ServletException, IOException {

String host = "<smtpservername>";
Properties props1 = new Properties();

props1.put("mail.smtp.host", host);
props1.put("mail.debug", "true");

Session session1 = Session.getInstance(props1);

try {
Transport bus1 = session1.getTransport("smtp");

bus1.connect();

Message msg1 = new MimeMessage(session1);

InternetAddress[] address1 = {new InternetAddress(req.getParameter("to"))};
InternetAddress[] address2 = {new InternetAddress(req.getParameter("from"))};

msg1.setFrom(new InternetAddress(req.getParameter("from")));

msg1.setRecipients(Message.RecipientType.TO, address1);

msg1.setSubject(req.getParameter("subject"));

msg1.setSentDate(new Date());

String file = req.getParameter("filelocation");
String content = req.getParameter("content");

if (file.equals(""))
msg1.setText(content);
else
setData(content, msg1, file);

msg1.saveChanges();

bus1.sendMessage(msg1, address1);
bus1.close();

PrintWriter out = res.getWriter();
out.println("<html><body>Mail has been sent.");
out.println("To send another email, <a href=SendMail.html>Click Here</a></body></html>");
}
catch (MessagingException mex) {
mex.printStackTrace();
while (mex.getNextException() != null) {
Exception ex = mex.getNextException();
ex.printStackTrace();
if (!(ex instanceof MessagingException)) break;
else mex = (MessagingException)ex;
}
PrintWriter out = res.getWriter();
out.println("<html><body>Mail sending failed!");
out.println("To retry, <a href=SendMail.html>Click Here</a></body></html>");
}
}

public static void setData(String cnt, Message msg, String filename)
throws MessagingException {
MimeBodyPart p1 = new MimeBodyPart();
MimeBodyPart p2 = new MimeBodyPart();

FileDataSource fds = new FileDataSource(filename);
p1.setDataHandler(new DataHandler(fds));
p1.setFileName(fds.getName());

p2.setText(cnt);

Multipart mp = new MimeMultipart();

mp.addBodyPart(p1);
mp.addBodyPart(p2);

msg.setContent(mp);
}
}

Thanks,

Harsh

[BSouther: Added UBB CODE tags]
[ December 22, 2006: Message edited by: Ben Souther ]

Welcome to JavaRanch.

The server won't be able to access a file on the client directly. Instead of entering the name of the file, use a file upload element in the HTML form. Then you can use the Commons FileUpload to extract the file contents from the other form data.

set your form attribute ENCTYPE="multipart/form-data", and then in servlet download your file to local machine and then send it as attachement.

Originally posted by pawank gupta:
set your form attribute ENCTYPE="multipart/form-data", and then in servlet download your file to local machine and then send it as attachement.

Just changing the enctype of the form is not enough.
Servlets (out of the box) can't handle multipart form data.
This is why Ulf posted the link to commons/fileupload.

[hi-jack removed] [ December 22, 2006: Message edited by: Ben Souther ]

[hi-jack removed]
[ December 22, 2006: Message edited by: Ben Souther ]

Talha-bin -Shakir,

Please do not hi-jack other people's threads.
If you have a question, start your own.

Thank you,
-Ben