[GitHub] james-project pull request: Update ToRecipientFolder.java to use f...

2016-01-26 Thread chibenwa
Github user chibenwa commented on the pull request: https://github.com/apache/james-project/pull/32#issuecomment-174969985 You could have a look to james-project/server/mailet/mailets/src/test/java/org/apache/james/transport/mailets/delivery/LocalDeliveryTest.java I think you

[GitHub] james-project pull request: James 1661

2016-01-25 Thread chibenwa
Github user chibenwa commented on the pull request: https://github.com/apache/james-project/pull/31#issuecomment-174482043 :+1: --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

[GitHub] james-project pull request: James 1653 Solving Java 6 compilation

2016-01-22 Thread chibenwa
Github user chibenwa closed the pull request at: https://github.com/apache/james-project/pull/28 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[GitHub] james-project pull request: JSIEVE-67

2016-01-22 Thread chibenwa
GitHub user chibenwa opened a pull request: https://github.com/apache/james-project/pull/30 JSIEVE-67 Document Sieve Integration in James. Details include : - implementation details to make new developpers life easier - How to customize your Sieve script storage

[GitHub] james-project pull request: Jsieve 103

2016-01-22 Thread chibenwa
GitHub user chibenwa opened a pull request: https://github.com/apache/james-project/pull/29 Jsieve 103 This PR is related to : apache/james-jsieve#7 This PR adapts James project to API changes in JSIEVE mailet project. These changes were first introduce in order to make it

[GitHub] james-project pull request: James 1653 Solving Java 6 compilation

2016-01-13 Thread chibenwa
GitHub user chibenwa opened a pull request: https://github.com/apache/james-project/pull/28 James 1653 Solving Java 6 compilation You can merge this pull request into a Git repository by running: $ git pull https://github.com/chibenwa/james-project JAMES-1653 Alternatively

[GitHub] james-project pull request: MAILBOX-263 improve async test stabili...

2016-01-11 Thread chibenwa
Github user chibenwa commented on the pull request: https://github.com/apache/james-project/pull/27#issuecomment-170520935 Great :+1: --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] james-project pull request: Jsieve 73 - Sieve body extension

2016-01-04 Thread chibenwa
Github user chibenwa closed the pull request at: https://github.com/apache/james-project/pull/23 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[GitHub] james-project pull request: James 1618

2016-01-04 Thread chibenwa
Github user chibenwa closed the pull request at: https://github.com/apache/james-project/pull/19 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[GitHub] james-project pull request: James 1651

2016-01-04 Thread chibenwa
Github user chibenwa commented on the pull request: https://github.com/apache/james-project/pull/24#issuecomment-168616608 :+1: --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

[GitHub] james-project pull request: Jsieve 73 - Sieve body extension

2015-12-22 Thread chibenwa
Github user chibenwa commented on the pull request: https://github.com/apache/james-project/pull/23#issuecomment-166621930 I'm going to force push here : - Deletes MesageSearcher and depend on mime4j/james-utils --- If your project is set up for it, you can reply to this

[GitHub] james-project pull request: Jsieve 73 - Sieve body extension

2015-12-22 Thread chibenwa
Github user chibenwa commented on the pull request: https://github.com/apache/james-project/pull/23#issuecomment-166621746 See https://github.com/apache/james-mime4j/pull/5 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] james-project pull request: Jsieve 73 - Sieve body extension

2015-12-20 Thread chibenwa
GitHub user chibenwa opened a pull request: https://github.com/apache/james-project/pull/23 Jsieve 73 - Sieve body extension This PR proposes patchs so that Jsieve implements RFC 5173 : Sieve Body extension. It contains SIEVE test integrtion for this functionnality. Note

[GitHub] james-project pull request: JAMES-1619 Unit tests for LocalDeliver...

2015-12-15 Thread chibenwa
Github user chibenwa closed the pull request at: https://github.com/apache/james-project/pull/20 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[GitHub] james-project pull request: James 1615

2015-12-15 Thread chibenwa
Github user chibenwa closed the pull request at: https://github.com/apache/james-project/pull/22 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[GitHub] james-project pull request: James 1615

2015-12-12 Thread chibenwa
GitHub user chibenwa opened a pull request: https://github.com/apache/james-project/pull/22 James 1615 - Provide retro compatibility on Sieve scripts - Use sieveRepository instead of direct fileSystem lookups - Configure sieveRepository using Spring You can merge this

[GitHub] james-project pull request: Mailbox 257

2015-12-12 Thread chibenwa
Github user chibenwa closed the pull request at: https://github.com/apache/james-project/pull/13 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[GitHub] james-project pull request: JAMES-1619 Unit tests for LocalDeliver...

2015-12-11 Thread chibenwa
GitHub user chibenwa opened a pull request: https://github.com/apache/james-project/pull/20 JAMES-1619 Unit tests for LocalDelivery mailet In this PR I present unit tests for LocalDelivery mailet. This mailet is used to deliver mail received from the SMTPServer to the appropriate

