You are viewing a plain text version of this content. The canonical link for it is here.
Posted to server-user@james.apache.org by timprepscius <ti...@piratemail.se> on 2012/07/18 01:29:46 UTC

ok some news about standard mailet

so, 

I need to get this change in:

svn diff
Index: src/main/java/org/apache/james/transport/matchers/SenderIsLocal.java
===================================================================
--- src/main/java/org/apache/james/transport/matchers/SenderIsLocal.java	(revision 1362688)
+++ src/main/java/org/apache/james/transport/matchers/SenderIsLocal.java	(working copy)
@@ -33,6 +33,9 @@
 public class SenderIsLocal extends GenericMatcher {
 
     public final Collection<MailAddress> match(Mail mail) throws MessagingException {
+	if (mail.getSender() == null)
+		return null;
+
         return (getMailetContext().isLocalEmail(mail.getSender())) ? Arrays.asList(mail.getSender()): null;
     }
 }



so I used trunk of mailet:

svn info
Path: .
URL: http://svn.apache.org/repos/asf/james/mailet/standard/trunk
Repository Root: http://svn.apache.org/repos/asf
Repository UUID: 13f79535-47bb-0310-9956-ffa450edef68
Revision: 1362688
Node Kind: directory
Schedule: normal
Last Changed Author: eric
Last Changed Rev: 1362051
Last Changed Date: 2012-07-16 10:19:21 -0400 (Mon, 16 Jul 2012)


I built it and deployed it into the lib directory:
james@blue:~/apache-james/lib$ ls -la *standard*
lrwxrwxrwx 1 james james     60 Jul 17 19:20 apache-standard-mailets-1.2-20120709.004650-715.jar -> apache-standard-mailets-1.2-20120709.004650-715.jar.original
-rw------- 1 james james 121046 Jul 10 08:39 apache-standard-mailets-1.2-20120709.004650-715.jar.original
-rw-r--r-- 1 james james 120541 Jul 17 18:33 apache-standard-mailets-1.2-SNAPSHOT.jar


---

and now james has lots of new errors.
actually I can't send any mail without a bounce being triggered.


INFO  19:15:20,910 | james.mailetcontext | Local delivery with ToSenderFolder mailet for mail Mail1342566918279-811ba289-672b-4a55-a2fd-e03499aec883-!383287 with sender …..
INFO  19:15:20,938 | james.mailetcontext | Error while storing mail.
java.lang.UnsupportedOperationException
        at java.util.AbstractList.remove(Unknown Source)
        at java.util.AbstractList$Itr.remove(Unknown Source)
        at org.apache.james.transport.mailets.AbstractRecipientRewriteTableMailet.service(AbstractRecipientRewriteTableMailet.java:78)
        at org.apache.james.transport.mailets.LocalDelivery.service(LocalDelivery.java:70)
        at org.apache.james.mailetcontainer.impl.camel.CamelProcessor.process(CamelProcessor.java:65)



I'm going to go to the previous tag, fix isSenderLocal there.

thought you would like to know, to possibly review changes which have happened between the 10th and the 17th.


-tim

---------------------------------------------------------------------
To unsubscribe, e-mail: server-user-unsubscribe@james.apache.org
For additional commands, e-mail: server-user-help@james.apache.org


Re: ok some news about standard mailet

Posted by Eric Charles <er...@apache.org>.
On 07/18/2012 01:24 PM, Stefano Bagnara wrote:
> 2012/7/18 Eric Charles <er...@apache.org>:
>> Hi Stefano,
>>
>> Thx for your help. The SenderIsLocal/ToSenderFolder is already at the
>> beginning of the root processor. I rechecked it once again, and mailet is
>> invoked twice
>
> Let me understand:
>
> Is SenderIsLocal (matcher) invoked once or twice? Is the same mail
> (same message name, same recipients)?
> Is ToSenderFolder invoked once or twice? Is the same mail (same
> message name, same recipients)?
>

matcher and mailet are invoked twice, but with different mails (at least 
the recipients - the list of recipient changes depending if mailet is in 
root or transport processor).

> Maybe we have a bug in the mailet container v3, or maybe you have some
> mailets after this one that puts a copy of the mail in the spool
> starting from root again.
>

I will tackle this down and provide a fix to the configuration and/or 
the mailet container.

> Stefano
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: server-user-unsubscribe@james.apache.org
> For additional commands, e-mail: server-user-help@james.apache.org
>


-- 
eric | http://about.echarles.net | @echarles


---------------------------------------------------------------------
To unsubscribe, e-mail: server-user-unsubscribe@james.apache.org
For additional commands, e-mail: server-user-help@james.apache.org


Re: ok some news about standard mailet

Posted by Stefano Bagnara <ap...@bago.org>.
2012/7/18 Eric Charles <er...@apache.org>:
> Hi Stefano,
>
> Thx for your help. The SenderIsLocal/ToSenderFolder is already at the
> beginning of the root processor. I rechecked it once again, and mailet is
> invoked twice

Let me understand:

Is SenderIsLocal (matcher) invoked once or twice? Is the same mail
(same message name, same recipients)?
Is ToSenderFolder invoked once or twice? Is the same mail (same
message name, same recipients)?

Maybe we have a bug in the mailet container v3, or maybe you have some
mailets after this one that puts a copy of the mail in the spool
starting from root again.

Stefano

---------------------------------------------------------------------
To unsubscribe, e-mail: server-user-unsubscribe@james.apache.org
For additional commands, e-mail: server-user-help@james.apache.org


Re: ok some news about standard mailet

Posted by Eric Charles <er...@apache.org>.
Hi Stefano,

Thx for your help. The SenderIsLocal/ToSenderFolder is already at the 
beginning of the root processor. I rechecked it once again, and mailet 
is invoked twice

   <processors>
     <processor state="root" enableJmx="true">
       <mailet match="SenderIsLocal" class="ToSenderFolder">
	      <folder>Sent</folder>
	      <consume>false</consume>
       </mailet>
       <mailet match="All" class="PostmasterAlias"/>
       <mailet match="RelayLimit=30" class="Null"/>
       <mailet match="All" class="SpamAssassin">
         <spamdHost>localhost</spamdHost>
         <spamdPort>783</spamdPort>
       </mailet>
...

Thx, Eric

