Hudson build is still unstable: james-server-trunk #1006

[Apache Hudson Server]

See 



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

svn commit: r1061141 - in /james/imap/trunk: api/src/main/java/org/apache/james/imap/api/ processor/src/main/java/org/apache/james/imap/processor/

[norman]

Author: norman
Date: Thu Jan 20 07:07:33 2011
New Revision: 1061141

URL: http://svn.apache.org/viewvc?rev=1061141&view=rev
Log:
Hardcoding the free and wildchar. There is no gain to not do so

Modified:

james/imap/trunk/api/src/main/java/org/apache/james/imap/api/ImapConstants.java

james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/AbstractMailboxProcessor.java

james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/LSubProcessor.java

james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/ListProcessor.java

Modified: 
james/imap/trunk/api/src/main/java/org/apache/james/imap/api/ImapConstants.java
URL: 
http://svn.apache.org/viewvc/james/imap/trunk/api/src/main/java/org/apache/james/imap/api/ImapConstants.java?rev=1061141&r1=1061140&r2=1061141&view=diff
==
--- 
james/imap/trunk/api/src/main/java/org/apache/james/imap/api/ImapConstants.java 
(original)
+++ 
james/imap/trunk/api/src/main/java/org/apache/james/imap/api/ImapConstants.java 
Thu Jan 20 07:07:33 2011
@@ -109,9 +109,6 @@ public interface ImapConstants {
 
 public static final String SUPPORTS_IDLE = "IDLE";
 
-public static final char NAMESPACE_PREFIX_CHAR = '#';
-
-public static final String NAMESPACE_PREFIX = 
String.valueOf(NAMESPACE_PREFIX_CHAR);
 
 public static final String INBOX_NAME = "INBOX";
 

Modified: 
james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/AbstractMailboxProcessor.java
URL: 
http://svn.apache.org/viewvc/james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/AbstractMailboxProcessor.java?rev=1061141&r1=1061140&r2=1061141&view=diff
==
--- 
james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/AbstractMailboxProcessor.java
 (original)
+++ 
james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/AbstractMailboxProcessor.java
 Thu Jan 20 07:07:33 2011
@@ -24,7 +24,6 @@ import java.util.Iterator;
 import javax.mail.Flags;
 
 import org.apache.james.imap.api.ImapCommand;
-import org.apache.james.imap.api.ImapConstants;
 import org.apache.james.imap.api.ImapMessage;
 import org.apache.james.imap.api.ImapSessionUtils;
 import org.apache.james.imap.api.display.HumanReadableText;
@@ -288,7 +287,7 @@ abstract public class AbstractMailboxPro
 if (mailboxName == null || mailboxName.length() == 0) {
 return new MailboxPath("", "", "");
 }
-if (mailboxName.charAt(0) == ImapConstants.NAMESPACE_PREFIX_CHAR) {
+if (mailboxName.charAt(0) == MailboxConstants.NAMESPACE_PREFIX_CHAR) {
 int namespaceLength = 
mailboxName.indexOf(mailboxSession.getPathDelimiter());
 if (namespaceLength > -1) {
 namespace = mailboxName.substring(0, namespaceLength);

Modified: 
james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/LSubProcessor.java
URL: 
http://svn.apache.org/viewvc/james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/LSubProcessor.java?rev=1061141&r1=1061140&r2=1061141&view=diff
==
--- 
james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/LSubProcessor.java
 (original)
+++ 
james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/LSubProcessor.java
 Thu Jan 20 07:07:33 2011
@@ -23,7 +23,6 @@ import java.util.ArrayList;
 import java.util.Collection;
 
 import org.apache.james.imap.api.ImapCommand;
-import org.apache.james.imap.api.ImapConstants;
 import org.apache.james.imap.api.ImapMessage;
 import org.apache.james.imap.api.ImapSessionUtils;
 import org.apache.james.imap.api.display.HumanReadableText;
@@ -60,12 +59,12 @@ public class LSubProcessor extends Abstr
 // If the mailboxName is fully qualified, ignore the reference name.
 String finalReferencename = referenceName;
 
-if (mailboxName.charAt(0) == ImapConstants.NAMESPACE_PREFIX_CHAR) {
+if (mailboxName.charAt(0) == MailboxConstants.NAMESPACE_PREFIX_CHAR) {
 finalReferencename = "";
 }
   
 // Is the interpreted (combined) pattern relative?
-boolean isRelative = ((finalReferencename + mailboxName).charAt(0) != 
ImapConstants.NAMESPACE_PREFIX_CHAR);
+boolean isRelative = ((finalReferencename + mailboxName).charAt(0) != 
MailboxConstants.NAMESPACE_PREFIX_CHAR);
 MailboxPath basePath = null;
 if (isRelative) {
 basePath = new MailboxPath(MailboxConstants.USER_NAMESPACE,
@@ -76,7 +75,7 @@ public class LSubProcessor extends Abstr
 }

 
-final MailboxQuery expression = new MailboxQuery(basePath, 
mailboxName, '*', '%', mailboxSession.getPathDelimiter());
+final MailboxQuery expression = new MailboxQue

svn commit: r1061140 - /james/server/trunk/mailbox-adapter/src/main/java/org/apache/james/adapter/mailbox/MailboxManagerManagement.java

[norman]

Author: norman
Date: Thu Jan 20 07:06:52 2011
New Revision: 1061140

URL: http://svn.apache.org/viewvc?rev=1061140&view=rev
Log:
Hardcoding the free and wildchar. There is no gain to not do so

Modified:

james/server/trunk/mailbox-adapter/src/main/java/org/apache/james/adapter/mailbox/MailboxManagerManagement.java

Modified: 
james/server/trunk/mailbox-adapter/src/main/java/org/apache/james/adapter/mailbox/MailboxManagerManagement.java
URL: 
http://svn.apache.org/viewvc/james/server/trunk/mailbox-adapter/src/main/java/org/apache/james/adapter/mailbox/MailboxManagerManagement.java?rev=1061140&r1=1061139&r2=1061140&view=diff
==
--- 
james/server/trunk/mailbox-adapter/src/main/java/org/apache/james/adapter/mailbox/MailboxManagerManagement.java
 (original)
+++ 
james/server/trunk/mailbox-adapter/src/main/java/org/apache/james/adapter/mailbox/MailboxManagerManagement.java
 Thu Jan 20 07:06:52 2011
@@ -65,7 +65,7 @@ public class MailboxManagerManagement ex
 try {
 session = mailboxManager.createSystemSession(username, log);
 mailboxManager.startProcessingRequest(session);
-List mList = mailboxManager.search(new 
MailboxQuery(MailboxPath.inbox(username), "", '*', '%', 
session.getPathDelimiter()), session);
+List mList = mailboxManager.search(new 
MailboxQuery(MailboxPath.inbox(username), "", session.getPathDelimiter()), 
session);
 for (int i = 0; i < mList.size(); i++) {
 mailboxManager.deleteMailbox(mList.get(i).getPath(),session);
 }
@@ -103,7 +103,7 @@ public class MailboxManagerManagement ex
 try {
 session = mailboxManager.createSystemSession(username, log);
 mailboxManager.startProcessingRequest(session);
-List mList = mailboxManager.search(new 
MailboxQuery(MailboxPath.inbox(username), "", '*', '%', 
session.getPathDelimiter()), session);
+List mList = mailboxManager.search(new 
MailboxQuery(MailboxPath.inbox(username), "", session.getPathDelimiter()), 
session);
 for (int i = 0; i < mList.size(); i++) {
 boxes.add(mList.get(i).getPath().getName());
 }



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

svn commit: r1061139 - in /james/mailbox/trunk/api/src: main/java/org/apache/james/mailbox/MailboxConstants.java main/java/org/apache/james/mailbox/MailboxQuery.java test/java/org/apache/james/mailbox

[norman]

Author: norman
Date: Thu Jan 20 07:06:08 2011
New Revision: 1061139

URL: http://svn.apache.org/viewvc?rev=1061139&view=rev
Log:
Hardcoding the free and wildchar. There is no gain to not do so

Modified:

james/mailbox/trunk/api/src/main/java/org/apache/james/mailbox/MailboxConstants.java

james/mailbox/trunk/api/src/main/java/org/apache/james/mailbox/MailboxQuery.java

james/mailbox/trunk/api/src/test/java/org/apache/james/mailbox/MailboxExpressionTest.java

Modified: 
james/mailbox/trunk/api/src/main/java/org/apache/james/mailbox/MailboxConstants.java
URL: 
http://svn.apache.org/viewvc/james/mailbox/trunk/api/src/main/java/org/apache/james/mailbox/MailboxConstants.java?rev=1061139&r1=1061138&r2=1061139&view=diff
==
--- 
james/mailbox/trunk/api/src/main/java/org/apache/james/mailbox/MailboxConstants.java
 (original)
+++ 
james/mailbox/trunk/api/src/main/java/org/apache/james/mailbox/MailboxConstants.java
 Thu Jan 20 07:06:08 2011
@@ -20,11 +20,13 @@
 package org.apache.james.mailbox;
 
 public interface MailboxConstants {
-
+   
+public static final char NAMESPACE_PREFIX_CHAR = '#';
+
 /**
  * The namespace used for store user inboxes
  */
-public static final String USER_NAMESPACE = "#private";
+public static final String USER_NAMESPACE = NAMESPACE_PREFIX_CHAR + 
"private";
 
 /**
  * The default delimiter used to seperated parent/child folders

Modified: 
james/mailbox/trunk/api/src/main/java/org/apache/james/mailbox/MailboxQuery.java
URL: 
http://svn.apache.org/viewvc/james/mailbox/trunk/api/src/main/java/org/apache/james/mailbox/MailboxQuery.java?rev=1061139&r1=1061138&r2=1061139&view=diff
==
--- 
james/mailbox/trunk/api/src/main/java/org/apache/james/mailbox/MailboxQuery.java
 (original)
+++ 
james/mailbox/trunk/api/src/main/java/org/apache/james/mailbox/MailboxQuery.java
 Thu Jan 20 07:06:08 2011
@@ -49,8 +49,7 @@ public class MailboxQuery {
  *matches any sequence of characters up to the next hierarchy
  *delimiter
  */
-public MailboxQuery(final MailboxPath base, final String expression,
-final char freeWildcard, final char localWildcard, final char 
pathDelimiter) {
+public MailboxQuery(final MailboxPath base, final String expression, final 
char pathDelimiter) {
 super();
 this.base = base;
 if (base.getName() == null)
@@ -61,8 +60,8 @@ public class MailboxQuery {
 this.expression = expression;
 }
 expressionLength = this.expression.length();
-this.freeWildcard = freeWildcard;
-this.localWildcard = localWildcard;
+this.freeWildcard = '*';
+this.localWildcard = '%';
 this.pathDelimiter = pathDelimiter;
 }
 

Modified: 
james/mailbox/trunk/api/src/test/java/org/apache/james/mailbox/MailboxExpressionTest.java
URL: 
http://svn.apache.org/viewvc/james/mailbox/trunk/api/src/test/java/org/apache/james/mailbox/MailboxExpressionTest.java?rev=1061139&r1=1061138&r2=1061139&view=diff
==
--- 
james/mailbox/trunk/api/src/test/java/org/apache/james/mailbox/MailboxExpressionTest.java
 (original)
+++ 
james/mailbox/trunk/api/src/test/java/org/apache/james/mailbox/MailboxExpressionTest.java
 Thu Jan 20 07:06:08 2011
@@ -36,7 +36,7 @@ public class MailboxExpressionTest {
 
 
 private MailboxQuery create(String expression) {
-return new MailboxQuery(BASE_PATH, expression, '*', '%', '.');
+return new MailboxQuery(BASE_PATH, expression, '.');
 }
 
 @Test
@@ -56,13 +56,13 @@ public class MailboxExpressionTest {
 
 @Test
 public void testCombinedNameEmptyPart() throws Exception {
-MailboxQuery expression = new MailboxQuery(BASE_PATH, "", '*', '%', 
'.');
+MailboxQuery expression = new MailboxQuery(BASE_PATH, "", '.');
 assertEquals(BASE, expression.getCombinedName());
 }
 
 @Test
 public void testNullCombinedName() throws Exception {
-MailboxQuery expression = new MailboxQuery(new MailboxPath(null, null, 
null), null, '*', '%', '.');
+MailboxQuery expression = new MailboxQuery(new MailboxPath(null, null, 
null), null, '.');
 assertNotNull(expression.getCombinedName());
 }
 
@@ -80,14 +80,14 @@ public class MailboxExpressionTest {
 
 @Test
 public void testCombinedNameBaseEndsWithDelimiter() throws Exception {
-MailboxQuery expression = new MailboxQuery(new MailboxPath(null, null, 
BASE + '.'), PART, '*', '%', '.');
+MailboxQuery expression = new MailboxQuery(new MailboxPath(null, null, 
BASE + '.'), PART, '.');
 assertEquals(BASE + "." + PART, expression.getCombinedName());
 }
 
 @Test
 public void testCombinedNameBaseEndsWithDelimiterPartStarts()
  

[jira] Commented: (JAMES-1184) Modules consolidation

[Norman Maurer (JIRA)]

[ 
https://issues.apache.org/jira/browse/JAMES-1184?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12984048#action_12984048
 ] 

Norman Maurer commented on JAMES-1184:
--

@Stefano

About the "utils":

* - Green classes can be moved to the only library depending on them 
(Lock+XMLResource to data-library, CombinedInputStream to core) 
Makes sense... I also removed CombinedInputStream and just use 
java.io.SequenceInputStream

* - Orange classes can be merged to protocol-library (most of them are JMX 
classes and protocol-library is already the home for most jmx stuff). Fetchmail 
will depend on protocols-library then (instead of util). 
Whats the benefit of this change ? Fetchmail depending on protocols-library 
sounds strange to me

* - Purple classes can be moved to a new library (not sure about the name... 
maybe we can keep util and simply know that libraries do not depends anymore 
from util and we can put util together with libraries in the "layers"). 
Whats the benefit of this change ? I think I can not follow you..

* - Yellow classes can be removed, or left together the Purple ones. 
If we don't use them we should remove them.. 

> Modules consolidation
> -
>
> Key: JAMES-1184
> URL: https://issues.apache.org/jira/browse/JAMES-1184
> Project: JAMES Server
>  Issue Type: Task
>  Components: Build System
>Affects Versions: 3.0-M2
>Reporter: Stefano Bagnara
>Assignee: Stefano Bagnara
> Fix For: 3.0-M3
>
> Attachments: graph-server-utils.gif, graph-server.gif
>
>
> Just opening it to keep track of the proposal to the mailing list.
> 
> - mail-file(3)
> - mail-jcr(1)
> - mail-jdbc(3)
> - user-jpa(4)
> - user-file(2)
> - user-jcr(2)
> - user-jdbc(4)
> - domainlist-jpa(2)
> - domainlist-xml(1)
> I only see benefits and no drawbacks in consolidating the 9 modules
> above to the following 4 modules:
>  persistence-jpa(6) = user-jpa(4)+domainlist-jpa(2)
>  persistence-file(6) = user-file(2)+mail-file(3)+domainlist-xml(1)
>  persistence-jcr(3) = user-jcr(2)+mail-jcr(1)
>  persistence-jdbc(7) = user-jdbc(4)+mail-jdbc(3)
> - mail-library(1)
> - domainlist-library(2)
> - user-library(9)
> We could consolidate them into a persistence-library module (from 3 to
> 1 module).
>  persistence-library(12) =
> mail-library(1)+domainlist-library(2)+user-library(9)
> - queue-library(2)
> - queue-jms(5)
> We could merge queue-library to queue-jms (a library module makes
> sense when we do something useful.. if it is trivial stuff it can be
> duplicated or moved to the api layer).
>  queue-jms(7) = queue-jms(5)+queue-library(2)
> -
> It seems there is agreement on the merging. We still have to decide whether 
> to name the modules "persistence-${type}" or simply "${type}" (or maybe 
> "data-${type}")

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] Commented: (JAMES-1184) Modules consolidation

[Norman Maurer (JIRA)]

[ 
https://issues.apache.org/jira/browse/JAMES-1184?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12984045#action_12984045
 ] 

Norman Maurer commented on JAMES-1184:
--

@Stefano:

I never used such a "nested" module structure as you proposed. I'm also not 
sure its really common and easy to understand by devs.. 



> Modules consolidation
> -
>
> Key: JAMES-1184
> URL: https://issues.apache.org/jira/browse/JAMES-1184
> Project: JAMES Server
>  Issue Type: Task
>  Components: Build System
>Affects Versions: 3.0-M2
>Reporter: Stefano Bagnara
>Assignee: Stefano Bagnara
> Fix For: 3.0-M3
>
> Attachments: graph-server-utils.gif, graph-server.gif
>
>
> Just opening it to keep track of the proposal to the mailing list.
> 
> - mail-file(3)
> - mail-jcr(1)
> - mail-jdbc(3)
> - user-jpa(4)
> - user-file(2)
> - user-jcr(2)
> - user-jdbc(4)
> - domainlist-jpa(2)
> - domainlist-xml(1)
> I only see benefits and no drawbacks in consolidating the 9 modules
> above to the following 4 modules:
>  persistence-jpa(6) = user-jpa(4)+domainlist-jpa(2)
>  persistence-file(6) = user-file(2)+mail-file(3)+domainlist-xml(1)
>  persistence-jcr(3) = user-jcr(2)+mail-jcr(1)
>  persistence-jdbc(7) = user-jdbc(4)+mail-jdbc(3)
> - mail-library(1)
> - domainlist-library(2)
> - user-library(9)
> We could consolidate them into a persistence-library module (from 3 to
> 1 module).
>  persistence-library(12) =
> mail-library(1)+domainlist-library(2)+user-library(9)
> - queue-library(2)
> - queue-jms(5)
> We could merge queue-library to queue-jms (a library module makes
> sense when we do something useful.. if it is trivial stuff it can be
> duplicated or moved to the api layer).
>  queue-jms(7) = queue-jms(5)+queue-library(2)
> -
> It seems there is agreement on the merging. We still have to decide whether 
> to name the modules "persistence-${type}" or simply "${type}" (or maybe 
> "data-${type}")

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

svn commit: r1061131 - in /james/server/trunk: core/src/main/java/org/apache/james/core/MimeMessageWrapper.java util/src/main/java/org/apache/james/util/stream/CombinedInputStream.java

[norman]

Author: norman
Date: Thu Jan 20 06:37:44 2011
New Revision: 1061131

URL: http://svn.apache.org/viewvc?rev=1061131&view=rev
Log:
Remove CombinedInputStream and use java.io.SequenceInputStream

Removed:

james/server/trunk/util/src/main/java/org/apache/james/util/stream/CombinedInputStream.java
Modified:

james/server/trunk/core/src/main/java/org/apache/james/core/MimeMessageWrapper.java

Modified: 
james/server/trunk/core/src/main/java/org/apache/james/core/MimeMessageWrapper.java
URL: 
http://svn.apache.org/viewvc/james/server/trunk/core/src/main/java/org/apache/james/core/MimeMessageWrapper.java?rev=1061131&r1=1061130&r2=1061131&view=diff
==
--- 
james/server/trunk/core/src/main/java/org/apache/james/core/MimeMessageWrapper.java
 (original)
+++ 
james/server/trunk/core/src/main/java/org/apache/james/core/MimeMessageWrapper.java
 Thu Jan 20 06:37:44 2011
@@ -28,6 +28,7 @@ import java.io.InputStream;
 import java.io.InputStreamReader;
 import java.io.LineNumberReader;
 import java.io.OutputStream;
+import java.io.SequenceInputStream;
 import java.util.Enumeration;
 import java.util.UUID;
 
@@ -41,7 +42,6 @@ import javax.mail.util.SharedByteArrayIn
 import org.apache.commons.io.IOUtils;
 import org.apache.james.lifecycle.api.Disposable;
 import org.apache.james.lifecycle.api.LifecycleUtil;
-import org.apache.james.util.stream.CombinedInputStream;
 
 /**
  * This object wraps a MimeMessage, only loading the underlying MimeMessage
@@ -645,7 +645,7 @@ public class MimeMessageWrapper
 new MailHeaders(in);
 
 // now construct the new stream using the in memory 
headers and the body from the original source
-return new CombinedInputStream(new InputStream[]{new 
InternetHeadersInputStream(getAllHeaderLines()), in});
+return new SequenceInputStream(new 
InternetHeadersInputStream(getAllHeaderLines()), in);
 } else {
 // the body was changed so we have no other solution to 
copy it into memory first :(
 ByteArrayOutputStream out = new ByteArrayOutputStream();



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] Commented: (JAMES-1184) Modules consolidation

[Stefano Bagnara (JIRA)]

[ 
https://issues.apache.org/jira/browse/JAMES-1184?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12983921#action_12983921
 ] 

Stefano Bagnara commented on JAMES-1184:


About utils there is a plan to avoid having it at the api level.

In this plan:
- Green classes can be moved to the only library depending on them 
(Lock+XMLResource to data-library, CombinedInputStream to core)
- Orange classes can be merged to protocol-library (most of them are JMX 
classes and protocol-library is already the home for most jmx stuff). Fetchmail 
will depend on protocols-library then (instead of util).
- Purple classes can be moved to a new library (not sure about the name... 
maybe we can keep util and simply know that libraries do not depends anymore 
from util and we can put util together with libraries in the "layers").
- Yellow classes can be removed, or left together the Purple ones.

This does not reduce the number of modules but reduce the size of the util 
module and bring it at the "library" layer so in the api layer we just have 
real apis.

As per the "multilevel" comment I'm just writing this as loud-thinking... just 
a place to scratchpad some thought as I have no time to do better analysis of 
pro/cons of both changes now


> Modules consolidation
> -
>
> Key: JAMES-1184
> URL: https://issues.apache.org/jira/browse/JAMES-1184
> Project: JAMES Server
>  Issue Type: Task
>  Components: Build System
>Affects Versions: 3.0-M2
>Reporter: Stefano Bagnara
>Assignee: Stefano Bagnara
> Fix For: 3.0-M3
>
> Attachments: graph-server-utils.gif, graph-server.gif
>
>
> Just opening it to keep track of the proposal to the mailing list.
> 
> - mail-file(3)
> - mail-jcr(1)
> - mail-jdbc(3)
> - user-jpa(4)
> - user-file(2)
> - user-jcr(2)
> - user-jdbc(4)
> - domainlist-jpa(2)
> - domainlist-xml(1)
> I only see benefits and no drawbacks in consolidating the 9 modules
> above to the following 4 modules:
>  persistence-jpa(6) = user-jpa(4)+domainlist-jpa(2)
>  persistence-file(6) = user-file(2)+mail-file(3)+domainlist-xml(1)
>  persistence-jcr(3) = user-jcr(2)+mail-jcr(1)
>  persistence-jdbc(7) = user-jdbc(4)+mail-jdbc(3)
> - mail-library(1)
> - domainlist-library(2)
> - user-library(9)
> We could consolidate them into a persistence-library module (from 3 to
> 1 module).
>  persistence-library(12) =
> mail-library(1)+domainlist-library(2)+user-library(9)
> - queue-library(2)
> - queue-jms(5)
> We could merge queue-library to queue-jms (a library module makes
> sense when we do something useful.. if it is trivial stuff it can be
> duplicated or moved to the api layer).
>  queue-jms(7) = queue-jms(5)+queue-library(2)
> -
> It seems there is agreement on the merging. We still have to decide whether 
> to name the modules "persistence-${type}" or simply "${type}" (or maybe 
> "data-${type}")

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] Updated: (JAMES-1184) Modules consolidation

[Stefano Bagnara (JIRA)]

 [ 
https://issues.apache.org/jira/browse/JAMES-1184?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Stefano Bagnara updated JAMES-1184:
---

Attachment: graph-server-utils.gif

I took util module into consideration and checked what module depends on what 
class.

> Modules consolidation
> -
>
> Key: JAMES-1184
> URL: https://issues.apache.org/jira/browse/JAMES-1184
> Project: JAMES Server
>  Issue Type: Task
>  Components: Build System
>Affects Versions: 3.0-M2
>Reporter: Stefano Bagnara
>Assignee: Stefano Bagnara
> Fix For: 3.0-M3
>
> Attachments: graph-server-utils.gif, graph-server.gif
>
>
> Just opening it to keep track of the proposal to the mailing list.
> 
> - mail-file(3)
> - mail-jcr(1)
> - mail-jdbc(3)
> - user-jpa(4)
> - user-file(2)
> - user-jcr(2)
> - user-jdbc(4)
> - domainlist-jpa(2)
> - domainlist-xml(1)
> I only see benefits and no drawbacks in consolidating the 9 modules
> above to the following 4 modules:
>  persistence-jpa(6) = user-jpa(4)+domainlist-jpa(2)
>  persistence-file(6) = user-file(2)+mail-file(3)+domainlist-xml(1)
>  persistence-jcr(3) = user-jcr(2)+mail-jcr(1)
>  persistence-jdbc(7) = user-jdbc(4)+mail-jdbc(3)
> - mail-library(1)
> - domainlist-library(2)
> - user-library(9)
> We could consolidate them into a persistence-library module (from 3 to
> 1 module).
>  persistence-library(12) =
> mail-library(1)+domainlist-library(2)+user-library(9)
> - queue-library(2)
> - queue-jms(5)
> We could merge queue-library to queue-jms (a library module makes
> sense when we do something useful.. if it is trivial stuff it can be
> duplicated or moved to the api layer).
>  queue-jms(7) = queue-jms(5)+queue-library(2)
> -
> It seems there is agreement on the merging. We still have to decide whether 
> to name the modules "persistence-${type}" or simply "${type}" (or maybe 
> "data-${type}")

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] Commented: (JAMES-1184) Modules consolidation

[Stefano Bagnara (JIRA)]

[ 
https://issues.apache.org/jira/browse/JAMES-1184?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12983893#action_12983893
 ] 

Stefano Bagnara commented on JAMES-1184:


OK, applied step2 changes.
Norman: I didn't merge mailetcontainer-library to mailetcontainer-camel mainly 
because container-spring have a direct dependency to mailetcontainer-library 
and this would have moved up the container-spring module. I think it is safe to 
not have direct "function" dependencies (I mean in source classes) in the 
container-spring module, so for the moment I preferred to limit the change.

I added some color to identify api+util, libraries, "tcp services", "data-api 
implementations" and remaining functions.

I was wondering if moving to a multilevel structure would better reflect the 
layers and help users to find what they need:
---
api
   dnsservice-api
   filesystem-api
   lifecycle-api
   queue-api
   data-api
   util
libs
   core
   data-library
   dnsservice-library
   mailetcontainer-library
   protocols-library
data
   file
   jcr
   jdbc
   jpa
   queue-activemq as activemq
   queue-jms as jms
   ldap
protocols (or services)
   imapserver
   lmtpserver
   pop3server
   smtpserver
   fetchmail
others (container? better name???)
   cli
   container-spring
   dnsservice-dnsjava
   mailbox-adapter
   mailetcontainer-camel
   mailets
   osgi
-

The reading key is:
- Most developers will not change api or libraries modules because only core 
developers need to change them.
- Most developers will be able to tweak james by just working on functions.
- If devs need special integration then they will probably have to work in the 
"data" group and tweak the specific persistence adapter or simply add a new 
custom module.
- If devs need to tweak protocols behaviours then they will work in protocols 
and maybe on external projects (james-protocols and james-imap).

I'm still thinking more about this, but I wanted to share this thought as it is 
related to code/modules organization.


> Modules consolidation
> -
>
> Key: JAMES-1184
> URL: https://issues.apache.org/jira/browse/JAMES-1184
> Project: JAMES Server
>  Issue Type: Task
>  Components: Build System
>Affects Versions: 3.0-M2
>Reporter: Stefano Bagnara
>Assignee: Stefano Bagnara
> Fix For: 3.0-M3
>
> Attachments: graph-server.gif
>
>
> Just opening it to keep track of the proposal to the mailing list.
> 
> - mail-file(3)
> - mail-jcr(1)
> - mail-jdbc(3)
> - user-jpa(4)
> - user-file(2)
> - user-jcr(2)
> - user-jdbc(4)
> - domainlist-jpa(2)
> - domainlist-xml(1)
> I only see benefits and no drawbacks in consolidating the 9 modules
> above to the following 4 modules:
>  persistence-jpa(6) = user-jpa(4)+domainlist-jpa(2)
>  persistence-file(6) = user-file(2)+mail-file(3)+domainlist-xml(1)
>  persistence-jcr(3) = user-jcr(2)+mail-jcr(1)
>  persistence-jdbc(7) = user-jdbc(4)+mail-jdbc(3)
> - mail-library(1)
> - domainlist-library(2)
> - user-library(9)
> We could consolidate them into a persistence-library module (from 3 to
> 1 module).
>  persistence-library(12) =
> mail-library(1)+domainlist-library(2)+user-library(9)
> - queue-library(2)
> - queue-jms(5)
> We could merge queue-library to queue-jms (a library module makes
> sense when we do something useful.. if it is trivial stuff it can be
> duplicated or moved to the api layer).
>  queue-jms(7) = queue-jms(5)+queue-library(2)
> -
> It seems there is agreement on the merging. We still have to decide whether 
> to name the modules "persistence-${type}" or simply "${type}" (or maybe 
> "data-${type}")

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] Updated: (JAMES-1184) Modules consolidation

[Stefano Bagnara (JIRA)]

 [ 
https://issues.apache.org/jira/browse/JAMES-1184?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Stefano Bagnara updated JAMES-1184:
---

Attachment: graph-server.gif

Updated dependency graph

> Modules consolidation
> -
>
> Key: JAMES-1184
> URL: https://issues.apache.org/jira/browse/JAMES-1184
> Project: JAMES Server
>  Issue Type: Task
>  Components: Build System
>Affects Versions: 3.0-M2
>Reporter: Stefano Bagnara
>Assignee: Stefano Bagnara
> Fix For: 3.0-M3
>
> Attachments: graph-server.gif
>
>
> Just opening it to keep track of the proposal to the mailing list.
> 
> - mail-file(3)
> - mail-jcr(1)
> - mail-jdbc(3)
> - user-jpa(4)
> - user-file(2)
> - user-jcr(2)
> - user-jdbc(4)
> - domainlist-jpa(2)
> - domainlist-xml(1)
> I only see benefits and no drawbacks in consolidating the 9 modules
> above to the following 4 modules:
>  persistence-jpa(6) = user-jpa(4)+domainlist-jpa(2)
>  persistence-file(6) = user-file(2)+mail-file(3)+domainlist-xml(1)
>  persistence-jcr(3) = user-jcr(2)+mail-jcr(1)
>  persistence-jdbc(7) = user-jdbc(4)+mail-jdbc(3)
> - mail-library(1)
> - domainlist-library(2)
> - user-library(9)
> We could consolidate them into a persistence-library module (from 3 to
> 1 module).
>  persistence-library(12) =
> mail-library(1)+domainlist-library(2)+user-library(9)
> - queue-library(2)
> - queue-jms(5)
> We could merge queue-library to queue-jms (a library module makes
> sense when we do something useful.. if it is trivial stuff it can be
> duplicated or moved to the api layer).
>  queue-jms(7) = queue-jms(5)+queue-library(2)
> -
> It seems there is agreement on the merging. We still have to decide whether 
> to name the modules "persistence-${type}" or simply "${type}" (or maybe 
> "data-${type}")

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

svn commit: r1060991 - /james/server/trunk/container-spring/pom.xml

[bago]

Author: bago
Date: Wed Jan 19 21:13:35 2011
New Revision: 1060991

URL: http://svn.apache.org/viewvc?rev=1060991&view=rev
Log:
second step of modules consolidation (JAMES-1184)

Modified:
james/server/trunk/container-spring/pom.xml

Modified: james/server/trunk/container-spring/pom.xml
URL: 
http://svn.apache.org/viewvc/james/server/trunk/container-spring/pom.xml?rev=1060991&r1=1060990&r2=1060991&view=diff
==
--- james/server/trunk/container-spring/pom.xml (original)
+++ james/server/trunk/container-spring/pom.xml Wed Jan 19 21:13:35 2011
@@ -506,7 +506,6 @@
 
   org.apache.james
   james-server-mailetcontainer-library
-  runtime
 
 
   org.apache.james



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

svn commit: r1060984 - /james/server/trunk/ldap/

[bago]

Author: bago
Date: Wed Jan 19 20:52:27 2011
New Revision: 1060984

URL: http://svn.apache.org/viewvc?rev=1060984&view=rev
Log:
second step of modules consolidation (JAMES-1184)

Modified:
james/server/trunk/ldap/   (props changed)

Propchange: james/server/trunk/ldap/
--
--- svn:ignore (original)
+++ svn:ignore Wed Jan 19 20:52:27 2011
@@ -3,3 +3,5 @@
 .project
 
 target
+
+.classpath



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

svn commit: r1060982 - in /james/server/trunk: data-api/ ldap/

[bago]

Author: bago
Date: Wed Jan 19 20:46:37 2011
New Revision: 1060982

URL: http://svn.apache.org/viewvc?rev=1060982&view=rev
Log:
second step of modules consolidation (JAMES-1184)

Modified:
james/server/trunk/data-api/   (props changed)
james/server/trunk/ldap/   (props changed)

Propchange: james/server/trunk/data-api/
--
--- svn:ignore (added)
+++ svn:ignore Wed Jan 19 20:46:37 2011
@@ -0,0 +1,7 @@
+.classpath
+
+.settings
+
+.project
+
+target

Propchange: james/server/trunk/ldap/
--
--- svn:ignore (added)
+++ svn:ignore Wed Jan 19 20:46:37 2011
@@ -0,0 +1,5 @@
+.settings
+
+.project
+
+target



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

svn commit: r1060981 - in /james/server/trunk: domainlist-api/ mail-api/ mailetcontainer-api/ repository-api/ user-api/ user-ldap/

[bago]

Author: bago
Date: Wed Jan 19 20:44:08 2011
New Revision: 1060981

URL: http://svn.apache.org/viewvc?rev=1060981&view=rev
Log:
second step of modules consolidation (JAMES-1184)

Removed:
james/server/trunk/domainlist-api/
james/server/trunk/mail-api/
james/server/trunk/mailetcontainer-api/
james/server/trunk/repository-api/
james/server/trunk/user-api/
james/server/trunk/user-ldap/


-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

svn commit: r1060970 - in /james/server/trunk: ./ cli/ container-spring/ data-api/ data-library/ fetchmail/ file/ jcr/ jdbc/ jpa/ ldap/ lmtpserver/ mailbox-adapter/ mailetcontainer-camel/ mailetcontai

[bago]

Author: bago
Date: Wed Jan 19 20:27:47 2011
New Revision: 1060970

URL: http://svn.apache.org/viewvc?rev=1060970&view=rev
Log:
second step of modules consolidation (JAMES-1184)

Modified:
james/server/trunk/cli/pom.xml
james/server/trunk/container-spring/pom.xml
james/server/trunk/data-api/pom.xml
james/server/trunk/data-library/pom.xml
james/server/trunk/fetchmail/pom.xml
james/server/trunk/file/pom.xml
james/server/trunk/jcr/pom.xml
james/server/trunk/jdbc/pom.xml
james/server/trunk/jpa/pom.xml
james/server/trunk/ldap/pom.xml
james/server/trunk/lmtpserver/pom.xml
james/server/trunk/mailbox-adapter/pom.xml
james/server/trunk/mailetcontainer-camel/pom.xml
james/server/trunk/mailetcontainer-library/pom.xml
james/server/trunk/mailets/pom.xml
james/server/trunk/pom.xml
james/server/trunk/smtpserver/pom.xml

Modified: james/server/trunk/cli/pom.xml
URL: 
http://svn.apache.org/viewvc/james/server/trunk/cli/pom.xml?rev=1060970&r1=1060969&r2=1060970&view=diff
==
--- james/server/trunk/cli/pom.xml (original)
+++ james/server/trunk/cli/pom.xml Wed Jan 19 20:27:47 2011
@@ -64,11 +64,7 @@
   
 
   org.apache.james
-  james-server-domainlist-api
-
-
-  org.apache.james
-  james-server-user-api
+  james-server-data-api
 
 
   commons-cli

Modified: james/server/trunk/container-spring/pom.xml
URL: 
http://svn.apache.org/viewvc/james/server/trunk/container-spring/pom.xml?rev=1060970&r1=1060969&r2=1060970&view=diff
==
--- james/server/trunk/container-spring/pom.xml (original)
+++ james/server/trunk/container-spring/pom.xml Wed Jan 19 20:27:47 2011
@@ -439,10 +439,6 @@
 
 
   org.apache.james
-  james-server-mailetcontainer-api
-
-
-  org.apache.james
   apache-mailet
 
 
@@ -545,21 +541,17 @@
 
 
   org.apache.james
-  james-server-user-api
-
-
-  org.apache.james
   james-server-data-library
   runtime
 
 
   org.apache.james
-  james-server-user-ldap
+  james-server-ldap
   runtime
 
 
   org.apache.james
-  james-server-domainlist-api
+  james-server-data-api
   compile
 
 
@@ -634,10 +626,6 @@
 
 
   org.apache.james
-  james-server-mail-api
-
-
-  org.apache.james
   apache-james-mailbox-tool
 
 

Modified: james/server/trunk/data-api/pom.xml
URL: 
http://svn.apache.org/viewvc/james/server/trunk/data-api/pom.xml?rev=1060970&r1=1060969&r2=1060970&view=diff
==
--- james/server/trunk/data-api/pom.xml (original)
+++ james/server/trunk/data-api/pom.xml Wed Jan 19 20:27:47 2011
@@ -25,8 +25,8 @@
   
   4.0.0
   org.apache.james
-  james-server-user-api
-  Apache James Server User API
+  james-server-data-api
+  Apache James Server Data API
   
 
 
@@ -68,5 +68,9 @@
   org.apache.james
   apache-mailet
 
+
+  ${javax.mail.groupId}
+  ${javax.mail.artifactId}
+ 
   
 
\ No newline at end of file

Modified: james/server/trunk/data-library/pom.xml
URL: 
http://svn.apache.org/viewvc/james/server/trunk/data-library/pom.xml?rev=1060970&r1=1060969&r2=1060970&view=diff
==
--- james/server/trunk/data-library/pom.xml (original)
+++ james/server/trunk/data-library/pom.xml Wed Jan 19 20:27:47 2011
@@ -38,15 +38,7 @@
 
 
   org.apache.james
-  james-server-user-api
-
-
-  org.apache.james
-  james-server-repository-api
-
-
-  org.apache.james
-  james-server-domainlist-api
+  james-server-data-api
 
 
   org.apache.james
@@ -54,10 +46,6 @@
 
 
   org.apache.james
-  james-server-mail-api
-
-
-  org.apache.james
   james-server-filesystem-api
 
 

Modified: james/server/trunk/fetchmail/pom.xml
URL: 
http://svn.apache.org/viewvc/james/server/trunk/fetchmail/pom.xml?rev=1060970&r1=1060969&r2=1060970&view=diff
==
--- james/server/trunk/fetchmail/pom.xml (original)
+++ james/server/trunk/fetchmail/pom.xml Wed Jan 19 20:27:47 2011
@@ -42,11 +42,7 @@
 
 
   org.apache.james
-  james-server-domainlist-api
-
-
-  org.apache.james
-  james-server-user-api
+  james-server-data-api
 
 
   org.apache.james

Modified: james/server/trunk/file/pom.xml
URL: 
http://svn.apache.org/viewvc/james/server/trunk/file/pom.xml?rev=1060970&r1=1060969&r2=1060970&view=diff
==
--- james/server/trunk/file/pom.xml (original)
+++ james/server/trunk/file/pom.x

svn commit: r1060954 - in /james/server/trunk/ldap: LICENSE.txt NOTICE.txt pom.xml src/

[bago]

Author: bago
Date: Wed Jan 19 20:05:30 2011
New Revision: 1060954

URL: http://svn.apache.org/viewvc?rev=1060954&view=rev
Log:
second step of modules consolidation (JAMES-1184)

Added:
james/server/trunk/ldap/LICENSE.txt   (props changed)
  - copied unchanged from r1060953, james/server/trunk/user-ldap/LICENSE.txt
james/server/trunk/ldap/NOTICE.txt   (props changed)
  - copied unchanged from r1060953, james/server/trunk/user-ldap/NOTICE.txt
james/server/trunk/ldap/pom.xml   (props changed)
  - copied unchanged from r1060953, james/server/trunk/user-ldap/pom.xml
james/server/trunk/ldap/src/   (props changed)
  - copied from r1060953, james/server/trunk/user-ldap/src/

Propchange: james/server/trunk/ldap/LICENSE.txt
--
--- svn:mergeinfo (added)
+++ svn:mergeinfo Wed Jan 19 20:05:30 2011
@@ -0,0 +1,2 @@
+/james/server/sandbox/active/mina_experiments/user-ldap/LICENSE.txt:824613-825114
+/james/server/sandbox/active/pure_spring_deployment/user-ldap/LICENSE.txt:893986-896410

Propchange: james/server/trunk/ldap/NOTICE.txt
--
--- svn:mergeinfo (added)
+++ svn:mergeinfo Wed Jan 19 20:05:30 2011
@@ -0,0 +1,2 @@
+/james/server/sandbox/active/mina_experiments/user-ldap/NOTICE.txt:824613-825114
+/james/server/sandbox/active/pure_spring_deployment/user-ldap/NOTICE.txt:893986-896410

Propchange: james/server/trunk/ldap/pom.xml
--
--- svn:mergeinfo (added)
+++ svn:mergeinfo Wed Jan 19 20:05:30 2011
@@ -0,0 +1,2 @@
+/james/server/sandbox/active/mina_experiments/user-ldap/pom.xml:824613-825114
+/james/server/sandbox/active/pure_spring_deployment/user-ldap/pom.xml:893986-896410

Propchange: james/server/trunk/ldap/src/
--
--- svn:mergeinfo (added)
+++ svn:mergeinfo Wed Jan 19 20:05:30 2011
@@ -0,0 +1,2 @@
+/james/server/sandbox/active/mina_experiments/user-ldap/src:824613-825114
+/james/server/sandbox/active/pure_spring_deployment/user-ldap/src:893986-896410



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

svn commit: r1060953 - /james/server/trunk/ldap/

[bago]

Author: bago
Date: Wed Jan 19 20:04:04 2011
New Revision: 1060953

URL: http://svn.apache.org/viewvc?rev=1060953&view=rev
Log:
second step of modules consolidation (JAMES-1184)

Added:
james/server/trunk/ldap/


-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

svn commit: r1060952 - /james/server/trunk/data-api/src/main/java/org/apache/james/mailrepository/

[bago]

Author: bago
Date: Wed Jan 19 20:03:18 2011
New Revision: 1060952

URL: http://svn.apache.org/viewvc?rev=1060952&view=rev
Log:
second step of modules consolidation (JAMES-1184)

Added:
james/server/trunk/data-api/src/main/java/org/apache/james/mailrepository/  
 (props changed)
  - copied from r1060951, 
james/server/trunk/mail-api/src/main/java/org/apache/james/mailrepository/

Propchange: 
james/server/trunk/data-api/src/main/java/org/apache/james/mailrepository/
--
--- svn:mergeinfo (added)
+++ svn:mergeinfo Wed Jan 19 20:03:18 2011
@@ -0,0 +1,2 @@
+/james/server/sandbox/active/mina_experiments/mail-api/src/main/java/org/apache/james/mailrepository:824613-825114
+/james/server/sandbox/active/pure_spring_deployment/mail-api/src/main/java/org/apache/james/mailrepository:893986-896410



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

svn commit: r1060951 - /james/server/trunk/data-api/src/test/

[bago]

Author: bago
Date: Wed Jan 19 20:02:19 2011
New Revision: 1060951

URL: http://svn.apache.org/viewvc?rev=1060951&view=rev
Log:
second step of modules consolidation (JAMES-1184)

Added:
james/server/trunk/data-api/src/test/   (props changed)
  - copied from r1060950, james/server/trunk/domainlist-api/src/test/

Propchange: james/server/trunk/data-api/src/test/
--
--- svn:mergeinfo (added)
+++ svn:mergeinfo Wed Jan 19 20:02:19 2011
@@ -0,0 +1,2 @@
+/james/server/sandbox/active/mina_experiments/domainlist-api/src/test:824613-825114
+/james/server/sandbox/active/pure_spring_deployment/domainlist-api/src/test:893986-896410



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

svn commit: r1060950 - /james/server/trunk/data-api/src/main/java/org/apache/james/domainlist/

[bago]

Author: bago
Date: Wed Jan 19 20:02:06 2011
New Revision: 1060950

URL: http://svn.apache.org/viewvc?rev=1060950&view=rev
Log:
second step of modules consolidation (JAMES-1184)

Added:
james/server/trunk/data-api/src/main/java/org/apache/james/domainlist/   
(props changed)
  - copied from r1060949, 
james/server/trunk/domainlist-api/src/main/java/org/apache/james/domainlist/

Propchange: 
james/server/trunk/data-api/src/main/java/org/apache/james/domainlist/
--
--- svn:mergeinfo (added)
+++ svn:mergeinfo Wed Jan 19 20:02:06 2011
@@ -0,0 +1,2 @@
+/james/server/sandbox/active/mina_experiments/domainlist-api/src/main/java/org/apache/james/domainlist:824613-825114
+/james/server/sandbox/active/pure_spring_deployment/domainlist-api/src/main/java/org/apache/james/domainlist:893986-896410



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

svn commit: r1060946 - in /james/server/trunk/data-api: LICENSE.txt NOTICE.txt pom.xml src/

[bago]

Author: bago
Date: Wed Jan 19 20:00:25 2011
New Revision: 1060946

URL: http://svn.apache.org/viewvc?rev=1060946&view=rev
Log:
second step of modules consolidation (JAMES-1184)

Added:
james/server/trunk/data-api/LICENSE.txt   (props changed)
  - copied unchanged from r1060945, james/server/trunk/user-api/LICENSE.txt
james/server/trunk/data-api/NOTICE.txt   (props changed)
  - copied unchanged from r1060945, james/server/trunk/user-api/NOTICE.txt
james/server/trunk/data-api/pom.xml   (props changed)
  - copied unchanged from r1060945, james/server/trunk/user-api/pom.xml
james/server/trunk/data-api/src/   (props changed)
  - copied from r1060945, james/server/trunk/user-api/src/

Propchange: james/server/trunk/data-api/LICENSE.txt
--
--- svn:mergeinfo (added)
+++ svn:mergeinfo Wed Jan 19 20:00:25 2011
@@ -0,0 +1,2 @@
+/james/server/sandbox/active/mina_experiments/user-api/LICENSE.txt:824613-825114
+/james/server/sandbox/active/pure_spring_deployment/user-api/LICENSE.txt:893986-896410

Propchange: james/server/trunk/data-api/NOTICE.txt
--
--- svn:mergeinfo (added)
+++ svn:mergeinfo Wed Jan 19 20:00:25 2011
@@ -0,0 +1,2 @@
+/james/server/sandbox/active/mina_experiments/user-api/NOTICE.txt:824613-825114
+/james/server/sandbox/active/pure_spring_deployment/user-api/NOTICE.txt:893986-896410

Propchange: james/server/trunk/data-api/pom.xml
--
svn:eol-style = native

Propchange: james/server/trunk/data-api/pom.xml
--
--- svn:mergeinfo (added)
+++ svn:mergeinfo Wed Jan 19 20:00:25 2011
@@ -0,0 +1,2 @@
+/james/server/sandbox/active/mina_experiments/user-api/pom.xml:824613-825114
+/james/server/sandbox/active/pure_spring_deployment/user-api/pom.xml:893986-896410

Propchange: james/server/trunk/data-api/pom.xml
--
svn:mime-type = text/plain

Propchange: james/server/trunk/data-api/src/
--
--- svn:mergeinfo (added)
+++ svn:mergeinfo Wed Jan 19 20:00:25 2011
@@ -0,0 +1,2 @@
+/james/server/sandbox/active/mina_experiments/user-api/src:824613-825114
+/james/server/sandbox/active/pure_spring_deployment/user-api/src:893986-896410



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

svn commit: r1060944 - /james/server/trunk/data-api/

[bago]

Author: bago
Date: Wed Jan 19 19:59:24 2011
New Revision: 1060944

URL: http://svn.apache.org/viewvc?rev=1060944&view=rev
Log:
second step of modules consolidation (JAMES-1184)

Added:
james/server/trunk/data-api/


-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

svn commit: r1060943 - /james/server/trunk/mailetcontainer-library/src/test/java/org/apache/james/mailetcontainer/api/

[bago]

Author: bago
Date: Wed Jan 19 19:59:02 2011
New Revision: 1060943

URL: http://svn.apache.org/viewvc?rev=1060943&view=rev
Log:
second step of modules consolidation (JAMES-1184)

Added:

james/server/trunk/mailetcontainer-library/src/test/java/org/apache/james/mailetcontainer/api/
   (props changed)
  - copied from r1060942, 
james/server/trunk/mailetcontainer-api/src/test/java/org/apache/james/mailetcontainer/api/

Propchange: 
james/server/trunk/mailetcontainer-library/src/test/java/org/apache/james/mailetcontainer/api/
--
--- svn:mergeinfo (added)
+++ svn:mergeinfo Wed Jan 19 19:59:02 2011
@@ -0,0 +1,2 @@
+/james/server/sandbox/active/mina_experiments/mailetcontainer-api/src/test/java/org/apache/james/mailetcontainer/api:824613-825114
+/james/server/sandbox/active/pure_spring_deployment/mailetcontainer-api/src/test/java/org/apache/james/mailetcontainer/api:893986-896410



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

svn commit: r1060942 - /james/server/trunk/mailetcontainer-library/src/main/java/org/apache/james/mailetcontainer/api/

[bago]

Author: bago
Date: Wed Jan 19 19:58:52 2011
New Revision: 1060942

URL: http://svn.apache.org/viewvc?rev=1060942&view=rev
Log:
second step of modules consolidation (JAMES-1184)

Added:

james/server/trunk/mailetcontainer-library/src/main/java/org/apache/james/mailetcontainer/api/
   (props changed)
  - copied from r1060941, 
james/server/trunk/mailetcontainer-api/src/main/java/org/apache/james/mailetcontainer/api/

Propchange: 
james/server/trunk/mailetcontainer-library/src/main/java/org/apache/james/mailetcontainer/api/
--
--- svn:mergeinfo (added)
+++ svn:mergeinfo Wed Jan 19 19:58:52 2011
@@ -0,0 +1,2 @@
+/james/server/sandbox/active/mina_experiments/mailetcontainer-api/src/main/java/org/apache/james/mailetcontainer/api:824613-825114
+/james/server/sandbox/active/pure_spring_deployment/mailetcontainer-api/src/main/java/org/apache/james/mailetcontainer/api:893986-896410



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

svn commit: r1060940 - /james/server/trunk/data-library/src/main/java/org/apache/james/repository/api/

[bago]

Author: bago
Date: Wed Jan 19 19:57:43 2011
New Revision: 1060940

URL: http://svn.apache.org/viewvc?rev=1060940&view=rev
Log:
second step of modules consolidation (JAMES-1184)

Added:

james/server/trunk/data-library/src/main/java/org/apache/james/repository/api/  
 (props changed)
  - copied from r1060939, 
james/server/trunk/repository-api/src/main/java/org/apache/james/repository/api/

Propchange: 
james/server/trunk/data-library/src/main/java/org/apache/james/repository/api/
--
--- svn:mergeinfo (added)
+++ svn:mergeinfo Wed Jan 19 19:57:43 2011
@@ -0,0 +1,2 @@
+/james/server/sandbox/active/mina_experiments/repository-api/src/main/java/org/apache/james/repository/api:824613-825114
+/james/server/sandbox/active/pure_spring_deployment/repository-api/src/main/java/org/apache/james/repository/api:893986-896410



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

Re: DB Access with james3

[Luc Saulière]

Thx for answering so quickly, I tried this in a new package inside
james-server-container-spring project...
I have a public static void main(String[] args) containing my private
staticMailboxManager mailboxManager which
is injected with the annotation @Resource(name="mailboxmanager") from the
spring-bean.xml.
But it does not succeed in initializing the variable and I get a
NullPointerException as soon as I use it...
How can I initialize my beans without running the entire James app?

Tks.


2011/1/19 Eric Charles 

> Hi Luc,
>
> Yes, code has changed in trunk since M2 (we don't use MailServer anymore).
> In M2, or in trunk, the way to store mail is the same.
> If you are deploying the your code/class in the james spring container, you
> need to inject the mailboxmanager (with @Resources(name="mailboxmanager")
> and have a block of code such as:
>
> MailboxPath mailboxPath = MailboxPath.inbox(userName30);
> MailboxSession mailboxSession =
> mailboxManager.createSystemSession(userName30, log)
> MessageManager messageManager = mailboxManager.getMailbox(mailboxPath,
> mailboxSession);
> messageManager.appendMessage(your-input-stream,
>new Date(),
>mailboxSession,
>isRecent,
>flags);
>mailboxManager.endProcessingRequest(mailboxSession);
>}
>
> I just realize I can't find a simple test case to in trunk to show this (or
> maybe I didn't look good).
> If we don't have this, maybe you could open a jira so we can further
> provide test samples.
> (this comes from  the fact that all our tests were made from an "imap
> perspective", we moved the imap test to another  project, but we probably
> should further enrich the pure mailbox test cases/samples).
>
> Tks,
>
> Eric
>
>
>
>
> On 19/01/2011 18:04, Luc Sauličre wrote:
>
>> Hello,
>> I'm developing with Math on the same mail app. We didn't succeed in
>> storing
>> mail as james3 does. In fact the SieveMailet class does not have a
>> setUsersRepo method, I think you're dealing with a newer version of
>> James3...
>> We are developing with the M2 one and SieveMailet have a setMailserver
>> method instead.
>> So, is there any way to store a mail (from another IMAP connection for
>> instance) in the James3 mySQL db, as James3 does (i.e. filling correctly
>> all
>> the appropriate tables...)?
>>
>> Thx for helping.
>> Luc.
>>
>> 2011/1/17 Eric Charles
>>
>>  Hi,
>>> The mails store magic occurs in LocalDelivery where the MailboxManager is
>>> injected.
>>> (more precise, LocalDelivery uses the SieveMailet initialized with the
>>> MailboxManager)
>>>sieveMailet.setUsersRepository(usersRepository);
>>>sieveMailet.setMailboxManager(mailboxManager);
>>>sieveMailet.init(m);
>>>sieveMailet.setQuiet(true);
>>> ...
>>>if (mail.getState() != Mail.GHOST) {
>>>sieveMailet.service(mail);
>>>}
>>>
>>> You only need to know that *MailRepository is not for users' mail storage
>>> http://james.apache.org/server/3/feature-persistence.html
>>>
>>> Both topics (mailet usage for delivery + separate mail stores) are
>>> subject
>>> to discussion will certainly change.
>>>
>>> Tks,
>>>
>>> Eric
>>>
>>>
>>>
>>> On 17/01/2011 11:43, math math wrote:
>>>
>>>  Hello everybody,

 I am developping a web mail project using james 3M2. I'm trying to store
 mails in a "james3" way with an external application using MySql DB.
 I've
 tried to store mails using mysq database repositories for a few days now
 but
 i didn't succeed to do so... I 've traced the store method of
 JDBCMailRepository class and also the ToRepository one. But still don't
 know
 how James 3 is storing mails in the DB.

 I would be very glad if someone could help me in this task.

 Thanks,
 Mat


  -
>>> To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
>>> For additional commands, e-mail: server-dev-h...@james.apache.org
>>>
>>>
>>>
>
> -
> To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
> For additional commands, e-mail: server-dev-h...@james.apache.org
>
>

Re: DB Access with james3

[Norman Maurer]

Sure there is.. But to be honest I would not do this as you will
always need to look out for db changes etc. So I think the prefered
way would be to inject an instance of MailboxManager in your custom
app and store the mails via it. Or is this not possible ?

Bye,
Norman


2011/1/19 Luc Saulière :
> Hello,
> I'm developing with Math on the same mail app. We didn't succeed in storing
> mail as james3 does. In fact the SieveMailet class does not have a
> setUsersRepo method, I think you're dealing with a newer version of
> James3...
> We are developing with the M2 one and SieveMailet have a setMailserver
> method instead.
> So, is there any way to store a mail (from another IMAP connection for
> instance) in the James3 mySQL db, as James3 does (i.e. filling correctly all
> the appropriate tables...)?
>
> Thx for helping.
> Luc.
>
> 2011/1/17 Eric Charles 
>
>> Hi,
>> The mails store magic occurs in LocalDelivery where the MailboxManager is
>> injected.
>> (more precise, LocalDelivery uses the SieveMailet initialized with the
>> MailboxManager)
>>        sieveMailet.setUsersRepository(usersRepository);
>>        sieveMailet.setMailboxManager(mailboxManager);
>>        sieveMailet.init(m);
>>        sieveMailet.setQuiet(true);
>> ...
>>        if (mail.getState() != Mail.GHOST) {
>>            sieveMailet.service(mail);
>>        }
>>
>> You only need to know that *MailRepository is not for users' mail storage
>> http://james.apache.org/server/3/feature-persistence.html
>>
>> Both topics (mailet usage for delivery + separate mail stores) are subject
>> to discussion will certainly change.
>>
>> Tks,
>>
>> Eric
>>
>>
>>
>> On 17/01/2011 11:43, math math wrote:
>>
>>> Hello everybody,
>>>
>>> I am developping a web mail project using james 3M2. I'm trying to store
>>> mails in a "james3" way with an external application using MySql DB. I've
>>> tried to store mails using mysq database repositories for a few days now
>>> but
>>> i didn't succeed to do so... I 've traced the store method of
>>> JDBCMailRepository class and also the ToRepository one. But still don't
>>> know
>>> how James 3 is storing mails in the DB.
>>>
>>> I would be very glad if someone could help me in this task.
>>>
>>> Thanks,
>>> Mat
>>>
>>>
>>
>> -
>> To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
>> For additional commands, e-mail: server-dev-h...@james.apache.org
>>
>>
>

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

Re: DB Access with james3

[Eric Charles]

Hi Luc,

Yes, code has changed in trunk since M2 (we don't use MailServer anymore).
In M2, or in trunk, the way to store mail is the same.
If you are deploying the your code/class in the james spring container, 
you need to inject the mailboxmanager (with 
@Resources(name="mailboxmanager") and have a block of code such as:


MailboxPath mailboxPath = MailboxPath.inbox(userName30);
MailboxSession mailboxSession = mailboxManager.createSystemSession(userName30, 
log)
MessageManager messageManager = mailboxManager.getMailbox(mailboxPath, 
mailboxSession);
messageManager.appendMessage(your-input-stream,
new Date(),
mailboxSession,
isRecent,
flags);
mailboxManager.endProcessingRequest(mailboxSession);
}

I just realize I can't find a simple test case to in trunk to show this 
(or maybe I didn't look good).
If we don't have this, maybe you could open a jira so we can further 
provide test samples.
(this comes from  the fact that all our tests were made from an "imap 
perspective", we moved the imap test to another  project, but we 
probably should further enrich the pure mailbox test cases/samples).


Tks,

Eric



On 19/01/2011 18:04, Luc Saulière wrote:

Hello,
I'm developing with Math on the same mail app. We didn't succeed in storing
mail as james3 does. In fact the SieveMailet class does not have a
setUsersRepo method, I think you're dealing with a newer version of
James3...
We are developing with the M2 one and SieveMailet have a setMailserver
method instead.
So, is there any way to store a mail (from another IMAP connection for
instance) in the James3 mySQL db, as James3 does (i.e. filling correctly all
the appropriate tables...)?

Thx for helping.
Luc.

2011/1/17 Eric Charles


Hi,
The mails store magic occurs in LocalDelivery where the MailboxManager is
injected.
(more precise, LocalDelivery uses the SieveMailet initialized with the
MailboxManager)
sieveMailet.setUsersRepository(usersRepository);
sieveMailet.setMailboxManager(mailboxManager);
sieveMailet.init(m);
sieveMailet.setQuiet(true);
...
if (mail.getState() != Mail.GHOST) {
sieveMailet.service(mail);
}

You only need to know that *MailRepository is not for users' mail storage
http://james.apache.org/server/3/feature-persistence.html

Both topics (mailet usage for delivery + separate mail stores) are subject
to discussion will certainly change.

Tks,

Eric



On 17/01/2011 11:43, math math wrote:


Hello everybody,

I am developping a web mail project using james 3M2. I'm trying to store
mails in a "james3" way with an external application using MySql DB. I've
tried to store mails using mysq database repositories for a few days now
but
i didn't succeed to do so... I 've traced the store method of
JDBCMailRepository class and also the ToRepository one. But still don't
know
how James 3 is storing mails in the DB.

I would be very glad if someone could help me in this task.

Thanks,
Mat



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org





-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

Re: DB Access with james3

[Luc Saulière]

Hello,
I'm developing with Math on the same mail app. We didn't succeed in storing
mail as james3 does. In fact the SieveMailet class does not have a
setUsersRepo method, I think you're dealing with a newer version of
James3...
We are developing with the M2 one and SieveMailet have a setMailserver
method instead.
So, is there any way to store a mail (from another IMAP connection for
instance) in the James3 mySQL db, as James3 does (i.e. filling correctly all
the appropriate tables...)?

Thx for helping.
Luc.

2011/1/17 Eric Charles 

> Hi,
> The mails store magic occurs in LocalDelivery where the MailboxManager is
> injected.
> (more precise, LocalDelivery uses the SieveMailet initialized with the
> MailboxManager)
>sieveMailet.setUsersRepository(usersRepository);
>sieveMailet.setMailboxManager(mailboxManager);
>sieveMailet.init(m);
>sieveMailet.setQuiet(true);
> ...
>if (mail.getState() != Mail.GHOST) {
>sieveMailet.service(mail);
>}
>
> You only need to know that *MailRepository is not for users' mail storage
> http://james.apache.org/server/3/feature-persistence.html
>
> Both topics (mailet usage for delivery + separate mail stores) are subject
> to discussion will certainly change.
>
> Tks,
>
> Eric
>
>
>
> On 17/01/2011 11:43, math math wrote:
>
>> Hello everybody,
>>
>> I am developping a web mail project using james 3M2. I'm trying to store
>> mails in a "james3" way with an external application using MySql DB. I've
>> tried to store mails using mysq database repositories for a few days now
>> but
>> i didn't succeed to do so... I 've traced the store method of
>> JDBCMailRepository class and also the ToRepository one. But still don't
>> know
>> how James 3 is storing mails in the DB.
>>
>> I would be very glad if someone could help me in this task.
>>
>> Thanks,
>> Mat
>>
>>
>
> -
> To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
> For additional commands, e-mail: server-dev-h...@james.apache.org
>
>

[jira] Commented: (JAMES-1183) Error during initial request parsing

[Myron L Stewart (JIRA)]

[ 
https://issues.apache.org/jira/browse/JAMES-1183?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12983755#action_12983755
 ] 

Myron L Stewart commented on JAMES-1183:


Will do Norman. Just downloaded it. will implement it today on a test server I 
have. Thank you.

> Error during initial request parsing
> 
>
> Key: JAMES-1183
> URL: https://issues.apache.org/jira/browse/JAMES-1183
> Project: JAMES Server
>  Issue Type: Bug
>  Components: IMAPServer
>Affects Versions: 3.0-M3
> Environment: Centos AMD Dual-Core 2.6 gHz 4GB RAM
>Reporter: Myron L Stewart
> Fix For: 3.0-M3
>
> Attachments: partial_wrapper.log
>
>
> I am noticing issues off and on when I try to copy messages offline into a 
> mailbox folder via IMAP. James throws and error:
> 
> DEBUG 18:20:28,945 | james.imapserver | Error during initial request parsing
> org.apache.james.imap.decode.DecodingException: Missing argument.
>   at 
> org.apache.james.imap.decode.ImapRequestLineReader.nextWordChar(ImapRequestLineReader.java:58)
>   at 
> org.apache.james.imap.decode.base.AbstractImapCommandParser.consumeWord(AbstractImapCommandParser.java:296)
>   at 
> org.apache.james.imap.decode.base.AbstractImapCommandParser.atom(AbstractImapCommandParser.java:149)
>   at 
> org.apache.james.imap.decode.main.DefaultImapDecoder.decodeCommandTagged(DefaultImapDecoder.java:83)
>   at 
> org.apache.james.imap.decode.main.DefaultImapDecoder.decode(DefaultImapDecoder.java:61)
>   at 
> org.apache.james.imap.main.AbstractImapRequestHandler.doProcessRequest(AbstractImapRequestHandler.java:62)
>   at 
> org.apache.james.imap.main.ImapRequestStreamHandler.handleRequest(ImapRequestStreamHandler.java:82)
>   at 
> org.apache.james.imapserver.netty.ImapStreamChannelUpstreamHandler.processStreamIo(ImapStreamChannelUpstreamHandler.java:88)
>   at 
> org.jboss.netty.handler.stream.StreamHandler$1.run(StreamHandler.java:127)
>   at 
> java.util.concurrent.ThreadPoolExecutor$Worker.runTask(ThreadPoolExecutor.java:886)
>   at 
> java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:908)
>   at java.lang.Thread.run(Thread.java:662)
> DEBUG 18:20:28,946 | james.imapserver | Cannot parse tag
> org.apache.james.imap.decode.DecodingException: Missing argument.
>   at 
> org.apache.james.imap.decode.ImapRequestLineReader.nextWordChar(ImapRequestLineReader.java:58)
>   at 
> org.apache.james.imap.decode.base.AbstractImapCommandParser.consumeWord(AbstractImapCommandParser.java:296)
>   at 
> org.apache.james.imap.decode.base.AbstractImapCommandParser.tag(AbstractImapCommandParser.java:158)
>   at 
> org.apache.james.imap.decode.main.DefaultImapDecoder.decode(DefaultImapDecoder.java:60)
>   at 
> org.apache.james.imap.main.AbstractImapRequestHandler.doProcessRequest(AbstractImapRequestHandler.java:62)
>   at 
> org.apache.james.imap.main.ImapRequestStreamHandler.handleRequest(ImapRequestStreamHandler.java:82)
>   at 
> org.apache.james.imapserver.netty.ImapStreamChannelUpstreamHandler.processStreamIo(ImapStreamChannelUpstreamHandler.java:88)
>   at 
> org.jboss.netty.handler.stream.StreamHandler$1.run(StreamHandler.java:127)
>   at 
> java.util.concurrent.ThreadPoolExecutor$Worker.runTask(ThreadPoolExecutor.java:886)
>   at 
> java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:908)
>   at java.lang.Thread.run(Thread.java:662)
> 
> It seems like james does not like certain tags. I've seen it complain about 
> html tags too. Do not know if this is fixed or if someone is working on it. 
> Trying to gather more information to see exact causes or what sort of e-mails 
> cause james to throw this error.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] Commented: (JAMES-1183) Error during initial request parsing

[Norman Maurer (JIRA)]

[ 
https://issues.apache.org/jira/browse/JAMES-1183?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12983752#action_12983752
 ] 

Norman Maurer commented on JAMES-1183:
--

Try this one:

https://repository.apache.org/content/repositories/snapshots/org/apache/james/james-server-container-spring/3.0-M3-SNAPSHOT/james-server-container-spring-3.0-M3-20110119.141317-271-bin.tar.gz

> Error during initial request parsing
> 
>
> Key: JAMES-1183
> URL: https://issues.apache.org/jira/browse/JAMES-1183
> Project: JAMES Server
>  Issue Type: Bug
>  Components: IMAPServer
>Affects Versions: 3.0-M3
> Environment: Centos AMD Dual-Core 2.6 gHz 4GB RAM
>Reporter: Myron L Stewart
> Fix For: 3.0-M3
>
> Attachments: partial_wrapper.log
>
>
> I am noticing issues off and on when I try to copy messages offline into a 
> mailbox folder via IMAP. James throws and error:
> 
> DEBUG 18:20:28,945 | james.imapserver | Error during initial request parsing
> org.apache.james.imap.decode.DecodingException: Missing argument.
>   at 
> org.apache.james.imap.decode.ImapRequestLineReader.nextWordChar(ImapRequestLineReader.java:58)
>   at 
> org.apache.james.imap.decode.base.AbstractImapCommandParser.consumeWord(AbstractImapCommandParser.java:296)
>   at 
> org.apache.james.imap.decode.base.AbstractImapCommandParser.atom(AbstractImapCommandParser.java:149)
>   at 
> org.apache.james.imap.decode.main.DefaultImapDecoder.decodeCommandTagged(DefaultImapDecoder.java:83)
>   at 
> org.apache.james.imap.decode.main.DefaultImapDecoder.decode(DefaultImapDecoder.java:61)
>   at 
> org.apache.james.imap.main.AbstractImapRequestHandler.doProcessRequest(AbstractImapRequestHandler.java:62)
>   at 
> org.apache.james.imap.main.ImapRequestStreamHandler.handleRequest(ImapRequestStreamHandler.java:82)
>   at 
> org.apache.james.imapserver.netty.ImapStreamChannelUpstreamHandler.processStreamIo(ImapStreamChannelUpstreamHandler.java:88)
>   at 
> org.jboss.netty.handler.stream.StreamHandler$1.run(StreamHandler.java:127)
>   at 
> java.util.concurrent.ThreadPoolExecutor$Worker.runTask(ThreadPoolExecutor.java:886)
>   at 
> java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:908)
>   at java.lang.Thread.run(Thread.java:662)
> DEBUG 18:20:28,946 | james.imapserver | Cannot parse tag
> org.apache.james.imap.decode.DecodingException: Missing argument.
>   at 
> org.apache.james.imap.decode.ImapRequestLineReader.nextWordChar(ImapRequestLineReader.java:58)
>   at 
> org.apache.james.imap.decode.base.AbstractImapCommandParser.consumeWord(AbstractImapCommandParser.java:296)
>   at 
> org.apache.james.imap.decode.base.AbstractImapCommandParser.tag(AbstractImapCommandParser.java:158)
>   at 
> org.apache.james.imap.decode.main.DefaultImapDecoder.decode(DefaultImapDecoder.java:60)
>   at 
> org.apache.james.imap.main.AbstractImapRequestHandler.doProcessRequest(AbstractImapRequestHandler.java:62)
>   at 
> org.apache.james.imap.main.ImapRequestStreamHandler.handleRequest(ImapRequestStreamHandler.java:82)
>   at 
> org.apache.james.imapserver.netty.ImapStreamChannelUpstreamHandler.processStreamIo(ImapStreamChannelUpstreamHandler.java:88)
>   at 
> org.jboss.netty.handler.stream.StreamHandler$1.run(StreamHandler.java:127)
>   at 
> java.util.concurrent.ThreadPoolExecutor$Worker.runTask(ThreadPoolExecutor.java:886)
>   at 
> java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:908)
>   at java.lang.Thread.run(Thread.java:662)
> 
> It seems like james does not like certain tags. I've seen it complain about 
> html tags too. Do not know if this is fixed or if someone is working on it. 
> Trying to gather more information to see exact causes or what sort of e-mails 
> cause james to throw this error.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] Commented: (JAMES-1183) Error during initial request parsing

[Myron L Stewart (JIRA)]

[ 
https://issues.apache.org/jira/browse/JAMES-1183?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12983740#action_12983740
 ] 

Myron L Stewart commented on JAMES-1183:


I sure can. I was just learning how to download code from ASF :-) I'll try the 
latest snapshot and give it a try. Thanks.

> Error during initial request parsing
> 
>
> Key: JAMES-1183
> URL: https://issues.apache.org/jira/browse/JAMES-1183
> Project: JAMES Server
>  Issue Type: Bug
>  Components: IMAPServer
>Affects Versions: 3.0-M3
> Environment: Centos AMD Dual-Core 2.6 gHz 4GB RAM
>Reporter: Myron L Stewart
> Fix For: 3.0-M3
>
> Attachments: partial_wrapper.log
>
>
> I am noticing issues off and on when I try to copy messages offline into a 
> mailbox folder via IMAP. James throws and error:
> 
> DEBUG 18:20:28,945 | james.imapserver | Error during initial request parsing
> org.apache.james.imap.decode.DecodingException: Missing argument.
>   at 
> org.apache.james.imap.decode.ImapRequestLineReader.nextWordChar(ImapRequestLineReader.java:58)
>   at 
> org.apache.james.imap.decode.base.AbstractImapCommandParser.consumeWord(AbstractImapCommandParser.java:296)
>   at 
> org.apache.james.imap.decode.base.AbstractImapCommandParser.atom(AbstractImapCommandParser.java:149)
>   at 
> org.apache.james.imap.decode.main.DefaultImapDecoder.decodeCommandTagged(DefaultImapDecoder.java:83)
>   at 
> org.apache.james.imap.decode.main.DefaultImapDecoder.decode(DefaultImapDecoder.java:61)
>   at 
> org.apache.james.imap.main.AbstractImapRequestHandler.doProcessRequest(AbstractImapRequestHandler.java:62)
>   at 
> org.apache.james.imap.main.ImapRequestStreamHandler.handleRequest(ImapRequestStreamHandler.java:82)
>   at 
> org.apache.james.imapserver.netty.ImapStreamChannelUpstreamHandler.processStreamIo(ImapStreamChannelUpstreamHandler.java:88)
>   at 
> org.jboss.netty.handler.stream.StreamHandler$1.run(StreamHandler.java:127)
>   at 
> java.util.concurrent.ThreadPoolExecutor$Worker.runTask(ThreadPoolExecutor.java:886)
>   at 
> java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:908)
>   at java.lang.Thread.run(Thread.java:662)
> DEBUG 18:20:28,946 | james.imapserver | Cannot parse tag
> org.apache.james.imap.decode.DecodingException: Missing argument.
>   at 
> org.apache.james.imap.decode.ImapRequestLineReader.nextWordChar(ImapRequestLineReader.java:58)
>   at 
> org.apache.james.imap.decode.base.AbstractImapCommandParser.consumeWord(AbstractImapCommandParser.java:296)
>   at 
> org.apache.james.imap.decode.base.AbstractImapCommandParser.tag(AbstractImapCommandParser.java:158)
>   at 
> org.apache.james.imap.decode.main.DefaultImapDecoder.decode(DefaultImapDecoder.java:60)
>   at 
> org.apache.james.imap.main.AbstractImapRequestHandler.doProcessRequest(AbstractImapRequestHandler.java:62)
>   at 
> org.apache.james.imap.main.ImapRequestStreamHandler.handleRequest(ImapRequestStreamHandler.java:82)
>   at 
> org.apache.james.imapserver.netty.ImapStreamChannelUpstreamHandler.processStreamIo(ImapStreamChannelUpstreamHandler.java:88)
>   at 
> org.jboss.netty.handler.stream.StreamHandler$1.run(StreamHandler.java:127)
>   at 
> java.util.concurrent.ThreadPoolExecutor$Worker.runTask(ThreadPoolExecutor.java:886)
>   at 
> java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:908)
>   at java.lang.Thread.run(Thread.java:662)
> 
> It seems like james does not like certain tags. I've seen it complain about 
> html tags too. Do not know if this is fixed or if someone is working on it. 
> Trying to gather more information to see exact causes or what sort of e-mails 
> cause james to throw this error.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

svn commit: r1060819 - /james/server/trunk/protocols-library/src/main/java/org/apache/james/protocols/lib/jmx/ServerMBean.java

[norman]

Author: norman
Date: Wed Jan 19 14:08:46 2011
New Revision: 1060819

URL: http://svn.apache.org/viewvc?rev=1060819&view=rev
Log:
expose timeout via jmx

Modified:

james/server/trunk/protocols-library/src/main/java/org/apache/james/protocols/lib/jmx/ServerMBean.java

Modified: 
james/server/trunk/protocols-library/src/main/java/org/apache/james/protocols/lib/jmx/ServerMBean.java
URL: 
http://svn.apache.org/viewvc/james/server/trunk/protocols-library/src/main/java/org/apache/james/protocols/lib/jmx/ServerMBean.java?rev=1060819&r1=1060818&r2=1060819&view=diff
==
--- 
james/server/trunk/protocols-library/src/main/java/org/apache/james/protocols/lib/jmx/ServerMBean.java
 (original)
+++ 
james/server/trunk/protocols-library/src/main/java/org/apache/james/protocols/lib/jmx/ServerMBean.java
 Wed Jan 19 14:08:46 2011
@@ -104,4 +104,11 @@ public interface ServerMBean {
  * @return stop
  */
 public boolean stop();
+
+/**
+ * Return the timeout in seconds 
+ * 
+ * @return timeout
+ */
+public int getTimeout();
 }



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

svn commit: r1060817 - /james/server/trunk/imapserver/src/main/java/org/apache/james/imapserver/netty/NettyImapRequestLineReader.java

[norman]

Author: norman
Date: Wed Jan 19 14:02:21 2011
New Revision: 1060817

URL: http://svn.apache.org/viewvc?rev=1060817&view=rev
Log:
only small javadocs improvements

Modified:

james/server/trunk/imapserver/src/main/java/org/apache/james/imapserver/netty/NettyImapRequestLineReader.java

Modified: 
james/server/trunk/imapserver/src/main/java/org/apache/james/imapserver/netty/NettyImapRequestLineReader.java
URL: 
http://svn.apache.org/viewvc/james/server/trunk/imapserver/src/main/java/org/apache/james/imapserver/netty/NettyImapRequestLineReader.java?rev=1060817&r1=1060816&r2=1060817&view=diff
==
--- 
james/server/trunk/imapserver/src/main/java/org/apache/james/imapserver/netty/NettyImapRequestLineReader.java
 (original)
+++ 
james/server/trunk/imapserver/src/main/java/org/apache/james/imapserver/netty/NettyImapRequestLineReader.java
 Wed Jan 19 14:02:21 2011
@@ -30,7 +30,7 @@ import org.jboss.netty.channel.Channel;
 
 /**
  * {@link ImapRequestLineReader} implementation which will write to a {@link 
Channel} and read from a {@link ChannelBuffer}. Please
- * see the docs on {@link #nextChar()} and {@link #read(int)} to understand 
the special behavoir of this implementation
+ * see the docs on {@link #nextChar()} and {@link #read(int)} to understand 
the special behavior of this implementation
  *
  */
 public class NettyImapRequestLineReader extends ImapRequestLineReader{
@@ -86,7 +86,11 @@ public class NettyImapRequestLineReader 
 channel.write(cRequest);
 }
 
-
+/**
+ * {@link RuntimeException} which will get thrown by {@link 
NettyImapRequestLineReader#nextChar()} and {@link 
NettyImapRequestLineReader#read(int)} if not 
+ * enough data is readable in the underlying {@link ChannelBuffer}
+ *
+ */
 @SuppressWarnings("serial")
 public final class NotEnoughDataException extends RuntimeException{
 



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

svn commit: r1060816 - in /james/server/trunk: lmtpserver/src/main/java/org/apache/james/lmtpserver/netty/ smtpserver/src/main/java/org/apache/james/smtpserver/netty/

[norman]

Author: norman
Date: Wed Jan 19 14:01:45 2011
New Revision: 1060816

URL: http://svn.apache.org/viewvc?rev=1060816&view=rev
Log:
Expose some more stuff via JMX

Modified:

james/server/trunk/lmtpserver/src/main/java/org/apache/james/lmtpserver/netty/LMTPServer.java

james/server/trunk/lmtpserver/src/main/java/org/apache/james/lmtpserver/netty/LMTPServerMBean.java

james/server/trunk/smtpserver/src/main/java/org/apache/james/smtpserver/netty/SMTPServer.java

james/server/trunk/smtpserver/src/main/java/org/apache/james/smtpserver/netty/SMTPServerMBean.java

Modified: 
james/server/trunk/lmtpserver/src/main/java/org/apache/james/lmtpserver/netty/LMTPServer.java
URL: 
http://svn.apache.org/viewvc/james/server/trunk/lmtpserver/src/main/java/org/apache/james/lmtpserver/netty/LMTPServer.java?rev=1060816&r1=1060815&r2=1060816&view=diff
==
--- 
james/server/trunk/lmtpserver/src/main/java/org/apache/james/lmtpserver/netty/LMTPServer.java
 (original)
+++ 
james/server/trunk/lmtpserver/src/main/java/org/apache/james/lmtpserver/netty/LMTPServer.java
 Wed Jan 19 14:01:45 2011
@@ -53,7 +53,10 @@ public class LMTPServer extends Abstract
 }
 
 
-@Override
+/*
+ * (non-Javadoc)
+ * @see 
org.apache.james.protocols.lib.netty.AbstractConfigurableAsyncServer#getDefaultPort()
+ */
 public int getDefaultPort() {
 return 24;
 }
@@ -198,45 +201,46 @@ public class LMTPServer extends Abstract
 /*
  * (non-Javadoc)
  * 
- * @see
- * 
org.apache.james.smtpserver.SMTPServerMBean#getAddressBracketsEnforcement
- * ()
+ * @see org.apache.james.smtpserver.SMTPServerMBean#getMaximalMessageSize()
  */
-public boolean getAddressBracketsEnforcement() {
-return lmtpConfig.useAddressBracketsEnforcement();
+public long getMaximalMessageSize() {
+return lmtpConfig.getMaxMessageSize();
 }
 
 /*
  * (non-Javadoc)
  * 
- * @see 
org.apache.james.smtpserver.SMTPServerMBean#getHeloEhloEnforcement()
+ * @see
+ * org.apache.james.protocols.smtp.SMTPServerMBean#getNetworkInterface()
  */
-public boolean getHeloEhloEnforcement() {
-return lmtpConfig.useHeloEhloEnforcement();
+public String getNetworkInterface() {
+return "unknown";
 }
 
 /*
  * (non-Javadoc)
- * 
- * @see org.apache.james.smtpserver.SMTPServerMBean#getMaximalMessageSize()
+ * @see 
org.apache.james.protocols.lib.netty.AbstractConfigurableAsyncServer#getDefaultJMXName()
  */
-public long getMaximalMessageSize() {
-return lmtpConfig.getMaxMessageSize();
+protected String getDefaultJMXName() {
+return "lmtpserver";
 }
 
+
 /*
  * (non-Javadoc)
- * 
- * @see
- * org.apache.james.protocols.smtp.SMTPServerMBean#getNetworkInterface()
+ * @see 
org.apache.james.smtpserver.netty.SMTPServerMBean#setMaximalMessageSize(long)
  */
-public String getNetworkInterface() {
-return "unknown";
+public void setMaximalMessageSize(long maxSize) {
+maxMessageSize = maxSize;
 }
 
-@Override
-protected String getDefaultJMXName() {
-return "lmtpserver";
+
+/*
+ * (non-Javadoc)
+ * @see org.apache.james.lmtpserver.netty.LMTPServerMBean#getHeloName()
+ */
+public String getHeloName() {
+return lmtpConfig.getHelloName();
 }
 
 }

Modified: 
james/server/trunk/lmtpserver/src/main/java/org/apache/james/lmtpserver/netty/LMTPServerMBean.java
URL: 
http://svn.apache.org/viewvc/james/server/trunk/lmtpserver/src/main/java/org/apache/james/lmtpserver/netty/LMTPServerMBean.java?rev=1060816&r1=1060815&r2=1060816&view=diff
==
--- 
james/server/trunk/lmtpserver/src/main/java/org/apache/james/lmtpserver/netty/LMTPServerMBean.java
 (original)
+++ 
james/server/trunk/lmtpserver/src/main/java/org/apache/james/lmtpserver/netty/LMTPServerMBean.java
 Wed Jan 19 14:01:45 2011
@@ -18,8 +18,31 @@
  /
 package org.apache.james.lmtpserver.netty;
 
-import org.apache.james.smtpserver.netty.SMTPServerMBean;
+import org.apache.james.protocols.lib.jmx.ServerMBean;
+import org.apache.james.protocols.smtp.SMTPServerMBean;
 
-public interface LMTPServerMBean extends SMTPServerMBean{
+public interface LMTPServerMBean extends ServerMBean, SMTPServerMBean{
 
+/**
+ * Return the maximum allowed size of the message
+ * 
+ * @return maxMessageSize
+ */
+public long getMaximalMessageSize();
+
+/**
+ * Set the maximum allowed size of messages. Set this to 0 to accept every 
message
+ * 
+ * @param maxSize
+ */
+public void setMaximalMessageSize(long maxSize);
+
+
+/**
+ * Return the hello name
+ * 
+ * @return helo
+ */
+public String g

svn commit: r1060804 - /james/server/trunk/container-spring/src/main/config/james/context/james-server-context.xml

[eric]

Author: eric
Date: Wed Jan 19 13:16:26 2011
New Revision: 1060804

URL: http://svn.apache.org/viewvc?rev=1060804&view=rev
Log:
Fix JMX tree by moving mailboxcopier and jame23importer under component.

Modified:

james/server/trunk/container-spring/src/main/config/james/context/james-server-context.xml

Modified: 
james/server/trunk/container-spring/src/main/config/james/context/james-server-context.xml
URL: 
http://svn.apache.org/viewvc/james/server/trunk/container-spring/src/main/config/james/context/james-server-context.xml?rev=1060804&r1=1060803&r2=1060804&view=diff
==
--- 
james/server/trunk/container-spring/src/main/config/james/context/james-server-context.xml
 (original)
+++ 
james/server/trunk/container-spring/src/main/config/james/context/james-server-context.xml
 Wed Jan 19 13:16:26 2011
@@ -382,9 +382,9 @@
   
   
   
-  
+  
+  
   
-  
 
   
   



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

svn commit: r1060792 - /james/server/trunk/cli/src/main/java/org/apache/james/cli/ServerProbe.java

[norman]

Author: norman
Date: Wed Jan 19 12:39:23 2011
New Revision: 1060792

URL: http://svn.apache.org/viewvc?rev=1060792&view=rev
Log:
Need to cast the MBean in java5 usage

Modified:
james/server/trunk/cli/src/main/java/org/apache/james/cli/ServerProbe.java

Modified: 
james/server/trunk/cli/src/main/java/org/apache/james/cli/ServerProbe.java
URL: 
http://svn.apache.org/viewvc/james/server/trunk/cli/src/main/java/org/apache/james/cli/ServerProbe.java?rev=1060792&r1=1060791&r2=1060792&view=diff
==
--- james/server/trunk/cli/src/main/java/org/apache/james/cli/ServerProbe.java 
(original)
+++ james/server/trunk/cli/src/main/java/org/apache/james/cli/ServerProbe.java 
Wed Jan 19 12:39:23 2011
@@ -93,11 +93,11 @@ public class ServerProbe {
 try
 {
 ObjectName name = new ObjectName(DOMAINLIST_OBJECT_NAME);
-domainListProcxy = 
MBeanServerInvocationHandler.newProxyInstance(mbeanServerConn, name, 
DomainListManagementMBean.class, true);
+domainListProcxy = (DomainListManagementMBean) 
MBeanServerInvocationHandler.newProxyInstance(mbeanServerConn, name, 
DomainListManagementMBean.class, true);
 name = new ObjectName(VIRTUALUSERTABLE_OBJECT_NAME);
-virtualUserTableProxy = 
MBeanServerInvocationHandler.newProxyInstance(mbeanServerConn, name, 
VirtualUserTableManagementMBean.class, true);
+virtualUserTableProxy = (VirtualUserTableManagementMBean) 
MBeanServerInvocationHandler.newProxyInstance(mbeanServerConn, name, 
VirtualUserTableManagementMBean.class, true);
 name = new ObjectName(USERSREPOSITORY_OBJECT_NAME);
-usersRepositoryProxy = 
MBeanServerInvocationHandler.newProxyInstance(mbeanServerConn, name, 
UsersRepositoryManagementMBean.class, true);
+usersRepositoryProxy = (UsersRepositoryManagementMBean) 
MBeanServerInvocationHandler.newProxyInstance(mbeanServerConn, name, 
UsersRepositoryManagementMBean.class, true);
 } catch (MalformedObjectNameException e)
 {
 throw new RuntimeException(



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

svn commit: r1060774 - /james/server/trunk/container-spring/pom.xml

[norman]

Author: norman
Date: Wed Jan 19 11:49:55 2011
New Revision: 1060774

URL: http://svn.apache.org/viewvc?rev=1060774&view=rev
Log:
Change scope of core to compile as the importer need it

Modified:
james/server/trunk/container-spring/pom.xml

Modified: james/server/trunk/container-spring/pom.xml
URL: 
http://svn.apache.org/viewvc/james/server/trunk/container-spring/pom.xml?rev=1060774&r1=1060773&r2=1060774&view=diff
==
--- james/server/trunk/container-spring/pom.xml (original)
+++ james/server/trunk/container-spring/pom.xml Wed Jan 19 11:49:55 2011
@@ -461,7 +461,7 @@
 
   org.apache.james
   james-server-core
-  runtime
+  compile
 
 
   org.apache.camel



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

svn commit: r1060764 - /james/server/trunk/cli/src/main/java/org/apache/james/cli/ServerProbe.java

[norman]

Author: norman
Date: Wed Jan 19 11:09:33 2011
New Revision: 1060764

URL: http://svn.apache.org/viewvc?rev=1060764&view=rev
Log:
Replace JMX class usage as it only exist in java6

Modified:
james/server/trunk/cli/src/main/java/org/apache/james/cli/ServerProbe.java

Modified: 
james/server/trunk/cli/src/main/java/org/apache/james/cli/ServerProbe.java
URL: 
http://svn.apache.org/viewvc/james/server/trunk/cli/src/main/java/org/apache/james/cli/ServerProbe.java?rev=1060764&r1=1060763&r2=1060764&view=diff
==
--- james/server/trunk/cli/src/main/java/org/apache/james/cli/ServerProbe.java 
(original)
+++ james/server/trunk/cli/src/main/java/org/apache/james/cli/ServerProbe.java 
Wed Jan 19 11:09:33 2011
@@ -22,8 +22,8 @@ import java.io.IOException;
 import java.util.Collection;
 import java.util.Map;
 
-import javax.management.JMX;
 import javax.management.MBeanServerConnection;
+import javax.management.MBeanServerInvocationHandler;
 import javax.management.MalformedObjectNameException;
 import javax.management.ObjectName;
 import javax.management.remote.JMXConnector;
@@ -93,11 +93,11 @@ public class ServerProbe {
 try
 {
 ObjectName name = new ObjectName(DOMAINLIST_OBJECT_NAME);
-domainListProcxy = JMX.newMBeanProxy(mbeanServerConn, name, 
DomainListManagementMBean.class);
+domainListProcxy = 
MBeanServerInvocationHandler.newProxyInstance(mbeanServerConn, name, 
DomainListManagementMBean.class, true);
 name = new ObjectName(VIRTUALUSERTABLE_OBJECT_NAME);
-virtualUserTableProxy = JMX.newMBeanProxy(mbeanServerConn, name, 
VirtualUserTableManagementMBean.class);
+virtualUserTableProxy = 
MBeanServerInvocationHandler.newProxyInstance(mbeanServerConn, name, 
VirtualUserTableManagementMBean.class, true);
 name = new ObjectName(USERSREPOSITORY_OBJECT_NAME);
-usersRepositoryProxy = JMX.newMBeanProxy(mbeanServerConn, name, 
UsersRepositoryManagementMBean.class);
+usersRepositoryProxy = 
MBeanServerInvocationHandler.newProxyInstance(mbeanServerConn, name, 
UsersRepositoryManagementMBean.class, true);
 } catch (MalformedObjectNameException e)
 {
 throw new RuntimeException(



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] Commented: (JAMES-1052) Provide a tool to migrate 2.3 mail-repositories to 3.0 mailbox-stores

[Eric Charles (JIRA)]

[ 
https://issues.apache.org/jira/browse/JAMES-1052?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12983655#action_12983655
 ] 

Eric Charles commented on JAMES-1052:
-

(from ml)

You can now import 2.3 users and mails into 3.0 via JMX (available in trunk).

It needs an additional conf file to define the 2.3 userrepository (difficult to 
give all params via JMX).
For users import, you have to give a default password (3.0 users will all be 
created with that default password).
For mail import, you have to give the mail repository path.

I successfully tested it creating some users and mails in james 2.3, copying 
the ./var/users/* and ./var/mail/inboxes/* to james 3.0, and running the tool.

Tks for your comments on the API and tests (some users will certainly wait on 
this to jump from 2.3 to 3.0).


> Provide a tool to migrate 2.3 mail-repositories to 3.0 mailbox-stores
> -
>
> Key: JAMES-1052
> URL: https://issues.apache.org/jira/browse/JAMES-1052
> Project: JAMES Server
>  Issue Type: New Feature
>Affects Versions: 3.0-M1
>Reporter: Eric Charles
>Assignee: Eric Charles
> Fix For: 3.0
>
>
> A migration tool must be provided to ease the 2.3 mail store pumping into the 
> new 3.0 maiblox store.
> This should be implemented against MailRepository and Mailbox/MessageManager 
> and not against low-level API (File, JDBC,...)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[Review] Import 2.3 users and mails to 3.0

[Eric Charles]

Hi,

You can now import 2.3 users and mails into 3.0 via JMX (available in 
trunk).


It needs an additional conf file to define the 2.3 userrepository 
(difficult to give all params via JMX).
For users import, you have to give a default password (3.0 users will 
all be created with that default password).

For mail import, you have to give the mail repository path.

I successfully tested it creating some users and mails in james 2.3, 
copying the ./var/users/* and ./var/mail/inboxes/* to james 3.0, and 
running the tool.


Tks for your comments on the API and tests (some users will certainly 
wait on this to jump from 2.3 to 3.0).


Eric


-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] Updated: (JAMES-1183) Error during initial request parsing

[Norman Maurer (JIRA)]

 [ 
https://issues.apache.org/jira/browse/JAMES-1183?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Norman Maurer updated JAMES-1183:
-

Fix Version/s: 3.0-M3

> Error during initial request parsing
> 
>
> Key: JAMES-1183
> URL: https://issues.apache.org/jira/browse/JAMES-1183
> Project: JAMES Server
>  Issue Type: Bug
>  Components: IMAPServer
>Affects Versions: 3.0-M3
> Environment: Centos AMD Dual-Core 2.6 gHz 4GB RAM
>Reporter: Myron L Stewart
> Fix For: 3.0-M3
>
> Attachments: partial_wrapper.log
>
>
> I am noticing issues off and on when I try to copy messages offline into a 
> mailbox folder via IMAP. James throws and error:
> 
> DEBUG 18:20:28,945 | james.imapserver | Error during initial request parsing
> org.apache.james.imap.decode.DecodingException: Missing argument.
>   at 
> org.apache.james.imap.decode.ImapRequestLineReader.nextWordChar(ImapRequestLineReader.java:58)
>   at 
> org.apache.james.imap.decode.base.AbstractImapCommandParser.consumeWord(AbstractImapCommandParser.java:296)
>   at 
> org.apache.james.imap.decode.base.AbstractImapCommandParser.atom(AbstractImapCommandParser.java:149)
>   at 
> org.apache.james.imap.decode.main.DefaultImapDecoder.decodeCommandTagged(DefaultImapDecoder.java:83)
>   at 
> org.apache.james.imap.decode.main.DefaultImapDecoder.decode(DefaultImapDecoder.java:61)
>   at 
> org.apache.james.imap.main.AbstractImapRequestHandler.doProcessRequest(AbstractImapRequestHandler.java:62)
>   at 
> org.apache.james.imap.main.ImapRequestStreamHandler.handleRequest(ImapRequestStreamHandler.java:82)
>   at 
> org.apache.james.imapserver.netty.ImapStreamChannelUpstreamHandler.processStreamIo(ImapStreamChannelUpstreamHandler.java:88)
>   at 
> org.jboss.netty.handler.stream.StreamHandler$1.run(StreamHandler.java:127)
>   at 
> java.util.concurrent.ThreadPoolExecutor$Worker.runTask(ThreadPoolExecutor.java:886)
>   at 
> java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:908)
>   at java.lang.Thread.run(Thread.java:662)
> DEBUG 18:20:28,946 | james.imapserver | Cannot parse tag
> org.apache.james.imap.decode.DecodingException: Missing argument.
>   at 
> org.apache.james.imap.decode.ImapRequestLineReader.nextWordChar(ImapRequestLineReader.java:58)
>   at 
> org.apache.james.imap.decode.base.AbstractImapCommandParser.consumeWord(AbstractImapCommandParser.java:296)
>   at 
> org.apache.james.imap.decode.base.AbstractImapCommandParser.tag(AbstractImapCommandParser.java:158)
>   at 
> org.apache.james.imap.decode.main.DefaultImapDecoder.decode(DefaultImapDecoder.java:60)
>   at 
> org.apache.james.imap.main.AbstractImapRequestHandler.doProcessRequest(AbstractImapRequestHandler.java:62)
>   at 
> org.apache.james.imap.main.ImapRequestStreamHandler.handleRequest(ImapRequestStreamHandler.java:82)
>   at 
> org.apache.james.imapserver.netty.ImapStreamChannelUpstreamHandler.processStreamIo(ImapStreamChannelUpstreamHandler.java:88)
>   at 
> org.jboss.netty.handler.stream.StreamHandler$1.run(StreamHandler.java:127)
>   at 
> java.util.concurrent.ThreadPoolExecutor$Worker.runTask(ThreadPoolExecutor.java:886)
>   at 
> java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:908)
>   at java.lang.Thread.run(Thread.java:662)
> 
> It seems like james does not like certain tags. I've seen it complain about 
> html tags too. Do not know if this is fixed or if someone is working on it. 
> Trying to gather more information to see exact causes or what sort of e-mails 
> cause james to throw this error.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

svn commit: r1060763 - /james/server/trunk/container-spring/src/main/java/org/apache/james/container/spring/tool/James23ImporterManagementMBean.java

[eric]

Author: eric
Date: Wed Jan 19 10:57:21 2011
New Revision: 1060763

URL: http://svn.apache.org/viewvc?rev=1060763&view=rev
Log:
javadoc the 2.3 import management interface (MAILBOX-34)

Modified:

james/server/trunk/container-spring/src/main/java/org/apache/james/container/spring/tool/James23ImporterManagementMBean.java

Modified: 
james/server/trunk/container-spring/src/main/java/org/apache/james/container/spring/tool/James23ImporterManagementMBean.java
URL: 
http://svn.apache.org/viewvc/james/server/trunk/container-spring/src/main/java/org/apache/james/container/spring/tool/James23ImporterManagementMBean.java?rev=1060763&r1=1060762&r2=1060763&view=diff
==
--- 
james/server/trunk/container-spring/src/main/java/org/apache/james/container/spring/tool/James23ImporterManagementMBean.java
 (original)
+++ 
james/server/trunk/container-spring/src/main/java/org/apache/james/container/spring/tool/James23ImporterManagementMBean.java
 Wed Jan 19 10:57:21 2011
@@ -28,18 +28,25 @@ import org.apache.james.mailbox.MailboxM
 public interface James23ImporterManagementMBean {
 
 /**
+ * First import users (each user is created with the given default  
password), the
+ * import the mails for each users from the given mail repository path.
+ * 
  * @param james23MailRepositoryPath
  * @param defaultPassword
  */
 void importUsersAndMailsFromJames23(String james23MailRepositoryPath, 
String defaultPassword) throws Exception;
 
 /**
+ * Import users (each user is created with the given default  password).
+ * 
  * @param defaultPassword
  * @throws Exception
  */
 void importUsersFromJames23(String defaultPassword) throws Exception;
 
 /**
+ * Import the mails for each users from the given mail repository path.
+ * 
  * @param james23MailRepositoryPath
  * @throws Exception
  */



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

svn commit: r1060760 - /james/server/trunk/container-spring/src/main/java/org/apache/james/container/spring/tool/James23Importer.java

[eric]

Author: eric
Date: Wed Jan 19 10:52:34 2011
New Revision: 1060760

URL: http://svn.apache.org/viewvc?rev=1060760&view=rev
Log:
Fix the mail inputstream import (MAILBOX-34)

Modified:

james/server/trunk/container-spring/src/main/java/org/apache/james/container/spring/tool/James23Importer.java

Modified: 
james/server/trunk/container-spring/src/main/java/org/apache/james/container/spring/tool/James23Importer.java
URL: 
http://svn.apache.org/viewvc/james/server/trunk/container-spring/src/main/java/org/apache/james/container/spring/tool/James23Importer.java?rev=1060760&r1=1060759&r2=1060760&view=diff
==
--- 
james/server/trunk/container-spring/src/main/java/org/apache/james/container/spring/tool/James23Importer.java
 (original)
+++ 
james/server/trunk/container-spring/src/main/java/org/apache/james/container/spring/tool/James23Importer.java
 Wed Jan 19 10:52:34 2011
@@ -27,6 +27,7 @@ import javax.mail.Flags;
 import javax.mail.MessagingException;
 
 import org.apache.commons.logging.Log;
+import org.apache.james.core.MimeMessageInputStream;
 import org.apache.james.domainlist.api.DomainList;
 import org.apache.james.domainlist.api.DomainListException;
 import org.apache.james.lifecycle.api.LogEnabled;
@@ -152,7 +153,7 @@ public class James23Importer implements 
 mailboxManager.createMailbox(mailboxPath, mailboxSession);
 }
 catch (MailboxExistsException e) {
-// Do nothing.
+// Do nothing, the mailbox already exists.
 }
 mailboxManager.endProcessingRequest(mailboxSession);
 
@@ -161,9 +162,8 @@ public class James23Importer implements 
 while (mailRepositoryIterator.hasNext()) {
 Mail mail = 
mailRepository.retrieve(mailRepositoryIterator.next());
 mailboxManager.startProcessingRequest(mailboxSession);
-// TODO We need to build the full message, not only the body.
-
messageManager.appendMessage(mail.getMessage().getInputStream(), 
-mail.getMessage().getReceivedDate(), 
+messageManager.appendMessage(new 
MimeMessageInputStream(mail.getMessage()), 
+new Date(), 
 mailboxSession, 
 isRecent, 
 flags);



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

svn commit: r1060747 - /james/server/trunk/jpa/pom.xml

[norman]

Author: norman
Date: Wed Jan 19 10:30:12 2011
New Revision: 1060747

URL: http://svn.apache.org/viewvc?rev=1060747&view=rev
Log:
Upgrade openjpa-maven-plugin to fix some compile error

Modified:
james/server/trunk/jpa/pom.xml

Modified: james/server/trunk/jpa/pom.xml
URL: 
http://svn.apache.org/viewvc/james/server/trunk/jpa/pom.xml?rev=1060747&r1=1060746&r2=1060747&view=diff
==
--- james/server/trunk/jpa/pom.xml (original)
+++ james/server/trunk/jpa/pom.xml Wed Jan 19 10:30:12 2011
@@ -34,7 +34,7 @@
   
 org.codehaus.mojo
 openjpa-maven-plugin
-1.0
+1.1
 
   
org/apache/james/user/jpa/model/JPAUser.class,org/apache/james/vut/jpa/model/JPAVirtualUser.class,org/apache/james/domainlist/jpa/model/JPADomain.class
   true



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] Commented: (JAMES-1184) Modules consolidation

[Eric Charles (JIRA)]

[ 
https://issues.apache.org/jira/browse/JAMES-1184?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12983634#action_12983634
 ] 

Eric Charles commented on JAMES-1184:
-

+1

> Modules consolidation
> -
>
> Key: JAMES-1184
> URL: https://issues.apache.org/jira/browse/JAMES-1184
> Project: JAMES Server
>  Issue Type: Task
>  Components: Build System
>Affects Versions: 3.0-M2
>Reporter: Stefano Bagnara
>Assignee: Stefano Bagnara
> Fix For: 3.0-M3
>
>
> Just opening it to keep track of the proposal to the mailing list.
> 
> - mail-file(3)
> - mail-jcr(1)
> - mail-jdbc(3)
> - user-jpa(4)
> - user-file(2)
> - user-jcr(2)
> - user-jdbc(4)
> - domainlist-jpa(2)
> - domainlist-xml(1)
> I only see benefits and no drawbacks in consolidating the 9 modules
> above to the following 4 modules:
>  persistence-jpa(6) = user-jpa(4)+domainlist-jpa(2)
>  persistence-file(6) = user-file(2)+mail-file(3)+domainlist-xml(1)
>  persistence-jcr(3) = user-jcr(2)+mail-jcr(1)
>  persistence-jdbc(7) = user-jdbc(4)+mail-jdbc(3)
> - mail-library(1)
> - domainlist-library(2)
> - user-library(9)
> We could consolidate them into a persistence-library module (from 3 to
> 1 module).
>  persistence-library(12) =
> mail-library(1)+domainlist-library(2)+user-library(9)
> - queue-library(2)
> - queue-jms(5)
> We could merge queue-library to queue-jms (a library module makes
> sense when we do something useful.. if it is trivial stuff it can be
> duplicated or moved to the api layer).
>  queue-jms(7) = queue-jms(5)+queue-library(2)
> -
> It seems there is agreement on the merging. We still have to decide whether 
> to name the modules "persistence-${type}" or simply "${type}" (or maybe 
> "data-${type}")

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

svn commit: r1060730 - /james/server/trunk/queue-jms/src/main/java/org/apache/james/queue/library/MailQueueManagement.java

[norman]

Author: norman
Date: Wed Jan 19 09:42:58 2011
New Revision: 1060730

URL: http://svn.apache.org/viewvc?rev=1060730&view=rev
Log:
the generic is only present in java 6

Modified:

james/server/trunk/queue-jms/src/main/java/org/apache/james/queue/library/MailQueueManagement.java

Modified: 
james/server/trunk/queue-jms/src/main/java/org/apache/james/queue/library/MailQueueManagement.java
URL: 
http://svn.apache.org/viewvc/james/server/trunk/queue-jms/src/main/java/org/apache/james/queue/library/MailQueueManagement.java?rev=1060730&r1=1060729&r2=1060730&view=diff
==
--- 
james/server/trunk/queue-jms/src/main/java/org/apache/james/queue/library/MailQueueManagement.java
 (original)
+++ 
james/server/trunk/queue-jms/src/main/java/org/apache/james/queue/library/MailQueueManagement.java
 Wed Jan 19 09:42:58 2011
@@ -164,7 +164,7 @@ public class MailQueueManagement extends
 "Attributes stored", 
 "Timestamp of when the next delivery attempt will be make"
 };
-OpenType[] types = new OpenType[] { 
+OpenType[] types = new OpenType[] { 
 SimpleType.STRING, 
 SimpleType.STRING, 
 SimpleType.STRING, 



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] Commented: (JAMES-1183) Error during initial request parsing

[Norman Maurer (JIRA)]

[ 
https://issues.apache.org/jira/browse/JAMES-1183?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12983611#action_12983611
 ] 

Norman Maurer commented on JAMES-1183:
--

Could you try with latest trunk and see if the problem is still there ?

> Error during initial request parsing
> 
>
> Key: JAMES-1183
> URL: https://issues.apache.org/jira/browse/JAMES-1183
> Project: JAMES Server
>  Issue Type: Bug
>  Components: IMAPServer
>Affects Versions: 3.0-M3
> Environment: Centos AMD Dual-Core 2.6 gHz 4GB RAM
>Reporter: Myron L Stewart
> Attachments: partial_wrapper.log
>
>
> I am noticing issues off and on when I try to copy messages offline into a 
> mailbox folder via IMAP. James throws and error:
> 
> DEBUG 18:20:28,945 | james.imapserver | Error during initial request parsing
> org.apache.james.imap.decode.DecodingException: Missing argument.
>   at 
> org.apache.james.imap.decode.ImapRequestLineReader.nextWordChar(ImapRequestLineReader.java:58)
>   at 
> org.apache.james.imap.decode.base.AbstractImapCommandParser.consumeWord(AbstractImapCommandParser.java:296)
>   at 
> org.apache.james.imap.decode.base.AbstractImapCommandParser.atom(AbstractImapCommandParser.java:149)
>   at 
> org.apache.james.imap.decode.main.DefaultImapDecoder.decodeCommandTagged(DefaultImapDecoder.java:83)
>   at 
> org.apache.james.imap.decode.main.DefaultImapDecoder.decode(DefaultImapDecoder.java:61)
>   at 
> org.apache.james.imap.main.AbstractImapRequestHandler.doProcessRequest(AbstractImapRequestHandler.java:62)
>   at 
> org.apache.james.imap.main.ImapRequestStreamHandler.handleRequest(ImapRequestStreamHandler.java:82)
>   at 
> org.apache.james.imapserver.netty.ImapStreamChannelUpstreamHandler.processStreamIo(ImapStreamChannelUpstreamHandler.java:88)
>   at 
> org.jboss.netty.handler.stream.StreamHandler$1.run(StreamHandler.java:127)
>   at 
> java.util.concurrent.ThreadPoolExecutor$Worker.runTask(ThreadPoolExecutor.java:886)
>   at 
> java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:908)
>   at java.lang.Thread.run(Thread.java:662)
> DEBUG 18:20:28,946 | james.imapserver | Cannot parse tag
> org.apache.james.imap.decode.DecodingException: Missing argument.
>   at 
> org.apache.james.imap.decode.ImapRequestLineReader.nextWordChar(ImapRequestLineReader.java:58)
>   at 
> org.apache.james.imap.decode.base.AbstractImapCommandParser.consumeWord(AbstractImapCommandParser.java:296)
>   at 
> org.apache.james.imap.decode.base.AbstractImapCommandParser.tag(AbstractImapCommandParser.java:158)
>   at 
> org.apache.james.imap.decode.main.DefaultImapDecoder.decode(DefaultImapDecoder.java:60)
>   at 
> org.apache.james.imap.main.AbstractImapRequestHandler.doProcessRequest(AbstractImapRequestHandler.java:62)
>   at 
> org.apache.james.imap.main.ImapRequestStreamHandler.handleRequest(ImapRequestStreamHandler.java:82)
>   at 
> org.apache.james.imapserver.netty.ImapStreamChannelUpstreamHandler.processStreamIo(ImapStreamChannelUpstreamHandler.java:88)
>   at 
> org.jboss.netty.handler.stream.StreamHandler$1.run(StreamHandler.java:127)
>   at 
> java.util.concurrent.ThreadPoolExecutor$Worker.runTask(ThreadPoolExecutor.java:886)
>   at 
> java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:908)
>   at java.lang.Thread.run(Thread.java:662)
> 
> It seems like james does not like certain tags. I've seen it complain about 
> html tags too. Do not know if this is fixed or if someone is working on it. 
> Trying to gather more information to see exact causes or what sort of e-mails 
> cause james to throw this error.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] Updated: (JAMES-424) Invalid DATA for Message shows up in Headers in mailstore

[Norman Maurer (JIRA)]

 [ 
https://issues.apache.org/jira/browse/JAMES-424?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Norman Maurer updated JAMES-424:


  Component/s: (was: MailStore & MailRepository)
Fix Version/s: 3.0-M3
 Assignee: Norman Maurer

Let us "fix" this 

> Invalid DATA for Message shows up in Headers in mailstore
> -
>
> Key: JAMES-424
> URL: https://issues.apache.org/jira/browse/JAMES-424
> Project: JAMES Server
>  Issue Type: Improvement
>  Components: SMTPServer
>Affects Versions: 2.2.0
> Environment: WinXP
>Reporter: Scott Carr
>Assignee: Norman Maurer
>Priority: Minor
> Fix For: 3.0-M3
>
> Attachments: headerFix.diff
>
>
> If you open a connection to the server, and send in the following in the DATA 
> for the message:
> This is a test
> .
> Then the data shows up in the headers instead of after the headers like it 
> should.  Here is what I recieved in the message:
> Return-Path: 
> Received: from localhost ([127.0.0.1])
>   by QRGMAIN (JAMES SMTP Server 2.2.0) with SMTP ID 792
>   for ;
>   Thu, 6 Oct 2005 21:40:21 -0500 (CDT)
> this is a test
> Date: Thu, 6 Oct 2005 21:40:21 -0500 (CDT)
> From: scott.c...@dccusa.com
> Delivered-To: scarr@localhost
> Content-Transfer-Encoding: 7bit
> Content-Type: text/plain; charset=us-ascii
> Mime-Version: 1.0
> Message-ID: <2758093.1128652823167.JavaMail.Administrator@QRGMAIN>
> Notice where 'this is a test' appears in the message.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] Created: (IMAP-248) Review package structure

[Norman Maurer (JIRA)]

Review package structure


 Key: IMAP-248
 URL: https://issues.apache.org/jira/browse/IMAP-248
 Project: JAMES Imap
  Issue Type: Task
Affects Versions: 0.2
Reporter: Norman Maurer
Assignee: Norman Maurer




-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] Resolved: (IMAP-130) Refactor api to easier support NIO frameworks such as Netty and MINA

[Norman Maurer (JIRA)]

 [ 
https://issues.apache.org/jira/browse/IMAP-130?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Norman Maurer resolved IMAP-130.


   Resolution: Fixed
Fix Version/s: 0.2
 Assignee: Norman Maurer

was done the last days. 

> Refactor api to easier support NIO frameworks such as Netty and MINA
> 
>
> Key: IMAP-130
> URL: https://issues.apache.org/jira/browse/IMAP-130
> Project: JAMES Imap
>  Issue Type: Improvement
>Reporter: Norman Maurer
>Assignee: Norman Maurer
> Fix For: 0.2
>
>
> We should refactor the api so it is easier to use "push" for the data. So it 
> would be more easy to use the framework with NIO frameworks such as Netty and 
> Mina

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] Commented: (IMAP-246) Use different read timeout when in IDLE

[Norman Maurer (JIRA)]

[ 
https://issues.apache.org/jira/browse/IMAP-246?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12983604#action_12983604
 ] 

Norman Maurer commented on IMAP-246:


>From RFC3501:

5.4.Autologout Timer

   If a server has an inactivity autologout timer, the duration of that
   timer MUST be at least 30 minutes.  The receipt of ANY command from
   the client during that interval SHOULD suffice to reset the
   autologout timer.


So 30 minutes is the smallest timeout we MUST use. I think we could still 
higher this when using IDLE, so just leave it open as "improvement" 

> Use different read timeout when in IDLE
> ---
>
> Key: IMAP-246
> URL: https://issues.apache.org/jira/browse/IMAP-246
> Project: JAMES Imap
>  Issue Type: Improvement
>  Components: Protocol
>Reporter: Niklas Therning
>Assignee: Niklas Therning
>Priority: Minor
>
> If the client doesn't send anything within the configured read timeout the 
> connection will be closed. During IDLE I think there should be another 
> timeout which is typically configured to be a lot higher than the ordinary 
> read timeout.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] Commented: (JAMES-1184) Modules consolidation

[Norman Maurer (JIRA)]

[ 
https://issues.apache.org/jira/browse/JAMES-1184?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12983603#action_12983603
 ] 

Norman Maurer commented on JAMES-1184:
--

Hi Stefano,

first of I reviewed your work and it makes sense. Thanks for taking the time. 
So now to your questions:

A) +1
B) I see your point.. Maybe we should even just merge mailetconainer-api, 
mailetcontainer-library, mailetconainer-camel to one module called 
mailetcontainer. If someone really wants to write its own implementation and 
not depend on camel he could just exclude the dependency. I think this would 
make most sense for now
C

C) +1
D) Would be resolved with proposed B) solution
E +1



> Modules consolidation
> -
>
> Key: JAMES-1184
> URL: https://issues.apache.org/jira/browse/JAMES-1184
> Project: JAMES Server
>  Issue Type: Task
>  Components: Build System
>Affects Versions: 3.0-M2
>Reporter: Stefano Bagnara
>Assignee: Stefano Bagnara
> Fix For: 3.0-M3
>
>
> Just opening it to keep track of the proposal to the mailing list.
> 
> - mail-file(3)
> - mail-jcr(1)
> - mail-jdbc(3)
> - user-jpa(4)
> - user-file(2)
> - user-jcr(2)
> - user-jdbc(4)
> - domainlist-jpa(2)
> - domainlist-xml(1)
> I only see benefits and no drawbacks in consolidating the 9 modules
> above to the following 4 modules:
>  persistence-jpa(6) = user-jpa(4)+domainlist-jpa(2)
>  persistence-file(6) = user-file(2)+mail-file(3)+domainlist-xml(1)
>  persistence-jcr(3) = user-jcr(2)+mail-jcr(1)
>  persistence-jdbc(7) = user-jdbc(4)+mail-jdbc(3)
> - mail-library(1)
> - domainlist-library(2)
> - user-library(9)
> We could consolidate them into a persistence-library module (from 3 to
> 1 module).
>  persistence-library(12) =
> mail-library(1)+domainlist-library(2)+user-library(9)
> - queue-library(2)
> - queue-jms(5)
> We could merge queue-library to queue-jms (a library module makes
> sense when we do something useful.. if it is trivial stuff it can be
> duplicated or moved to the api layer).
>  queue-jms(7) = queue-jms(5)+queue-library(2)
> -
> It seems there is agreement on the merging. We still have to decide whether 
> to name the modules "persistence-${type}" or simply "${type}" (or maybe 
> "data-${type}")

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

Re: org.apache.mailet.Mail to InputStream

[Eric Charles]

Hi Norman,

I used the mail.getMessage().toInputStream() which returned only the 
body (not the headers).

So I will use the MimeMessageInputStream(mail.getMessage()).
Tks,

Eric


On 19/01/2011 09:01, Norman Maurer wrote:

Hi Eric,

I'm a bit confused.. The MimeMessage contains everything you need. I
think this should do the job:

new MimeMessageInputStream(mail.getMessage);

Bye,
Norman


2011/1/19 Eric Charles:

Hi,

For the 2.3 import tool, I need to convert the org.apache.mailet.Mail to
InputStream (not only the MimeMessage, but the whole mail with subject,
headers,...).
Is there an existing utility class, or should I write one (one where to
place it ?) ?

Tks,

Eric


-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org




-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

Re: org.apache.mailet.Mail to InputStream

[Norman Maurer]

Hi Eric,

I'm a bit confused.. The MimeMessage contains everything you need. I
think this should do the job:

new MimeMessageInputStream(mail.getMessage);

Bye,
Norman


2011/1/19 Eric Charles :
> Hi,
>
> For the 2.3 import tool, I need to convert the org.apache.mailet.Mail to
> InputStream (not only the MimeMessage, but the whole mail with subject,
> headers,...).
> Is there an existing utility class, or should I write one (one where to
> place it ?) ?
>
> Tks,
>
> Eric
>
>
> -
> To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
> For additional commands, e-mail: server-dev-h...@james.apache.org
>
>

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org