[GitHub] james-project pull request: James 1618

2015-12-09 Thread chibenwa
GitHub user chibenwa opened a pull request: https://github.com/apache/james-project/pull/19 James 1618 In this pull request : - I cleaned up manageSieve project - I made it RFC 5804 compliant (By implementing missing commands, and correcting existing ones) - I

[GitHub] james-project pull request: Mailbox 259 - Allow the user to choose...

2015-11-28 Thread chibenwa
Github user chibenwa closed the pull request at: https://github.com/apache/james-project/pull/15 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[GitHub] james-project pull request: Mailbox 258 - proposal to handle Mailb...

2015-11-28 Thread chibenwa
Github user chibenwa closed the pull request at: https://github.com/apache/james-project/pull/14 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[GitHub] james-project pull request: Mailbox 211 : distributed event system...

2015-11-28 Thread chibenwa
Github user chibenwa closed the pull request at: https://github.com/apache/james-project/pull/12 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[GitHub] james-project pull request: Protocols 68

2015-11-27 Thread chibenwa
Github user chibenwa commented on the pull request: https://github.com/apache/james-project/pull/11#issuecomment-160107839 Delete ~2000 not needed lines of code... Unit tests pending, but it should be OK. --- If your project is set up for it, you can reply to this email and have

[GitHub] james-project pull request: JAMES-1635 Increase embedded Cassandra...

2015-11-27 Thread chibenwa
Github user chibenwa closed the pull request at: https://github.com/apache/james-project/pull/17 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[GitHub] james-project pull request: JAMES-1635 Increase embedded Cassandra...

2015-11-27 Thread chibenwa
Github user chibenwa commented on the pull request: https://github.com/apache/james-project/pull/17#issuecomment-160107688 Committed --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] james-project pull request: JAMES-1635 Increase embedded Cassandra...

2015-11-04 Thread chibenwa
Github user chibenwa commented on a diff in the pull request: https://github.com/apache/james-project/pull/17#discussion_r43896960 --- Diff: backends-common/cassandra/src/test/java/org/apache/james/backends/cassandra/CassandraCluster.java --- @@ -42,10 +42,12

[GitHub] james-project pull request: Mailbox 211 : distributed event system...

2015-11-04 Thread chibenwa
Github user chibenwa commented on the pull request: https://github.com/apache/james-project/pull/12#issuecomment-153723146 I updated the PR after Matthieu comments. It still needs to be unit test validated ( will take ~ an hour ). --- If your project is set up for it, you

[GitHub] james-project pull request: JAMES-1635 Increase embedded Cassandra...

2015-11-04 Thread chibenwa
GitHub user chibenwa opened a pull request: https://github.com/apache/james-project/pull/17 JAMES-1635 Increase embedded Cassandra startup timeout Increase embedded Cassandra startup timeout to the value configured in CassandraCluster used for unit tests. This will avoid

[GitHub] james-project pull request: Mailbox 257

2015-11-02 Thread chibenwa
Github user chibenwa commented on a diff in the pull request: https://github.com/apache/james-project/pull/13#discussion_r43659960 --- Diff: mailbox/tool/src/main/java/org/apache/james/mailbox/indexer/registrations/MailboxRegistration.java --- @@ -0,0 +1,62

[GitHub] james-project pull request: Mailbox 257

2015-11-02 Thread chibenwa
Github user chibenwa commented on a diff in the pull request: https://github.com/apache/james-project/pull/13#discussion_r43607641 --- Diff: mailbox/tool/src/main/java/org/apache/james/mailbox/indexer/registrations/MailboxRegistration.java --- @@ -0,0 +1,62

[GitHub] james-project pull request: Mailbox 257

2015-11-02 Thread chibenwa
Github user chibenwa commented on a diff in the pull request: https://github.com/apache/james-project/pull/13#discussion_r43607008 --- Diff: mailbox/tool/src/main/java/org/apache/james/mailbox/indexer/ReIndexerImpl.java --- @@ -0,0 +1,138

[GitHub] james-project pull request: Mailbox 257

2015-11-02 Thread chibenwa
Github user chibenwa commented on the pull request: https://github.com/apache/james-project/pull/13#issuecomment-152959494 Thanks for the review. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] james-project pull request: Mailbox 257

2015-11-02 Thread chibenwa
Github user chibenwa commented on a diff in the pull request: https://github.com/apache/james-project/pull/13#discussion_r43606970 --- Diff: mailbox/tool/src/main/java/org/apache/james/mailbox/indexer/registrations/MailboxRegistration.java --- @@ -0,0 +1,62

[GitHub] james-project pull request: Mailbox 257

2015-11-02 Thread chibenwa
Github user chibenwa commented on a diff in the pull request: https://github.com/apache/james-project/pull/13#discussion_r43606917 --- Diff: mailbox/tool/src/main/java/org/apache/james/mailbox/indexer/ReIndexerImpl.java --- @@ -0,0 +1,138

[GitHub] james-project pull request: Mailbox 257