On 07/18/2012 12:36 PM, Stefano Bagnara wrote:
> 2012/7/18 Eric Charles <er...@apache.org>:
>> For recipients with a mix of local and non-local, the matcher and mailet is
>> called twice:
>> - A first time with the complete list (if mailet/matcher in root process,)
>> or with the local recipient (if mailet/matcher in transport processor).
>> - A second time with the remote recipient.
>
> if a matcher is called twice then it means a previous matcher splitted the mail.
> So maybe you have a previous matcher that split the local/remote
> recipients (RecipientIsLocal ?)
>
>> With the current impl, we end-up with 2 mails in the Sent Folder.
>>
>> Of course, anyone to confirm this behaviour?
>
> Maybe you simply have to move the SenderIsLocal/ToSenderFolder BEFORE
> any other matcher that is going to split the mail.
>
> Stefano
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: server-user-unsubscribe@james.apache.org
> For additional commands, e-mail: server-user-help@james.apache.org
>


-- 
eric | http://about.echarles.net | @echarles


---------------------------------------------------------------------
To unsubscribe, e-mail: server-user-unsubscribe@james.apache.org
For additional commands, e-mail: server-user-help@james.apache.org


Re: ok some news about standard mailet

Posted by Stefano Bagnara <ap...@bago.org>.
2012/7/18 Eric Charles <er...@apache.org>:
> For recipients with a mix of local and non-local, the matcher and mailet is
> called twice:
> - A first time with the complete list (if mailet/matcher in root process,)
> or with the local recipient (if mailet/matcher in transport processor).
> - A second time with the remote recipient.

if a matcher is called twice then it means a previous matcher splitted the mail.
So maybe you have a previous matcher that split the local/remote
recipients (RecipientIsLocal ?)

> With the current impl, we end-up with 2 mails in the Sent Folder.
>
> Of course, anyone to confirm this behaviour?

Maybe you simply have to move the SenderIsLocal/ToSenderFolder BEFORE
any other matcher that is going to split the mail.

Stefano

---------------------------------------------------------------------
To unsubscribe, e-mail: server-user-unsubscribe@james.apache.org
For additional commands, e-mail: server-user-help@james.apache.org


Re: ok some news about standard mailet

Posted by Eric Charles <er...@apache.org>.
On 07/18/2012 10:53 AM, Stefano Bagnara wrote:
> 2012/7/18 Eric Charles <er...@apache.org>:
>> Hi Tim,
>>
>> SenderIsLocal is typically used in combination with ToSenderFolder.
>>
>> In that case, we need to store only once the mail in the Sent Folder, but
>> the current mailet architecture is designed/focused on recipients, not
>> senders.
>>
>> Matcher/Mailets will be invoked numerous times for a single mail (for all
>> the local recipients, and once for the remote one).
>
> Matchers and Mailets are invoked only once for each mail. When a
> Matcher returns a partial match (and ONLY on PARTIAL) then the
> original mail is splitted and from that point we will have 2 different
> emails that will go through the following matchers/mailets. Otherwise
> you will keep working with the same original mail.
>

I have rechecked it, and with the current impl, it seems what Stefano 
describes is valid only if all recipients are local or non-local.

I have reverted the SenderIsLocal matcher to return the complete list of 
recpients if the sender is local.

For recipients with a mix of local and non-local, the matcher and mailet 
is called twice:
- A first time with the complete list (if mailet/matcher in root 
process,) or with the local recipient (if mailet/matcher in transport 
processor).
- A second time with the remote recipient.

With the current impl, we end-up with 2 mails in the Sent Folder.

Of course, anyone to confirm this behaviour?

Thx, Eric

> So, if you want a matcher to match based on the sender you will have
> to return all the recipients in case it matches, none in case it
> doesn't match (as suggested by Tim).
>
> A matcher should only return recipients, not other addresses.
>

Sounds in line with docs and code.

>> So we endup with multiple copies of the mail in the Sent Folder.
>
> This will not happen, unless the "saving" mailet has a loop in its logic.
>

There is no loop in the ToSenderFolder.

> Stefano
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: server-user-unsubscribe@james.apache.org
> For additional commands, e-mail: server-user-help@james.apache.org
>


-- 
eric | http://about.echarles.net | @echarles


---------------------------------------------------------------------
To unsubscribe, e-mail: server-user-unsubscribe@james.apache.org
For additional commands, e-mail: server-user-help@james.apache.org


Re: ok some news about standard mailet

Posted by Stefano Bagnara <ap...@bago.org>.
2012/7/18 Eric Charles <er...@apache.org>:
> Hi Tim,
>
> SenderIsLocal is typically used in combination with ToSenderFolder.
>
> In that case, we need to store only once the mail in the Sent Folder, but
> the current mailet architecture is designed/focused on recipients, not
> senders.
>
> Matcher/Mailets will be invoked numerous times for a single mail (for all
> the local recipients, and once for the remote one).

Matchers and Mailets are invoked only once for each mail. When a
Matcher returns a partial match (and ONLY on PARTIAL) then the
original mail is splitted and from that point we will have 2 different
emails that will go through the following matchers/mailets. Otherwise
you will keep working with the same original mail.

So, if you want a matcher to match based on the sender you will have
to return all the recipients in case it matches, none in case it
doesn't match (as suggested by Tim).

A matcher should only return recipients, not other addresses.

> So we endup with multiple copies of the mail in the Sent Folder.

This will not happen, unless the "saving" mailet has a loop in its logic.

Stefano

---------------------------------------------------------------------
To unsubscribe, e-mail: server-user-unsubscribe@james.apache.org
For additional commands, e-mail: server-user-help@james.apache.org


Re: ok some news about standard mailet

Posted by Eric Charles <er...@apache.org>.
Hi Tim,

SenderIsLocal is typically used in combination with ToSenderFolder.

In that case, we need to store only once the mail in the Sent Folder, 
but the current mailet architecture is designed/focused on recipients, 
not senders.

Matcher/Mailets will be invoked numerous times for a single mail (for 
all the local recipients, and once for the remote one).

So we endup with multiple copies of the mail in the Sent Folder.

I need to think about it. Any other thoughts welcome :)

Thx, Eric

