Re: [PR] JAMES-2586 Integration tests for JMAP postgres [james-project]

2024-02-27 Thread via GitHub
hungphan227 commented on code in PR #2029: URL: https://github.com/apache/james-project/pull/2029#discussion_r1505490515 ## backends-common/postgres/src/test/java/org/apache/james/backends/postgres/PostgresExtension.java: ## @@ -178,6 +178,10 @@ public void

(james-project) branch master updated: JAMES-4012 Lenient MDN parsing for human readable part (#2057)

2024-02-27 Thread btellier
This is an automated email from the ASF dual-hosted git repository. btellier pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/james-project.git The following commit(s) were added to refs/heads/master by this push: new a63d257f2c JAMES-4012 Lenient MDN

Re: [PR] JAMES-4012 Lenient MDN parsing for human readable part [james-project]

2024-02-27 Thread via GitHub
chibenwa merged PR #2057: URL: https://github.com/apache/james-project/pull/2057 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

(james-site) branch asf-staging updated: Site checkin for project Apache James: Jenkins Tools

2024-02-27 Thread git-site-role
This is an automated email from the ASF dual-hosted git repository. git-site-role pushed a commit to branch asf-staging in repository https://gitbox.apache.org/repos/asf/james-site.git The following commit(s) were added to refs/heads/asf-staging by this push: new b793df1ba Site checkin for

Re: [PR] JAMES-2586 Integration tests for JMAP postgres [james-project]

2024-02-27 Thread via GitHub
Arsnael commented on PR #2029: URL: https://github.com/apache/james-project/pull/2029#issuecomment-1968365306 @hungphan227 related? https://ci-builds.apache.org/job/james/job/ApacheJames/job/PR-2029/15/ -- This is an automated message from the Apache Git Service. To respond to the

Re: [PR] JAMES-4003 [FIX] Avoid forwarding bounces as it might create infinite loops [james-project]

2024-02-27 Thread via GitHub
chibenwa commented on code in PR #2022: URL: https://github.com/apache/james-project/pull/2022#discussion_r1505428445 ## server/mailet/mailets/src/main/java/org/apache/james/transport/mailets/RecipientRewriteTableProcessor.java: ## @@ -182,7 +186,8 @@ public

Re: [PR] WIP Enforce a separate keyspace for blob store cache [james-project]

2024-02-27 Thread via GitHub
chibenwa commented on code in PR #2060: URL: https://github.com/apache/james-project/pull/2060#discussion_r1505427901 ## server/container/guice/cassandra/src/main/java/org/apache/james/modules/mailbox/CassandraCacheSessionModule.java: ## @@ -72,12 +71,8 @@ static class

Re: [PR] JAMES 3996 Redis event bus POC [james-project]

2024-02-27 Thread via GitHub
Maxxx873 commented on PR #2028: URL: https://github.com/apache/james-project/pull/2028#issuecomment-1968287999 > > Hello! If I understand correctly, then after completing this task, the task that I previously started https://issues.apache.org/jira/browse/JAMES-3956 will be closed. You do

Re: [PR] JAMES 3996 Redis event bus POC [james-project]

2024-02-27 Thread via GitHub
quantranhong1999 commented on PR #2028: URL: https://github.com/apache/james-project/pull/2028#issuecomment-1968250395 > Hello! If I understand correctly, then after completing this task, the task that I previously started https://issues.apache.org/jira/browse/JAMES-3956 will be closed.

Re: [PR] JAMES 3996 Redis event bus POC [james-project]

2024-02-27 Thread via GitHub
Maxxx873 commented on PR #2028: URL: https://github.com/apache/james-project/pull/2028#issuecomment-1968243951 Hello! If I understand correctly, then after completing this task, the task that I previously started https://issues.apache.org/jira/browse/JAMES-3956 will be closed. You do it

Re: [PR] JAMES-2586 Integration tests for JMAP postgres [james-project]

2024-02-27 Thread via GitHub
quantranhong1999 commented on code in PR #2029: URL: https://github.com/apache/james-project/pull/2029#discussion_r1505331219 ## backends-common/postgres/src/test/java/org/apache/james/backends/postgres/PostgresExtension.java: ## @@ -178,6 +178,10 @@ public void

Re: [PR] [FIX] Remove a benign log in DeletedMessageVaultDeletionCallback [james-project]

2024-02-27 Thread via GitHub
Arsnael merged PR #2059: URL: https://github.com/apache/james-project/pull/2059 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

(james-project) 01/02: [FIX] Drop unused code in DeletedMessageVaultDeletionCallback

2024-02-27 Thread rcordier
This is an automated email from the ASF dual-hosted git repository. rcordier pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/james-project.git commit f7a898ad36e1f7ce41d98cb2f67da1fcc4cc2b56 Author: Benoit TELLIER AuthorDate: Tue Feb 27 11:41:27 2024 +0100

Re: [PR] [FIX] java.util.NoSuchElementException: requestDecoder [james-project]

2024-02-27 Thread via GitHub
Arsnael merged PR #2061: URL: https://github.com/apache/james-project/pull/2061 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

(james-project) branch master updated (80b88e42ee -> df87f0b4b1)

2024-02-27 Thread rcordier
This is an automated email from the ASF dual-hosted git repository. rcordier pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/james-project.git from 80b88e42ee [FIX] Lower log message in MaybeSender add df87f0b4b1 [FIX]

(james-project) branch master updated (e48f8fcde7 -> 80b88e42ee)

2024-02-27 Thread rcordier
This is an automated email from the ASF dual-hosted git repository. rcordier pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/james-project.git from e48f8fcde7 [BUILD] RemoteDeliveryErrorHandlingTest was unstable new f7a898ad36 [FIX] Drop unused code

(james-project) 02/02: [FIX] Lower log message in MaybeSender

2024-02-27 Thread rcordier
This is an automated email from the ASF dual-hosted git repository. rcordier pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/james-project.git commit 80b88e42ee6ebc3460f3a85c1be3376ffa22 Author: Benoit TELLIER AuthorDate: Tue Feb 27 11:41:54 2024 +0100

Re: [PR] Revert git commit id maven plugin [james-project]

2024-02-27 Thread via GitHub
Arsnael commented on PR #2049: URL: https://github.com/apache/james-project/pull/2049#issuecomment-1968165547 Thanks @jeantil for the suggestion, I think that might do it actually and it's rather clean. I'm in favor, thanks for making the PR too @chibenwa Closing this as a result.

Re: [PR] Revert git commit id maven plugin [james-project]

2024-02-27 Thread via GitHub
Arsnael closed pull request #2049: Revert git commit id maven plugin URL: https://github.com/apache/james-project/pull/2049 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

(james-project) branch master updated: [BUILD] RemoteDeliveryErrorHandlingTest was unstable

2024-02-27 Thread rcordier
This is an automated email from the ASF dual-hosted git repository. rcordier pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/james-project.git The following commit(s) were added to refs/heads/master by this push: new e48f8fcde7 [BUILD]

Re: [PR] [BUILD] RemoteDeliveryErrorHandlingTest was unstable [james-project]

2024-02-27 Thread via GitHub
Arsnael merged PR #2064: URL: https://github.com/apache/james-project/pull/2064 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

(james-project) branch master updated: JAMES-4013 Relay MDN/send

2024-02-27 Thread rcordier
This is an automated email from the ASF dual-hosted git repository. rcordier pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/james-project.git The following commit(s) were added to refs/heads/master by this push: new 326b892ac1 JAMES-4013 Relay MDN/send

Re: [PR] JAMES-4013 Relay MDN/send [james-project]

2024-02-27 Thread via GitHub
Arsnael merged PR #2062: URL: https://github.com/apache/james-project/pull/2062 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

(james-project) branch master updated: JAMES-4011 Use MIME4J to decode filename

2024-02-27 Thread rcordier
This is an automated email from the ASF dual-hosted git repository. rcordier pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/james-project.git The following commit(s) were added to refs/heads/master by this push: new 4f77aab13a JAMES-4011 Use MIME4J to

Re: [PR] JAMES-4011 Use MIME4J to decode filename [james-project]

2024-02-27 Thread via GitHub
Arsnael merged PR #2056: URL: https://github.com/apache/james-project/pull/2056 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

(james-project) branch master updated: [UPGRADE] MIME4J 0.8.9 -> 0.8.10 (#2058)

2024-02-27 Thread rcordier
This is an automated email from the ASF dual-hosted git repository. rcordier pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/james-project.git The following commit(s) were added to refs/heads/master by this push: new fc1966682f [UPGRADE] MIME4J 0.8.9 ->

Re: [PR] [UPGRADE] MIME4J 0.8.9 -> 0.8.10 [james-project]

2024-02-27 Thread via GitHub
Arsnael merged PR #2058: URL: https://github.com/apache/james-project/pull/2058 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] WIP Enforce a separate keyspace for blob store cache [james-project]

2024-02-27 Thread via GitHub
Arsnael commented on code in PR #2060: URL: https://github.com/apache/james-project/pull/2060#discussion_r1505244208 ## server/container/guice/cassandra/src/main/java/org/apache/james/modules/mailbox/CassandraCacheSessionModule.java: ## @@ -72,12 +71,8 @@ static class

Re: [PR] WIP Enforce a separate keyspace for blob store cache [james-project]

2024-02-27 Thread via GitHub
Arsnael commented on code in PR #2060: URL: https://github.com/apache/james-project/pull/2060#discussion_r1505244208 ## server/container/guice/cassandra/src/main/java/org/apache/james/modules/mailbox/CassandraCacheSessionModule.java: ## @@ -72,12 +71,8 @@ static class

Re: [PR] JAMES-4003 [FIX] Avoid forwarding bounces as it might create infinite loops [james-project]

2024-02-27 Thread via GitHub
Arsnael commented on code in PR #2022: URL: https://github.com/apache/james-project/pull/2022#discussion_r1505240328 ## server/mailet/mailets/src/main/java/org/apache/james/transport/mailets/RecipientRewriteTableProcessor.java: ## @@ -182,7 +186,8 @@ public

[PR] [BUILD] RemoteDeliveryErrorHandlingTest was unstable [james-project]

2024-02-27 Thread via GitHub
chibenwa opened a new pull request, #2064: URL: https://github.com/apache/james-project/pull/2064 As we did not await task execution the seond mail repository reprocessing could be triggered while the first one was not even started which would go undetected. Strictier awaits does

(james-project) 01/01: [BUILD] RemoteDeliveryErrorHandlingTest was unstable

2024-02-27 Thread btellier
This is an automated email from the ASF dual-hosted git repository. btellier pushed a commit to branch build-stability-27-02-20204 in repository https://gitbox.apache.org/repos/asf/james-project.git commit 95b736e7825311e128e42818e41129e8268aa8c2 Author: Benoit TELLIER AuthorDate: Tue Feb 27

(james-project) branch build-stability-27-02-20204 created (now 95b736e782)

2024-02-27 Thread btellier
This is an automated email from the ASF dual-hosted git repository. btellier pushed a change to branch build-stability-27-02-20204 in repository https://gitbox.apache.org/repos/asf/james-project.git at 95b736e782 [BUILD] RemoteDeliveryErrorHandlingTest was unstable This branch includes

(james-project) branch master updated: JAMES-3885 Webadmin route to change of username - support `force` parameter - do not require old user to exist (#2054)

2024-02-27 Thread btellier
This is an automated email from the ASF dual-hosted git repository. btellier pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/james-project.git The following commit(s) were added to refs/heads/master by this push: new 4f064d3ee3 JAMES-3885 Webadmin route

Re: [PR] JAMES-3885 Webadmin route to change of username - support `force` parameter - do not require old user to exist [james-project]

2024-02-27 Thread via GitHub
chibenwa merged PR #2054: URL: https://github.com/apache/james-project/pull/2054 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] MailSizeEsmtpExtension - change log level when Quota exceeded [james-project]

2024-02-27 Thread via GitHub
chibenwa merged PR #2053: URL: https://github.com/apache/james-project/pull/2053 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

(james-project) branch master updated: MailSizeEsmtpExtension - change log level when Quota exceeded (#2053)

2024-02-27 Thread btellier
This is an automated email from the ASF dual-hosted git repository. btellier pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/james-project.git The following commit(s) were added to refs/heads/master by this push: new 4f38296f28 MailSizeEsmtpExtension -

Re: [PR] Revert git commit id maven plugin [james-project]

2024-02-27 Thread via GitHub
chibenwa commented on PR #2049: URL: https://github.com/apache/james-project/pull/2049#issuecomment-1966592116 https://github.com/apache/james-project/pull/2063 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[PR] [BUILD] Use git-commit-id-maven-plugin in ci-test [james-project]

2024-02-27 Thread via GitHub
chibenwa opened a new pull request, #2063: URL: https://github.com/apache/james-project/pull/2063 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [PR] [BUILD] Use git-commit-id-maven-plugin in server/apps only [james-project]

2024-02-27 Thread via GitHub
chibenwa closed pull request #2050: [BUILD] Use git-commit-id-maven-plugin in server/apps only URL: https://github.com/apache/james-project/pull/2050 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

Re: [PR] [BUILD] Use git-commit-id-maven-plugin in server/apps only [james-project]

2024-02-27 Thread via GitHub
chibenwa commented on PR #2050: URL: https://github.com/apache/james-project/pull/2050#issuecomment-1966559478 You convinced me @jeantil -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[PR] JAMES-4013 Relay MDN/send [james-project]

2024-02-27 Thread via GitHub
chibenwa opened a new pull request, #2062: URL: https://github.com/apache/james-project/pull/2062 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [PR] [BUILD] Use git-commit-id-maven-plugin in server/apps only [james-project]

2024-02-27 Thread via GitHub
jeantil commented on PR #2050: URL: https://github.com/apache/james-project/pull/2050#issuecomment-1966546136 It's should be possible to completely disable the plugin by command line parameter. I'll try to look it up but as I said limited connectivity Ideally the plugin should be

[PR] [FIX] java.util.NoSuchElementException: requestDecoder [james-project]

2024-02-27 Thread via GitHub
chibenwa opened a new pull request, #2061: URL: https://github.com/apache/james-project/pull/2061 Instead, we replace the framer in-place -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[PR] WIP Enforce a separate keyspace for blob store cache [james-project]

2024-02-27 Thread via GitHub
chibenwa opened a new pull request, #2060: URL: https://github.com/apache/james-project/pull/2060 This was broken, likely upon the cassandra driver upgrade -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

[PR] [FIX] Remove a benign log in DeletedMessageVaultDeletionCallback [james-project]

2024-02-27 Thread via GitHub
chibenwa opened a new pull request, #2059: URL: https://github.com/apache/james-project/pull/2059 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [PR] JAMES-4003 [FIX] Avoid forwarding bounces as it might create infinite loops [james-project]

2024-02-27 Thread via GitHub
chibenwa commented on PR #2022: URL: https://github.com/apache/james-project/pull/2022#issuecomment-1966254047 Any thoughts on this work? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[PR] [UPGRADE] MIME4J 0.8.9 -> 0.8.10 [james-project]

2024-02-27 Thread via GitHub
chibenwa opened a new pull request, #2058: URL: https://github.com/apache/james-project/pull/2058 Solves https://github.com/linagora/james-project/issues/5080 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] JAMES-4007 Getting a second shot at IMAP large litteral management [james-project]

2024-02-27 Thread via GitHub
Arsnael commented on PR #2052: URL: https://github.com/apache/james-project/pull/2052#issuecomment-1966185075 09:47:37,400 [ERROR] Failed to execute goal org.apache.maven.plugins:maven-checkstyle-plugin:3.3.0:check (check-style) on project james-server-protocols-imap4: You have 2

Re: [PR] JAMES-2586 Integration tests for JMAP postgres [james-project]

2024-02-27 Thread via GitHub
Arsnael commented on PR #2029: URL: https://github.com/apache/james-project/pull/2029#issuecomment-1966104255 I think should do it... the rabbitmq.properties file that you added @hungphan227 created issues. Because of it it was thinking of using rabbitmq for all tests classes in

[PR] JAMES-4012 Lenient MDN parsing for human readable part [james-project]

2024-02-27 Thread via GitHub
chibenwa opened a new pull request, #2057: URL: https://github.com/apache/james-project/pull/2057 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [PR] JAMES-3885 Webadmin route to change of username - support `force` parameter - do not require old user to exist [james-project]

2024-02-27 Thread via GitHub
Arsnael commented on code in PR #2054: URL: https://github.com/apache/james-project/pull/2054#discussion_r1503795915 ## server/protocols/webadmin/webadmin-data/src/test/java/org/apache/james/webadmin/routes/UsernameChangeRoutesTest.java: ## @@ -191,6 +192,18 @@ void

Re: [PR] [BUILD] Use git-commit-id-maven-plugin in server/apps only [james-project]

2024-02-27 Thread via GitHub
Arsnael commented on PR #2050: URL: https://github.com/apache/james-project/pull/2050#issuecomment-1965986660 > We do not have a command line option to diable that plugin in tmail build? https://github.com/apache/james-project/pull/2049#issuecomment-1963452919 -- This is an

Re: [PR] JAMES-3885 Webadmin route to change of username - support `force` parameter - do not require old user to exist [james-project]

2024-02-27 Thread via GitHub
vttranlina commented on code in PR #2054: URL: https://github.com/apache/james-project/pull/2054#discussion_r1503794703 ## server/protocols/webadmin/webadmin-data/src/test/java/org/apache/james/webadmin/routes/UsernameChangeRoutesTest.java: ## @@ -191,6 +192,18 @@ void

Re: [PR] JAMES-3885 Webadmin route to change of username - support `force` parameter - do not require old user to exist [james-project]

2024-02-27 Thread via GitHub
Arsnael commented on code in PR #2054: URL: https://github.com/apache/james-project/pull/2054#discussion_r1503791303 ## server/protocols/webadmin/webadmin-data/src/test/java/org/apache/james/webadmin/routes/UsernameChangeRoutesTest.java: ## @@ -191,6 +192,18 @@ void