2015-11-02 Thread chibenwa
Github user chibenwa commented on a diff in the pull request: https://github.com/apache/james-project/pull/13#discussion_r43606920 --- Diff: mailbox/tool/src/main/java/org/apache/james/mailbox/indexer/ReIndexerImpl.java --- @@ -0,0 +1,138

[GitHub] james-project pull request: Mailbox 257

2015-11-02 Thread chibenwa
Github user chibenwa commented on a diff in the pull request: https://github.com/apache/james-project/pull/13#discussion_r43606900 --- Diff: mailbox/tool/src/main/java/org/apache/james/mailbox/indexer/ReIndexerImpl.java --- @@ -0,0 +1,138

[GitHub] james-project pull request: Mailbox 257

2015-11-02 Thread chibenwa
Github user chibenwa commented on a diff in the pull request: https://github.com/apache/james-project/pull/13#discussion_r43606581 --- Diff: mailbox/tool/src/main/java/org/apache/james/mailbox/indexer/ReIndexerImpl.java --- @@ -0,0 +1,138

[GitHub] james-project pull request: Mailbox 257

2015-11-02 Thread chibenwa
Github user chibenwa commented on a diff in the pull request: https://github.com/apache/james-project/pull/13#discussion_r43606590 --- Diff: mailbox/tool/src/main/java/org/apache/james/mailbox/indexer/ReIndexerImpl.java --- @@ -0,0 +1,138

[GitHub] james-project pull request: Mailbox 257

2015-11-02 Thread chibenwa
Github user chibenwa commented on a diff in the pull request: https://github.com/apache/james-project/pull/13#discussion_r43606188 --- Diff: mailbox/tool/src/main/java/org/apache/james/mailbox/indexer/ReIndexerImpl.java --- @@ -0,0 +1,138

[GitHub] james-project pull request: Mailbox 257

2015-11-02 Thread chibenwa
Github user chibenwa commented on a diff in the pull request: https://github.com/apache/james-project/pull/13#discussion_r43606130 --- Diff: mailbox/tool/src/main/java/org/apache/james/mailbox/indexer/ReIndexerImpl.java --- @@ -0,0 +1,138

[GitHub] james-project pull request: Mailbox 257

2015-11-02 Thread chibenwa
Github user chibenwa commented on a diff in the pull request: https://github.com/apache/james-project/pull/13#discussion_r43606072 --- Diff: mailbox/tool/src/main/java/org/apache/james/mailbox/indexer/FakeReIndexer.java --- @@ -0,0 +1,36

[GitHub] james-project pull request: Mailbox 259 - Allow the user to choose...

2015-10-31 Thread chibenwa
GitHub user chibenwa opened a pull request: https://github.com/apache/james-project/pull/15 Mailbox 259 - Allow the user to choose between synchronous and asynchronous event delivery I added to the event system defined in MAILBOX-211 the possibility to define how to handle event

[GitHub] james-project pull request: Mailbox 258 - proposal to handle Mailb...

2015-10-31 Thread chibenwa
GitHub user chibenwa opened a pull request: https://github.com/apache/james-project/pull/14 Mailbox 258 - proposal to handle Mailbox Renamed events. Proposale about MAILBOX-258 ... It is costly in terms of performance, so if you have other ideas, I take it... By now I

[GitHub] james-project pull request: Mailbox 257

2015-10-31 Thread chibenwa
GitHub user chibenwa opened a pull request: https://github.com/apache/james-project/pull/13 Mailbox 257 This tool can be used if you change the technology you used as an indexer. It an also be used to correct index inconsistency. These inconsistencies can

[GitHub] james-project pull request: Mailbox 211 : distributed event system...

2015-10-30 Thread chibenwa
GitHub user chibenwa opened a pull request: https://github.com/apache/james-project/pull/12 Mailbox 211 : distributed event systems In this PR I propose 3 kind of event system : - the default one : the previous one, revisited. Better failure handling, and package relocation

[GitHub] james-project pull request: James 1616

2015-10-30 Thread chibenwa
Github user chibenwa closed the pull request at: https://github.com/apache/james-project/pull/7 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] james-project pull request: Protocols 68

2015-10-30 Thread chibenwa
Github user chibenwa commented on the pull request: https://github.com/apache/james-project/pull/11#issuecomment-152489970 Related to https://github.com/apache/james-jsieve/pull/3 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] james-project pull request: Protocols 68

2015-10-30 Thread chibenwa
GitHub user chibenwa opened a pull request: https://github.com/apache/james-project/pull/11 Protocols 68 Relocate existing ManageSieve stuff. You can merge this pull request into a Git repository by running: $ git pull https://github.com/chibenwa/james-project PROTOCOLS-68

[GitHub] james-project pull request: James 1626

2015-10-26 Thread chibenwa
Github user chibenwa commented on the pull request: https://github.com/apache/james-project/pull/9#issuecomment-151163425 I really like the docker files becoming parts of james-project. --- If your project is set up for it, you can reply to this email and have your reply appear on