On 07/18/2012 02:29 AM, timprepscius wrote:
> ok
>
> so if I change the return to be getRecipients
> then things work properly:
>
>   svn diff
> Index: src/main/java/org/apache/james/transport/matchers/SenderIsLocal.java
> ===================================================================
> --- src/main/java/org/apache/james/transport/matchers/SenderIsLocal.java	(revision 1362688)
> +++ src/main/java/org/apache/james/transport/matchers/SenderIsLocal.java	(working copy)
> @@ -27,12 +27,21 @@
>   import org.apache.mailet.MailAddress;
>   import org.apache.mailet.base.GenericMatcher;
>
> +import java.util.ArrayList;
> +import java.util.List;
> +
>   /**
>    * Matches mail where the sender is local.
>    */
>   public class SenderIsLocal extends GenericMatcher {
>
>       public final Collection<MailAddress> match(Mail mail) throws MessagingException {
> -        return (getMailetContext().isLocalEmail(mail.getSender())) ? Arrays.asList(mail.getSender()): null;
> +
> +	if (mail.getSender() != null && getMailetContext().isLocalEmail(mail.getSender()))
> +	{
> +		return mail.getRecipients();
> +	}
> +
> +	return null;
>       }
>   }
>
>
> On Jul 17, 2012, at 8:10 PM, timprepscius wrote:
>
>> ok testing is showing that this fixes the exception, however.
>>
>> now james is thinking that I am send mails to myself.
>> and I receive (in box) mails I send.
>>
>> I can only surmise this is a function of the new "recipient" being returned by the to sender.
>>
>> I'll try not to use "getSender()" see what happens
>>
>> -tim
>>
>> On Jul 17, 2012, at 7:59 PM, timprepscius wrote:
>>
>>> testing now
>>>
>>> On Jul 17, 2012, at 7:58 PM, timprepscius wrote:
>>>
>>>> possibly this fix:
>>>>
>>>> svn diff
>>>> Index: src/main/java/org/apache/james/transport/matchers/SenderIsLocal.java
>>>> ===================================================================
>>>> --- src/main/java/org/apache/james/transport/matchers/SenderIsLocal.java	(revision 1362688)
>>>> +++ src/main/java/org/apache/james/transport/matchers/SenderIsLocal.java	(working copy)
>>>> @@ -27,12 +27,23 @@
>>>> import org.apache.mailet.MailAddress;
>>>> import org.apache.mailet.base.GenericMatcher;
>>>>
>>>> +import java.util.ArrayList;
>>>> +import java.util.List;
>>>> +
>>>> /**
>>>> * Matches mail where the sender is local.
>>>> */
>>>> public class SenderIsLocal extends GenericMatcher {
>>>>
>>>>    public final Collection<MailAddress> match(Mail mail) throws MessagingException {
>>>> -        return (getMailetContext().isLocalEmail(mail.getSender())) ? Arrays.asList(mail.getSender()): null;
>>>> +
>>>> +	if (mail.getSender() != null && getMailetContext().isLocalEmail(mail.getSender()))
>>>> +	{
>>>> +		List<MailAddress> recipients = new ArrayList<MailAddress>();
>>>> +		recipients.add(mail.getSender());
>>>> +		return recipients;
>>>> +	}
>>>> +
>>>> +	return null;
>>>>    }
>>>> }
>>>>
>>>>
>>>> On Jul 17, 2012, at 7:52 PM, timprepscius wrote:
>>>>
>>>>> ok, I have a feeling that
>>>>>
>>>>> the Arras.asList is the culprit of these errors.
>>>>>
>>>>> the reason being that array I believe is being incorporated into some mail structure and then manipulated.
>>>>> which is impossible.
>>>>>
>>>>> -tim
>>>>>
>>>>> On Jul 17, 2012, at 7:29 PM, timprepscius wrote:
>>>>>
>>>>>> so,
>>>>>>
>>>>>> I need to get this change in:
>>>>>>
>>>>>> svn diff
>>>>>> Index: src/main/java/org/apache/james/transport/matchers/SenderIsLocal.java
>>>>>> ===================================================================
>>>>>> --- src/main/java/org/apache/james/transport/matchers/SenderIsLocal.java	(revision 1362688)
>>>>>> +++ src/main/java/org/apache/james/transport/matchers/SenderIsLocal.java	(working copy)
>>>>>> @@ -33,6 +33,9 @@
>>>>>> public class SenderIsLocal extends GenericMatcher {
>>>>>>
>>>>>> public final Collection<MailAddress> match(Mail mail) throws MessagingException {
>>>>>> +	if (mail.getSender() == null)
>>>>>> +		return null;
>>>>>> +
>>>>>>      return (getMailetContext().isLocalEmail(mail.getSender())) ? Arrays.asList(mail.getSender()): null;
>>>>>> }
>>>>>> }
>>>>>>
>>>>>>
>>>>>>
>>>>>> so I used trunk of mailet:
>>>>>>
>>>>>> svn info
>>>>>> Path: .
>>>>>> URL: http://svn.apache.org/repos/asf/james/mailet/standard/trunk
>>>>>> Repository Root: http://svn.apache.org/repos/asf
>>>>>> Repository UUID: 13f79535-47bb-0310-9956-ffa450edef68
>>>>>> Revision: 1362688
>>>>>> Node Kind: directory
>>>>>> Schedule: normal
>>>>>> Last Changed Author: eric
>>>>>> Last Changed Rev: 1362051
>>>>>> Last Changed Date: 2012-07-16 10:19:21 -0400 (Mon, 16 Jul 2012)
>>>>>>
>>>>>>
>>>>>> I built it and deployed it into the lib directory:
>>>>>> james@blue:~/apache-james/lib$ ls -la *standard*
>>>>>> lrwxrwxrwx 1 james james     60 Jul 17 19:20 apache-standard-mailets-1.2-20120709.004650-715.jar -> apache-standard-mailets-1.2-20120709.004650-715.jar.original
>>>>>> -rw------- 1 james james 121046 Jul 10 08:39 apache-standard-mailets-1.2-20120709.004650-715.jar.original
>>>>>> -rw-r--r-- 1 james james 120541 Jul 17 18:33 apache-standard-mailets-1.2-SNAPSHOT.jar
>>>>>>
>>>>>>
>>>>>> ---
>>>>>>
>>>>>> and now james has lots of new errors.
>>>>>> actually I can't send any mail without a bounce being triggered.
>>>>>>
>>>>>>
>>>>>> INFO  19:15:20,910 | james.mailetcontext | Local delivery with ToSenderFolder mailet for mail Mail1342566918279-811ba289-672b-4a55-a2fd-e03499aec883-!383287 with sender …..
>>>>>> INFO  19:15:20,938 | james.mailetcontext | Error while storing mail.
>>>>>> java.lang.UnsupportedOperationException
>>>>>>     at java.util.AbstractList.remove(Unknown Source)
>>>>>>     at java.util.AbstractList$Itr.remove(Unknown Source)
>>>>>>     at org.apache.james.transport.mailets.AbstractRecipientRewriteTableMailet.service(AbstractRecipientRewriteTableMailet.java:78)
>>>>>>     at org.apache.james.transport.mailets.LocalDelivery.service(LocalDelivery.java:70)
>>>>>>     at org.apache.james.mailetcontainer.impl.camel.CamelProcessor.process(CamelProcessor.java:65)
>>>>>>
>>>>>>
>>>>>>
>>>>>> I'm going to go to the previous tag, fix isSenderLocal there.
>>>>>>
>>>>>> thought you would like to know, to possibly review changes which have happened between the 10th and the 17th.
>>>>>>
>>>>>>
>>>>>> -tim
>>>>>>
>>>>>> ---------------------------------------------------------------------
>>>>>> To unsubscribe, e-mail: server-user-unsubscribe@james.apache.org
>>>>>> For additional commands, e-mail: server-user-help@james.apache.org
>>>>>>
>>>>>>
>>>>>
>>>>>
>>>>>
>>>>> ---------------------------------------------------------------------
>>>>> To unsubscribe, e-mail: server-user-unsubscribe@james.apache.org
>>>>> For additional commands, e-mail: server-user-help@james.apache.org
>>>>>
>>>>>
>>>>
>>>>
>>>>
>>>> ---------------------------------------------------------------------
>>>> To unsubscribe, e-mail: server-user-unsubscribe@james.apache.org
>>>> For additional commands, e-mail: server-user-help@james.apache.org
>>>>
>>>>
>>>
>>
>>
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: server-user-unsubscribe@james.apache.org
>> For additional commands, e-mail: server-user-help@james.apache.org
>>
>>
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: server-user-unsubscribe@james.apache.org
> For additional commands, e-mail: server-user-help@james.apache.org
>


