ImapIdleChannelAdapter and MailReceivingMessageSource both now require a MessageReceiver instance as the constructor argument rather than accepting a URI, creating a MessageReceiver, and then passing properties to that receiver. Also, the 'mail:inbound-channel-adapter' element now supports the 'java-mail-properties' attribute.

This commit is contained in:
Mark Fisher
2008-10-30 21:04:30 +00:00
parent 46866bd05a
commit 16e20517ef
5 changed files with 39 additions and 52 deletions

View File

@@ -16,8 +16,6 @@
package org.springframework.integration.mail;
import java.util.Properties;
import javax.mail.Message;
import javax.mail.MessagingException;
import javax.mail.internet.MimeMessage;
@@ -57,24 +55,11 @@ public class ImapIdleChannelAdapter extends AbstractMessageProducingEndpoint imp
this.mailReceiver = mailReceiver;
}
public ImapIdleChannelAdapter(String url) {
Assert.isTrue(url.startsWith("imap"), "url must start with 'imap'");
this.mailReceiver = new ImapMailReceiver(url);
}
public void setTaskExecutor(TaskExecutor taskExecutor) {
this.taskExecutor = taskExecutor;
}
public void setJavaMailProperties(Properties javaMailProperties) {
this.mailReceiver.setJavaMailProperties(javaMailProperties);
}
public void setShouldDeleteMessages(boolean shouldDeleteMessages) {
this.mailReceiver.setShouldDeleteMessages(shouldDeleteMessages);
}
protected void handleMailMessagingException(MessagingException e) {
if (logger.isWarnEnabled()) {
logger.warn("error occurred in idle task", e);

View File

@@ -51,19 +51,6 @@ public class MailReceivingMessageSource implements MessageSource<javax.mail.Mess
this.mailReceiver = mailReceiver;
}
public MailReceivingMessageSource(String url) {
if (url.startsWith("imap")) {
this.mailReceiver = new Pop3MailReceiver(url);
}
else if (url.startsWith("pop3")) {
this.mailReceiver = new ImapMailReceiver(url);
}
else {
throw new UnsupportedOperationException("unsupported mail protocol '"
+ url.substring(0, url.indexOf(':')) + "'");
}
}
public Message<javax.mail.Message> receive() {
try {

View File

@@ -19,9 +19,11 @@ package org.springframework.integration.mail.config;
import org.w3c.dom.Element;
import org.springframework.beans.factory.support.BeanDefinitionBuilder;
import org.springframework.beans.factory.support.BeanDefinitionReaderUtils;
import org.springframework.beans.factory.xml.AbstractSingleBeanDefinitionParser;
import org.springframework.beans.factory.xml.ParserContext;
import org.springframework.integration.mail.ImapIdleChannelAdapter;
import org.springframework.integration.mail.ImapMailReceiver;
import org.springframework.util.Assert;
import org.springframework.util.StringUtils;
@@ -47,23 +49,30 @@ public class ImapIdleChannelAdapterParser extends AbstractSingleBeanDefinitionPa
protected void doParse(Element element, ParserContext parserContext, BeanDefinitionBuilder builder) {
String channel = element.getAttribute("channel");
String uri = element.getAttribute("store-uri");
String taskExecutorRef = element.getAttribute("task-executor");
String propertiesRef = element.getAttribute("java-mail-properties");
Assert.hasText(channel, "the 'channel' attribute is required");
builder.addConstructorArgReference(this.parseImapMailReceiver(element, parserContext));
builder.addPropertyReference("outputChannel", channel);
String taskExecutorRef = element.getAttribute("task-executor");
if (StringUtils.hasText(taskExecutorRef)) {
builder.addPropertyReference("taskExecutor", taskExecutorRef);
}
}
private String parseImapMailReceiver(Element element, ParserContext parserContext) {
String uri = element.getAttribute("store-uri");
Assert.hasText(uri, "the 'store-uri' attribute is required");
Assert.isTrue(uri.toLowerCase().startsWith("imap"),
"store-uri must start with 'imap' for the imap idle channel adapter");
builder.addConstructorArgValue(uri);
BeanDefinitionBuilder receiverBuilder = BeanDefinitionBuilder.genericBeanDefinition(ImapMailReceiver.class);
receiverBuilder.addConstructorArgValue(uri);
String propertiesRef = element.getAttribute("java-mail-properties");
if (StringUtils.hasText(propertiesRef)) {
builder.addPropertyReference("javaMailProperties", propertiesRef);
receiverBuilder.addPropertyReference("javaMailProperties", propertiesRef);
}
if (StringUtils.hasLength(taskExecutorRef)) {
builder.addPropertyReference("taskExecutor", taskExecutorRef);
}
builder.addPropertyValue("shouldDeleteMessages",
receiverBuilder.addPropertyValue("shouldDeleteMessages",
!"false".equals(element.getAttribute("should-delete-messages")));
builder.addPropertyReference("outputChannel", channel);
return BeanDefinitionReaderUtils.registerWithGeneratedName(
receiverBuilder.getBeanDefinition(), parserContext.getRegistry());
}
}

View File

@@ -27,6 +27,7 @@ import org.springframework.integration.mail.MailReceiver;
import org.springframework.integration.mail.MailReceivingMessageSource;
import org.springframework.integration.mail.Pop3MailReceiver;
import org.springframework.util.Assert;
import org.springframework.util.StringUtils;
/**
* Parser for the &lt;inbound-channel-adapter&gt; element of Spring
@@ -39,23 +40,27 @@ public class MailInboundChannelAdapterParser extends AbstractPollingInboundChann
@Override
protected String parseSource(Element element, ParserContext parserContext) {
String uri = element.getAttribute("store-uri");
//String propertiesRef = element.getAttribute("javaMailProperties");
Assert.hasText(uri, "the 'store-uri' attribute is required");
boolean isPop3 = uri.toLowerCase().startsWith("pop3");
boolean isImap = uri.toLowerCase().startsWith("imap");
Assert.isTrue(isPop3 || isImap, "the 'store-uri' must begin with 'pop3' or 'imap'");
MailReceiver mailReceiver = isPop3 ? new Pop3MailReceiver(uri) : new ImapMailReceiver(uri);
BeanDefinitionBuilder builder = BeanDefinitionBuilder.genericBeanDefinition(MailReceivingMessageSource.class);
/*
if (StringUtils.hasText(propertiesRef)) {
folderConnectionBuilder.addPropertyReference("javaMailProperties",
propertiesRef);
}
*/
builder.addConstructorArgValue(mailReceiver);
builder.addConstructorArgReference(this.parseMailReceiver(element, parserContext));
return BeanDefinitionReaderUtils.registerWithGeneratedName(
builder.getBeanDefinition(), parserContext.getRegistry());
}
private String parseMailReceiver(Element element, ParserContext parserContext) {
String uri = element.getAttribute("store-uri");
Assert.hasText(uri, "the 'store-uri' attribute is required");
boolean isPop3 = uri.toLowerCase().startsWith("pop3");
boolean isImap = uri.toLowerCase().startsWith("imap");
Assert.isTrue(isPop3 || isImap, "the 'store-uri' must begin with 'pop3' or 'imap'");
Class<? extends MailReceiver> receiverClass = isPop3 ? Pop3MailReceiver.class : ImapMailReceiver.class;
BeanDefinitionBuilder receiverBuilder = BeanDefinitionBuilder.genericBeanDefinition(receiverClass);
receiverBuilder.addConstructorArgValue(uri);
String propertiesRef = element.getAttribute("java-mail-properties");
if (StringUtils.hasText(propertiesRef)) {
receiverBuilder.addPropertyReference("javaMailProperties", propertiesRef);
}
return BeanDefinitionReaderUtils.registerWithGeneratedName(
receiverBuilder.getBeanDefinition(), parserContext.getRegistry());
}
}

View File

@@ -49,6 +49,7 @@
<xsd:attribute name="id" type="xsd:string"/>
<xsd:attribute name="channel" type="xsd:string"/>
<xsd:attribute name="store-uri" type="xsd:string" use="required"/>
<xsd:attribute name="java-mail-properties" type="xsd:string"/>
</xsd:complexType>
</xsd:element>