Basic Javamail Authentication

Question

Hi,
Could somebody tell me what is wrong with the following code? I have to authenticate myself to receive a email since I am inside a firewall..
The following is what I have right now....
import javax.mail.*;
import javax.mail.internet.*;
import java.util.Properties;
import javax.servlet.*;
public class MailMessage {
public MailMessage(String to, String subject, String bodyText)
throws ServletException
try
String host = "smtp of my server";
String from = "[email protected]";
String toAddress = "[email protected]";
String cc = "[email protected]";
// Get system properties
Properties props = System.getProperties();
props.put("mail.smtp.host", host);
// Only allow registered users to send mail
props.put("mail.smtp.auth", "true");
Session session = Session.getInstance(props, new MyAuthenticator());
session.setDebug(true);
// Define message
MimeMessage message = new MimeMessage(session);
// To
InternetAddress Address = new InternetAddress(toAddress);
message.addRecipient(Message.RecipientType.TO, toAddress);
// From
InternetAddress fromAddress = new InternetAddress(from);
message.setFrom(fromAddress);
message.setSubject(subject);
message.setText(bodyText);
Transport transport = session.getTransport("smtp");
// Send message
transport.send(message);
catch (Exception e)
throw new ServletException(
"<b>Email Message not sent. Contact admin </b>\n" + e);
class MyAuthenticator extends Authenticator
public PasswordAuthentication getPasswordAuthentication()
String userName = "Administrator";
String password = "openup" ;
return new PasswordAuthentication(userName, password);
I get the following error which I don't understand..
addRecipient(javax.mail.Message.RecipientType, javax.mail.Address) in javax.mail.Message cannot be applied to (javax.mail.Message.RecipientType, java.lang.String)
What does this mean? Am I not using the authenticator class properly? I am lost now!
Thanks for any help!

Answer 1

1) He's not explicitly casting to (InternetAddress)
2) You can't cast a String to an InternetAddress anyway.
He did the right thing in creating a new InternetAddress from his toAddress (String), but he just passed the wrong one in to the function: it only takes an InternetAddress (not a String).
Let's do it differently:
String strTo... // String representation of recipient passed in
InternetAddress addrTo; // this is what we need for the function
addrTo = new InternetAddress(strTo);
// here's the original equivalent
message.addRecipient(Message.RecipientType.TO, strTo); // wrong. this doesn't take a String
// needs this:
message.addRecipient(MessageRecipientType.TO,addrTo); // yep. takes InternetAddress.