-- 
eric | http://about.echarles.net | @echarles


---------------------------------------------------------------------
To unsubscribe, e-mail: server-user-unsubscribe@james.apache.org
For additional commands, e-mail: server-user-help@james.apache.org


Re: ok some news about standard mailet

Posted by timprepscius <ti...@piratemail.se>.
ok

so if I change the return to be getRecipients
then things work properly:

 svn diff
Index: src/main/java/org/apache/james/transport/matchers/SenderIsLocal.java
===================================================================
--- src/main/java/org/apache/james/transport/matchers/SenderIsLocal.java	(revision 1362688)
+++ src/main/java/org/apache/james/transport/matchers/SenderIsLocal.java	(working copy)
@@ -27,12 +27,21 @@
 import org.apache.mailet.MailAddress;
 import org.apache.mailet.base.GenericMatcher;
 
+import java.util.ArrayList;
+import java.util.List;
+
 /**
  * Matches mail where the sender is local.
  */
 public class SenderIsLocal extends GenericMatcher {
 
     public final Collection<MailAddress> match(Mail mail) throws MessagingException {
-        return (getMailetContext().isLocalEmail(mail.getSender())) ? Arrays.asList(mail.getSender()): null;
+
+	if (mail.getSender() != null && getMailetContext().isLocalEmail(mail.getSender()))
+	{
+		return mail.getRecipients();
+	}
+
+	return null;
     }
 }


On Jul 17, 2012, at 8:10 PM, timprepscius wrote:

