JavaMail giving "Domain contains illegal character null" when sending two email addresses

[Mike London]

Hello,

Although this code seems to work fine standalone, when I use it in Tomcat from a service with two email addresses, I get this error: "Domain contains illegal character null"

For example, the "TO" might contain: "[e-mail],[e-mail]"

(A single address works!).

Now, stepping through the code, I indeed get the correct "TO" POSTed to the service (both email addresses), but on the Transport.send(), I get this nasty "Domain contains illegal character null" error. Looking at the debug output, when the code crashes on Transport.send(), it says:

javax.mail.internet.AddressException: Domain contains illegal character in string ``[e-mail], [e-mail]''

Can anyone suggest what is going on and how to fix this issue?

Thanks very much,

-- mike

// Code excerpt ...

.
.
.
            final String to = recipient;

            // Sender's email ID needs to be mentioned
            final String from = fromAccount;

            // Get system properties
            final Properties properties = System.getProperties();
            properties.put("mail.smtp.auth", true);
            properties.put("mail.smtp.starttls.enable", true);
            properties.put("mail.smtp.host", host);
            properties.put("mail.smtp.port", port);   // 587
            properties.put("mail.smtp.password", password);
            properties.put("mail.transport.protocol", "smtp");
            properties.put("mail.smtp.debug", "false");
            properties.put("mail.smtp.socketFactory.class", "javax.net.ssl.SSLSocketFactory");
            properties.put("mail.smtp.socketFactory.port", port);
            properties.put("mail.smtp.socketFactory.fallback", "true");

            // Get the default Session object.
            final Session session = Session.getInstance(properties, new Authenticator()
            {
                protected PasswordAuthentication getPasswordAuthentication()
                {
                    return new PasswordAuthentication(from, password);
                }
            });

            // OK, send the message already...
            try
            {
                // log info so we know that somebody used the service!
                log.info("Email message sent to: " + to);
                log.info("Remote IP: " + request.ip());  //client's IP
                log.info("Subject: " + subjectLine);
                log.info("From: " + from);

                // Create a default MimeMessage object.
                Message message = new MimeMessage(session);

                // Set From: header field of the header.
                //  message.setFrom(new InternetAddress(from));

                message.setFrom(new InternetAddress(to, from));
                message.setRecipients(Message.RecipientType.TO, InternetAddress.parse(to));
                message.setRecipients(Message.RecipientType.CC, InternetAddress.parse(cc));
                message.setRecipients(Message.RecipientType.BCC, InternetAddress.parse(bcc));

                // Set Subject: header field
                message.setSubject(subjectLine);

                // Create the message part
                BodyPart messageBodyPart = new MimeBodyPart();

                // Now set the actual message
                messageBodyPart.setText(emailMessage);

                // Create a multipartmessage
                Multipart multipart = new MimeMultipart();

                // Set text message part
                multipart.addBodyPart(messageBodyPart);

                // Part two is attachment, if any...
                // make sure we have a real file and not a typo or other non-file entry...
                pathToAttachment = pathToAttachment.trim();\
                
                // attachment code omitted (not part of issue...)

        
                }
                // Send the complete message parts, if no error from above
                if (errors.toString().trim().length() == 0)
                {
                    message.setContent(multipart);
                    message.saveChanges();
     [b]               Transport.send(message);   // fails here...[/b]
                }

[Rob Spoor]

Mike London wrote:

message.setFrom(new InternetAddress(to, from));

I don't think that first argument is right there, is it?

[Mike London]

Rob Spoor wrote:

Mike London wrote:

message.setFrom(new InternetAddress(to, from));

I don't think that first argument is right there, is it?

Actually, I think that WAS the problem. Once I removed that line and replaced it with the correctly formatted "setFrom()" it worked!

Good eyes there.

Thank you so much, Rob!!!

(I would give you a 'cow' if my permissions allowed it. LOL)


-- mike

[Paul Clapham]

Mike London wrote:(I would give you a 'cow' if my permissions allowed it. LOL)

Done for you.

[Rob Spoor]

You're welcome, and thanks for the cow!