> ok testing is showing that this fixes the exception, however.
> 
> now james is thinking that I am send mails to myself.
> and I receive (in box) mails I send.
> 
> I can only surmise this is a function of the new "recipient" being returned by the to sender.
> 
> I'll try not to use "getSender()" see what happens
> 
> -tim
> 
> On Jul 17, 2012, at 7:59 PM, timprepscius wrote:
> 
>> testing now
>> 
>> On Jul 17, 2012, at 7:58 PM, timprepscius wrote:
>> 
>>> possibly this fix:
>>> 
>>> svn diff
>>> Index: src/main/java/org/apache/james/transport/matchers/SenderIsLocal.java
>>> ===================================================================
>>> --- src/main/java/org/apache/james/transport/matchers/SenderIsLocal.java	(revision 1362688)
>>> +++ src/main/java/org/apache/james/transport/matchers/SenderIsLocal.java	(working copy)
>>> @@ -27,12 +27,23 @@
>>> import org.apache.mailet.MailAddress;
>>> import org.apache.mailet.base.GenericMatcher;
>>> 
>>> +import java.util.ArrayList;
>>> +import java.util.List;
>>> +
>>> /**
>>> * Matches mail where the sender is local.
>>> */
>>> public class SenderIsLocal extends GenericMatcher {
>>> 
>>>   public final Collection<MailAddress> match(Mail mail) throws MessagingException {
>>> -        return (getMailetContext().isLocalEmail(mail.getSender())) ? Arrays.asList(mail.getSender()): null;
>>> +
>>> +	if (mail.getSender() != null && getMailetContext().isLocalEmail(mail.getSender()))
>>> +	{
>>> +		List<MailAddress> recipients = new ArrayList<MailAddress>();
>>> +		recipients.add(mail.getSender());
>>> +		return recipients;
>>> +	}
>>> +
>>> +	return null;
>>>   }
>>> }
>>> 
>>> 
>>> On Jul 17, 2012, at 7:52 PM, timprepscius wrote:
>>> 
>>>> ok, I have a feeling that 
>>>> 
>>>> the Arras.asList is the culprit of these errors.
>>>> 
>>>> the reason being that array I believe is being incorporated into some mail structure and then manipulated.
>>>> which is impossible.
>>>> 
>>>> -tim
>>>> 
>>>> On Jul 17, 2012, at 7:29 PM, timprepscius wrote:
>>>> 
>>>>> so, 
>>>>> 
>>>>> I need to get this change in:
>>>>> 
>>>>> svn diff
>>>>> Index: src/main/java/org/apache/james/transport/matchers/SenderIsLocal.java
>>>>> ===================================================================
>>>>> --- src/main/java/org/apache/james/transport/matchers/SenderIsLocal.java	(revision 1362688)
>>>>> +++ src/main/java/org/apache/james/transport/matchers/SenderIsLocal.java	(working copy)
>>>>> @@ -33,6 +33,9 @@
>>>>> public class SenderIsLocal extends GenericMatcher {
>>>>> 
>>>>> public final Collection<MailAddress> match(Mail mail) throws MessagingException {
>>>>> +	if (mail.getSender() == null)
>>>>> +		return null;
>>>>> +
>>>>>     return (getMailetContext().isLocalEmail(mail.getSender())) ? Arrays.asList(mail.getSender()): null;
>>>>> }
>>>>> }
>>>>> 
>>>>> 
>>>>> 
>>>>> so I used trunk of mailet:
>>>>> 
>>>>> svn info
>>>>> Path: .
>>>>> URL: http://svn.apache.org/repos/asf/james/mailet/standard/trunk
>>>>> Repository Root: http://svn.apache.org/repos/asf
>>>>> Repository UUID: 13f79535-47bb-0310-9956-ffa450edef68
>>>>> Revision: 1362688
>>>>> Node Kind: directory
>>>>> Schedule: normal
>>>>> Last Changed Author: eric
>>>>> Last Changed Rev: 1362051
>>>>> Last Changed Date: 2012-07-16 10:19:21 -0400 (Mon, 16 Jul 2012)
>>>>> 
>>>>> 
>>>>> I built it and deployed it into the lib directory:
>>>>> james@blue:~/apache-james/lib$ ls -la *standard*
>>>>> lrwxrwxrwx 1 james james     60 Jul 17 19:20 apache-standard-mailets-1.2-20120709.004650-715.jar -> apache-standard-mailets-1.2-20120709.004650-715.jar.original
>>>>> -rw------- 1 james james 121046 Jul 10 08:39 apache-standard-mailets-1.2-20120709.004650-715.jar.original
>>>>> -rw-r--r-- 1 james james 120541 Jul 17 18:33 apache-standard-mailets-1.2-SNAPSHOT.jar
>>>>> 
>>>>> 
>>>>> ---
>>>>> 
>>>>> and now james has lots of new errors.
>>>>> actually I can't send any mail without a bounce being triggered.
>>>>> 
>>>>> 
>>>>> INFO  19:15:20,910 | james.mailetcontext | Local delivery with ToSenderFolder mailet for mail Mail1342566918279-811ba289-672b-4a55-a2fd-e03499aec883-!383287 with sender …..
>>>>> INFO  19:15:20,938 | james.mailetcontext | Error while storing mail.
>>>>> java.lang.UnsupportedOperationException
>>>>>    at java.util.AbstractList.remove(Unknown Source)
>>>>>    at java.util.AbstractList$Itr.remove(Unknown Source)
>>>>>    at org.apache.james.transport.mailets.AbstractRecipientRewriteTableMailet.service(AbstractRecipientRewriteTableMailet.java:78)
>>>>>    at org.apache.james.transport.mailets.LocalDelivery.service(LocalDelivery.java:70)
>>>>>    at org.apache.james.mailetcontainer.impl.camel.CamelProcessor.process(CamelProcessor.java:65)
>>>>> 
>>>>> 
>>>>> 
>>>>> I'm going to go to the previous tag, fix isSenderLocal there.
>>>>> 
>>>>> thought you would like to know, to possibly review changes which have happened between the 10th and the 17th.
>>>>> 
>>>>> 
>>>>> -tim
>>>>> 
>>>>> ---------------------------------------------------------------------
>>>>> To unsubscribe, e-mail: server-user-unsubscribe@james.apache.org
>>>>> For additional commands, e-mail: server-user-help@james.apache.org
>>>>> 
>>>>> 
>>>> 
>>>> 
>>>> 
>>>> ---------------------------------------------------------------------
>>>> To unsubscribe, e-mail: server-user-unsubscribe@james.apache.org
>>>> For additional commands, e-mail: server-user-help@james.apache.org
>>>> 
>>>> 
>>> 
>>> 
>>> 
>>> ---------------------------------------------------------------------
>>> To unsubscribe, e-mail: server-user-unsubscribe@james.apache.org
>>> For additional commands, e-mail: server-user-help@james.apache.org
>>> 
>>> 
>> 
> 
> 
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: server-user-unsubscribe@james.apache.org
> For additional commands, e-mail: server-user-help@james.apache.org
> 
> 



---------------------------------------------------------------------
To unsubscribe, e-mail: server-user-unsubscribe@james.apache.org
For additional commands, e-mail: server-user-help@james.apache.org


Re: ok some news about standard mailet

Posted by timprepscius <ti...@piratemail.se>.
ok testing is showing that this fixes the exception, however.

now james is thinking that I am send mails to myself.
and I receive (in box) mails I send.

I can only surmise this is a function of the new "recipient" being returned by the to sender.

I'll try not to use "getSender()" see what happens

-tim

On Jul 17, 2012, at 7:59 PM, timprepscius wrote:

> testing now
> 
> On Jul 17, 2012, at 7:58 PM, timprepscius wrote:
> 
>> possibly this fix:
>> 
>> svn diff
>> Index: src/main/java/org/apache/james/transport/matchers/SenderIsLocal.java
>> ===================================================================
>> --- src/main/java/org/apache/james/transport/matchers/SenderIsLocal.java	(revision 1362688)
>> +++ src/main/java/org/apache/james/transport/matchers/SenderIsLocal.java	(working copy)
>> @@ -27,12 +27,23 @@
>> import org.apache.mailet.MailAddress;
>> import org.apache.mailet.base.GenericMatcher;
>> 
>> +import java.util.ArrayList;
>> +import java.util.List;
>> +
>> /**
>> * Matches mail where the sender is local.
>> */
>> public class SenderIsLocal extends GenericMatcher {
>> 
>>    public final Collection<MailAddress> match(Mail mail) throws MessagingException {
>> -        return (getMailetContext().isLocalEmail(mail.getSender())) ? Arrays.asList(mail.getSender()): null;
>> +
>> +	if (mail.getSender() != null && getMailetContext().isLocalEmail(mail.getSender()))
>> +	{
>> +		List<MailAddress> recipients = new ArrayList<MailAddress>();
>> +		recipients.add(mail.getSender());
>> +		return recipients;
>> +	}
>> +
>> +	return null;
>>    }
>> }
>> 
>> 
>> On Jul 17, 2012, at 7:52 PM, timprepscius wrote:
>> 
>>> ok, I have a feeling that 
>>> 
>>> the Arras.asList is the culprit of these errors.
>>> 
>>> the reason being that array I believe is being incorporated into some mail structure and then manipulated.
>>> which is impossible.
>>> 
>>> -tim
>>> 
>>> On Jul 17, 2012, at 7:29 PM, timprepscius wrote:
>>> 
>>>> so, 
>>>> 
>>>> I need to get this change in:
>>>> 
>>>> svn diff
>>>> Index: src/main/java/org/apache/james/transport/matchers/SenderIsLocal.java
>>>> ===================================================================
>>>> --- src/main/java/org/apache/james/transport/matchers/SenderIsLocal.java	(revision 1362688)
>>>> +++ src/main/java/org/apache/james/transport/matchers/SenderIsLocal.java	(working copy)
>>>> @@ -33,6 +33,9 @@
>>>> public class SenderIsLocal extends GenericMatcher {
>>>> 
>>>>  public final Collection<MailAddress> match(Mail mail) throws MessagingException {
>>>> +	if (mail.getSender() == null)
>>>> +		return null;
>>>> +
>>>>      return (getMailetContext().isLocalEmail(mail.getSender())) ? Arrays.asList(mail.getSender()): null;
>>>>  }
>>>> }
>>>> 
>>>> 
>>>> 
>>>> so I used trunk of mailet:
>>>> 
>>>> svn info
>>>> Path: .
>>>> URL: http://svn.apache.org/repos/asf/james/mailet/standard/trunk
>>>> Repository Root: http://svn.apache.org/repos/asf
>>>> Repository UUID: 13f79535-47bb-0310-9956-ffa450edef68
>>>> Revision: 1362688
>>>> Node Kind: directory
>>>> Schedule: normal
>>>> Last Changed Author: eric
>>>> Last Changed Rev: 1362051
>>>> Last Changed Date: 2012-07-16 10:19:21 -0400 (Mon, 16 Jul 2012)
>>>> 
>>>> 
>>>> I built it and deployed it into the lib directory:
>>>> james@blue:~/apache-james/lib$ ls -la *standard*
>>>> lrwxrwxrwx 1 james james     60 Jul 17 19:20 apache-standard-mailets-1.2-20120709.004650-715.jar -> apache-standard-mailets-1.2-20120709.004650-715.jar.original
>>>> -rw------- 1 james james 121046 Jul 10 08:39 apache-standard-mailets-1.2-20120709.004650-715.jar.original
>>>> -rw-r--r-- 1 james james 120541 Jul 17 18:33 apache-standard-mailets-1.2-SNAPSHOT.jar
>>>> 
>>>> 
>>>> ---
>>>> 
>>>> and now james has lots of new errors.
>>>> actually I can't send any mail without a bounce being triggered.
>>>> 
>>>> 
>>>> INFO  19:15:20,910 | james.mailetcontext | Local delivery with ToSenderFolder mailet for mail Mail1342566918279-811ba289-672b-4a55-a2fd-e03499aec883-!383287 with sender …..
>>>> INFO  19:15:20,938 | james.mailetcontext | Error while storing mail.
>>>> java.lang.UnsupportedOperationException
>>>>     at java.util.AbstractList.remove(Unknown Source)
>>>>     at java.util.AbstractList$Itr.remove(Unknown Source)
>>>>     at org.apache.james.transport.mailets.AbstractRecipientRewriteTableMailet.service(AbstractRecipientRewriteTableMailet.java:78)
>>>>     at org.apache.james.transport.mailets.LocalDelivery.service(LocalDelivery.java:70)
>>>>     at org.apache.james.mailetcontainer.impl.camel.CamelProcessor.process(CamelProcessor.java:65)
>>>> 
>>>> 
>>>> 
>>>> I'm going to go to the previous tag, fix isSenderLocal there.
>>>> 
>>>> thought you would like to know, to possibly review changes which have happened between the 10th and the 17th.
>>>> 
>>>> 
>>>> -tim
>>>> 
>>>> ---------------------------------------------------------------------
>>>> To unsubscribe, e-mail: server-user-unsubscribe@james.apache.org
>>>> For additional commands, e-mail: server-user-help@james.apache.org
>>>> 
>>>> 
>>> 
>>> 
>>> 
>>> ---------------------------------------------------------------------
>>> To unsubscribe, e-mail: server-user-unsubscribe@james.apache.org
>>> For additional commands, e-mail: server-user-help@james.apache.org
>>> 
>>> 
>> 
>> 
>> 
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: server-user-unsubscribe@james.apache.org
>> For additional commands, e-mail: server-user-help@james.apache.org
>> 
>> 
> 



---------------------------------------------------------------------
To unsubscribe, e-mail: server-user-unsubscribe@james.apache.org
For additional commands, e-mail: server-user-help@james.apache.org


Re: ok some news about standard mailet

Posted by timprepscius <ti...@piratemail.se>.
testing now

On Jul 17, 2012, at 7:58 PM, timprepscius wrote:

> possibly this fix:
> 
> svn diff
> Index: src/main/java/org/apache/james/transport/matchers/SenderIsLocal.java
> ===================================================================
> --- src/main/java/org/apache/james/transport/matchers/SenderIsLocal.java	(revision 1362688)
> +++ src/main/java/org/apache/james/transport/matchers/SenderIsLocal.java	(working copy)
> @@ -27,12 +27,23 @@
> import org.apache.mailet.MailAddress;
> import org.apache.mailet.base.GenericMatcher;
> 
> +import java.util.ArrayList;
> +import java.util.List;
> +
> /**
>  * Matches mail where the sender is local.
>  */
> public class SenderIsLocal extends GenericMatcher {
> 
>     public final Collection<MailAddress> match(Mail mail) throws MessagingException {
> -        return (getMailetContext().isLocalEmail(mail.getSender())) ? Arrays.asList(mail.getSender()): null;
> +
> +	if (mail.getSender() != null && getMailetContext().isLocalEmail(mail.getSender()))
> +	{
> +		List<MailAddress> recipients = new ArrayList<MailAddress>();
> +		recipients.add(mail.getSender());
> +		return recipients;
> +	}
> +
> +	return null;
>     }
> }
> 
> 
> On Jul 17, 2012, at 7:52 PM, timprepscius wrote:
> 
>> ok, I have a feeling that 
>> 
>> the Arras.asList is the culprit of these errors.
>> 
>> the reason being that array I believe is being incorporated into some mail structure and then manipulated.
>> which is impossible.
>> 
>> -tim
>> 
>> On Jul 17, 2012, at 7:29 PM, timprepscius wrote:
>> 
>>> so, 
>>> 
>>> I need to get this change in:
>>> 
>>> svn diff
>>> Index: src/main/java/org/apache/james/transport/matchers/SenderIsLocal.java
>>> ===================================================================
>>> --- src/main/java/org/apache/james/transport/matchers/SenderIsLocal.java	(revision 1362688)
>>> +++ src/main/java/org/apache/james/transport/matchers/SenderIsLocal.java	(working copy)
>>> @@ -33,6 +33,9 @@
>>> public class SenderIsLocal extends GenericMatcher {
>>> 
>>>   public final Collection<MailAddress> match(Mail mail) throws MessagingException {
>>> +	if (mail.getSender() == null)
>>> +		return null;
>>> +
>>>       return (getMailetContext().isLocalEmail(mail.getSender())) ? Arrays.asList(mail.getSender()): null;
>>>   }
>>> }
>>> 
>>> 
>>> 
>>> so I used trunk of mailet:
>>> 
>>> svn info
>>> Path: .
>>> URL: http://svn.apache.org/repos/asf/james/mailet/standard/trunk
>>> Repository Root: http://svn.apache.org/repos/asf
>>> Repository UUID: 13f79535-47bb-0310-9956-ffa450edef68
>>> Revision: 1362688
>>> Node Kind: directory
>>> Schedule: normal
>>> Last Changed Author: eric
>>> Last Changed Rev: 1362051
>>> Last Changed Date: 2012-07-16 10:19:21 -0400 (Mon, 16 Jul 2012)
>>> 
>>> 
>>> I built it and deployed it into the lib directory:
>>> james@blue:~/apache-james/lib$ ls -la *standard*
>>> lrwxrwxrwx 1 james james     60 Jul 17 19:20 apache-standard-mailets-1.2-20120709.004650-715.jar -> apache-standard-mailets-1.2-20120709.004650-715.jar.original
>>> -rw------- 1 james james 121046 Jul 10 08:39 apache-standard-mailets-1.2-20120709.004650-715.jar.original
>>> -rw-r--r-- 1 james james 120541 Jul 17 18:33 apache-standard-mailets-1.2-SNAPSHOT.jar
>>> 
>>> 
>>> ---
>>> 
>>> and now james has lots of new errors.
>>> actually I can't send any mail without a bounce being triggered.
>>> 
>>> 
>>> INFO  19:15:20,910 | james.mailetcontext | Local delivery with ToSenderFolder mailet for mail Mail1342566918279-811ba289-672b-4a55-a2fd-e03499aec883-!383287 with sender …..
>>> INFO  19:15:20,938 | james.mailetcontext | Error while storing mail.
>>> java.lang.UnsupportedOperationException
>>>      at java.util.AbstractList.remove(Unknown Source)
>>>      at java.util.AbstractList$Itr.remove(Unknown Source)
>>>      at org.apache.james.transport.mailets.AbstractRecipientRewriteTableMailet.service(AbstractRecipientRewriteTableMailet.java:78)
>>>      at org.apache.james.transport.mailets.LocalDelivery.service(LocalDelivery.java:70)
>>>      at org.apache.james.mailetcontainer.impl.camel.CamelProcessor.process(CamelProcessor.java:65)
>>> 
>>> 
>>> 
>>> I'm going to go to the previous tag, fix isSenderLocal there.
>>> 
>>> thought you would like to know, to possibly review changes which have happened between the 10th and the 17th.
>>> 
>>> 
>>> -tim
>>> 
>>> ---------------------------------------------------------------------
>>> To unsubscribe, e-mail: server-user-unsubscribe@james.apache.org
>>> For additional commands, e-mail: server-user-help@james.apache.org
>>> 
>>> 
>> 
>> 
>> 
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: server-user-unsubscribe@james.apache.org
>> For additional commands, e-mail: server-user-help@james.apache.org
>> 
>> 
> 
> 
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: server-user-unsubscribe@james.apache.org
> For additional commands, e-mail: server-user-help@james.apache.org
> 
> 



---------------------------------------------------------------------
To unsubscribe, e-mail: server-user-unsubscribe@james.apache.org
For additional commands, e-mail: server-user-help@james.apache.org


Re: ok some news about standard mailet

Posted by timprepscius <ti...@piratemail.se>.
possibly this fix:

svn diff
Index: src/main/java/org/apache/james/transport/matchers/SenderIsLocal.java
===================================================================
--- src/main/java/org/apache/james/transport/matchers/SenderIsLocal.java	(revision 1362688)
+++ src/main/java/org/apache/james/transport/matchers/SenderIsLocal.java	(working copy)
@@ -27,12 +27,23 @@
 import org.apache.mailet.MailAddress;
 import org.apache.mailet.base.GenericMatcher;
 
+import java.util.ArrayList;
+import java.util.List;
+
 /**
  * Matches mail where the sender is local.
  */
 public class SenderIsLocal extends GenericMatcher {
 
     public final Collection<MailAddress> match(Mail mail) throws MessagingException {
-        return (getMailetContext().isLocalEmail(mail.getSender())) ? Arrays.asList(mail.getSender()): null;
+
+	if (mail.getSender() != null && getMailetContext().isLocalEmail(mail.getSender()))
+	{
+		List<MailAddress> recipients = new ArrayList<MailAddress>();
+		recipients.add(mail.getSender());
+		return recipients;
+	}
+
+	return null;
     }
 }


On Jul 17, 2012, at 7:52 PM, timprepscius wrote:

> ok, I have a feeling that 
> 
> the Arras.asList is the culprit of these errors.
> 
> the reason being that array I believe is being incorporated into some mail structure and then manipulated.
> which is impossible.
> 
> -tim
> 
> On Jul 17, 2012, at 7:29 PM, timprepscius wrote:
> 
>> so, 
>> 
>> I need to get this change in:
>> 
>> svn diff
>> Index: src/main/java/org/apache/james/transport/matchers/SenderIsLocal.java
>> ===================================================================
>> --- src/main/java/org/apache/james/transport/matchers/SenderIsLocal.java	(revision 1362688)
>> +++ src/main/java/org/apache/james/transport/matchers/SenderIsLocal.java	(working copy)
>> @@ -33,6 +33,9 @@
>> public class SenderIsLocal extends GenericMatcher {
>> 
>>    public final Collection<MailAddress> match(Mail mail) throws MessagingException {
>> +	if (mail.getSender() == null)
>> +		return null;
>> +
>>        return (getMailetContext().isLocalEmail(mail.getSender())) ? Arrays.asList(mail.getSender()): null;
>>    }
>> }
>> 
>> 
>> 
>> so I used trunk of mailet:
>> 
>> svn info
>> Path: .
>> URL: http://svn.apache.org/repos/asf/james/mailet/standard/trunk
>> Repository Root: http://svn.apache.org/repos/asf
>> Repository UUID: 13f79535-47bb-0310-9956-ffa450edef68
>> Revision: 1362688
>> Node Kind: directory
>> Schedule: normal
>> Last Changed Author: eric
>> Last Changed Rev: 1362051
>> Last Changed Date: 2012-07-16 10:19:21 -0400 (Mon, 16 Jul 2012)
>> 
>> 
>> I built it and deployed it into the lib directory:
>> james@blue:~/apache-james/lib$ ls -la *standard*
>> lrwxrwxrwx 1 james james     60 Jul 17 19:20 apache-standard-mailets-1.2-20120709.004650-715.jar -> apache-standard-mailets-1.2-20120709.004650-715.jar.original
>> -rw------- 1 james james 121046 Jul 10 08:39 apache-standard-mailets-1.2-20120709.004650-715.jar.original
>> -rw-r--r-- 1 james james 120541 Jul 17 18:33 apache-standard-mailets-1.2-SNAPSHOT.jar
>> 
>> 
>> ---
>> 
>> and now james has lots of new errors.
>> actually I can't send any mail without a bounce being triggered.
>> 
>> 
>> INFO  19:15:20,910 | james.mailetcontext | Local delivery with ToSenderFolder mailet for mail Mail1342566918279-811ba289-672b-4a55-a2fd-e03499aec883-!383287 with sender …..
>> INFO  19:15:20,938 | james.mailetcontext | Error while storing mail.
>> java.lang.UnsupportedOperationException
>>       at java.util.AbstractList.remove(Unknown Source)
>>       at java.util.AbstractList$Itr.remove(Unknown Source)
>>       at org.apache.james.transport.mailets.AbstractRecipientRewriteTableMailet.service(AbstractRecipientRewriteTableMailet.java:78)
>>       at org.apache.james.transport.mailets.LocalDelivery.service(LocalDelivery.java:70)
>>       at org.apache.james.mailetcontainer.impl.camel.CamelProcessor.process(CamelProcessor.java:65)
>> 
>> 
>> 
>> I'm going to go to the previous tag, fix isSenderLocal there.
>> 
>> thought you would like to know, to possibly review changes which have happened between the 10th and the 17th.
>> 
>> 
>> -tim
>> 
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: server-user-unsubscribe@james.apache.org
>> For additional commands, e-mail: server-user-help@james.apache.org
>> 
>> 
> 
> 
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: server-user-unsubscribe@james.apache.org
> For additional commands, e-mail: server-user-help@james.apache.org
> 
> 



---------------------------------------------------------------------
To unsubscribe, e-mail: server-user-unsubscribe@james.apache.org
For additional commands, e-mail: server-user-help@james.apache.org


Re: ok some news about standard mailet

Posted by timprepscius <ti...@piratemail.se>.
ok, I have a feeling that 

the Arras.asList is the culprit of these errors.

the reason being that array I believe is being incorporated into some mail structure and then manipulated.
which is impossible.

-tim

On Jul 17, 2012, at 7:29 PM, timprepscius wrote:

> so, 
> 
> I need to get this change in:
> 
> svn diff
> Index: src/main/java/org/apache/james/transport/matchers/SenderIsLocal.java
> ===================================================================
> --- src/main/java/org/apache/james/transport/matchers/SenderIsLocal.java	(revision 1362688)
> +++ src/main/java/org/apache/james/transport/matchers/SenderIsLocal.java	(working copy)
> @@ -33,6 +33,9 @@
> public class SenderIsLocal extends GenericMatcher {
> 
>     public final Collection<MailAddress> match(Mail mail) throws MessagingException {
> +	if (mail.getSender() == null)
> +		return null;
> +
>         return (getMailetContext().isLocalEmail(mail.getSender())) ? Arrays.asList(mail.getSender()): null;
>     }
> }
> 
> 
> 
> so I used trunk of mailet:
> 
> svn info
> Path: .
> URL: http://svn.apache.org/repos/asf/james/mailet/standard/trunk
> Repository Root: http://svn.apache.org/repos/asf
> Repository UUID: 13f79535-47bb-0310-9956-ffa450edef68
> Revision: 1362688
> Node Kind: directory
> Schedule: normal
> Last Changed Author: eric
> Last Changed Rev: 1362051
> Last Changed Date: 2012-07-16 10:19:21 -0400 (Mon, 16 Jul 2012)
> 
> 
> I built it and deployed it into the lib directory:
> james@blue:~/apache-james/lib$ ls -la *standard*
> lrwxrwxrwx 1 james james     60 Jul 17 19:20 apache-standard-mailets-1.2-20120709.004650-715.jar -> apache-standard-mailets-1.2-20120709.004650-715.jar.original
> -rw------- 1 james james 121046 Jul 10 08:39 apache-standard-mailets-1.2-20120709.004650-715.jar.original
> -rw-r--r-- 1 james james 120541 Jul 17 18:33 apache-standard-mailets-1.2-SNAPSHOT.jar
> 
> 
> ---
> 
> and now james has lots of new errors.
> actually I can't send any mail without a bounce being triggered.
> 
> 
> INFO  19:15:20,910 | james.mailetcontext | Local delivery with ToSenderFolder mailet for mail Mail1342566918279-811ba289-672b-4a55-a2fd-e03499aec883-!383287 with sender …..
> INFO  19:15:20,938 | james.mailetcontext | Error while storing mail.
> java.lang.UnsupportedOperationException
>        at java.util.AbstractList.remove(Unknown Source)
>        at java.util.AbstractList$Itr.remove(Unknown Source)
>        at org.apache.james.transport.mailets.AbstractRecipientRewriteTableMailet.service(AbstractRecipientRewriteTableMailet.java:78)
>        at org.apache.james.transport.mailets.LocalDelivery.service(LocalDelivery.java:70)
>        at org.apache.james.mailetcontainer.impl.camel.CamelProcessor.process(CamelProcessor.java:65)
> 
> 
> 
> I'm going to go to the previous tag, fix isSenderLocal there.
> 
> thought you would like to know, to possibly review changes which have happened between the 10th and the 17th.
> 
> 
> -tim
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: server-user-unsubscribe@james.apache.org
> For additional commands, e-mail: server-user-help@james.apache.org
> 
> 



---------------------------------------------------------------------
To unsubscribe, e-mail: server-user-unsubscribe@james.apache.org
For additional commands, e-mail: server-user-help@james.apache.org