[PR] JAMES-4147 SanitizeMimeMessageId Mailet [james-project]

2025-09-20 Thread via GitHub
chibenwa opened a new pull request, #2818: URL: https://github.com/apache/james-project/pull/2818 Could have been convenient for finding a mail based on processing logs in OpenSearch... -- This is an automated message from the Apache Git Service. To respond to the message, please log on t

[PR] JAMES-3755 Allow RSpamDScanner to override url settings [james-project]

2025-09-19 Thread via GitHub
chibenwa opened a new pull request, #2819: URL: https://github.com/apache/james-project/pull/2819 A possible use case is to use 1 RSpamD cluster on user incoming spam, trained in perUserBayes mode, and another RSpamD cluster configured to check outgoing email for spams with a tolerant thres

Re: [PR] [ENHANCEMENT] DefaultCheckTokenClient: increase pending acquire max [james-project]

2025-09-19 Thread via GitHub
chibenwa commented on code in PR #2816: URL: https://github.com/apache/james-project/pull/2816#discussion_r2362383847 ## server/protocols/jwt/src/main/java/org/apache/james/jwt/DefaultCheckTokenClient.java: ## @@ -42,11 +43,15 @@ public class DefaultCheckTokenClient implements

Re: [PR] [ENHANCEMENT] DefaultCheckTokenClient: increase pending acquire max [james-project]

2025-09-19 Thread via GitHub
chibenwa commented on code in PR #2816: URL: https://github.com/apache/james-project/pull/2816#discussion_r2362383847 ## server/protocols/jwt/src/main/java/org/apache/james/jwt/DefaultCheckTokenClient.java: ## @@ -42,11 +43,15 @@ public class DefaultCheckTokenClient implements

[PR] Ability to run rules on folders [james-project]

2025-09-19 Thread via GitHub
Arsnael opened a new pull request, #2817: URL: https://github.com/apache/james-project/pull/2817 WIP, just started... -- 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 uns

[PR] [ENHANCEMENT] DefaultCheckTokenClient: increase pending acquire max [james-project]

2025-09-19 Thread via GitHub
quantranhong1999 opened a new pull request, #2816: URL: https://github.com/apache/james-project/pull/2816 The default pending acquire was low (~32, depending on machines). The OIDC client could fail under high load. ```java "exception":"reactor.netty.internal.shaded.reactor.pool.P

Re: [PR] [BUILD] Improve test execution time - batch 2 [james-project]

2025-09-19 Thread via GitHub
quantranhong1999 merged PR #2813: URL: https://github.com/apache/james-project/pull/2813 -- 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: notifications-u

Re: [PR] [BUILD] Improve test execution time - batch 2 [james-project]

2025-09-18 Thread via GitHub
quantranhong1999 commented on PR #2813: URL: https://github.com/apache/james-project/pull/2813#issuecomment-3310367445 37 minutes of build (25m tests) now :-) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abo

Re: [PR] [BUILD] Improve test execution time [james-project]

2025-09-18 Thread via GitHub
chibenwa commented on PR #2813: URL: https://github.com/apache/james-project/pull/2813#issuecomment-3302172291 https://github.com/user-attachments/assets/65cd680e-15f4-45b0-a0c4-6d254483eb40"; /> master took 2h05 Test run: 43 147 Test run on mater: 44 509 Delta in

Re: [PR] [BUILD] Improve test execution time - batch 1: consensual part [james-project]

2025-09-18 Thread via GitHub
jeantil commented on PR #2814: URL: https://github.com/apache/james-project/pull/2814#issuecomment-3303377289 out of curiosity why aim for `forkCount` instead of -T2 ? the test suite is executed by ` mvn -B -e -fae clean test -Dorg.slf4j.simpleLogger.showDateTime=true ` which do

Re: [PR] [BUILD TIME] james-server-webadmin-mailbox: use lucene instead of Ope… [james-project]

2025-09-18 Thread via GitHub
jeantil commented on PR #2815: URL: https://github.com/apache/james-project/pull/2815#issuecomment-3305811124 nice ! -- 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 unsu

Re: [PR] [BUILD] Improve test execution time - batch 2 [james-project]

2025-09-18 Thread via GitHub
quantranhong1999 commented on PR #2813: URL: https://github.com/apache/james-project/pull/2813#issuecomment-3306483984 I just force pushed the second batch. This batch focuses more on removing duplicate ITs. -- This is an automated message from the Apache Git Service. To respond to the me

[PR] JAMES-4057 Align subject search with other implementations [james-project]

2025-09-18 Thread via GitHub
chibenwa opened a new pull request, #2812: URL: https://github.com/apache/james-project/pull/2812 (sorry for missing thins out) -- 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 commen

Re: [PR] JAMES-4057 Align subject search with other implementations [james-project]

2025-09-18 Thread via GitHub
chibenwa merged PR #2812: URL: https://github.com/apache/james-project/pull/2812 -- 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: notifications-unsubscr.

Re: [PR] [BUILD TIME] james-server-webadmin-mailbox: use lucene instead of Ope… [james-project]

2025-09-17 Thread via GitHub
chibenwa commented on PR #2815: URL: https://github.com/apache/james-project/pull/2815#issuecomment-3305658622 We gained 5 minutes of build time on `james-server-webadmin-mailbox` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHu

Re: [PR] [BUILD] Improve test execution time - batch 2 [james-project]

2025-09-17 Thread via GitHub
quantranhong1999 commented on code in PR #2813: URL: https://github.com/apache/james-project/pull/2813#discussion_r2357374879 ## server/apps/cassandra-app/pom.xml: ## @@ -326,6 +326,13 @@ + +org.ap

Re: [PR] [BUILD] Improve test execution time - batch 2 [james-project]

2025-09-17 Thread via GitHub
quantranhong1999 commented on code in PR #2813: URL: https://github.com/apache/james-project/pull/2813#discussion_r2357375199 ## server/protocols/webadmin/webadmin-mailbox/pom.xml: ## @@ -259,6 +259,7 @@ -javaagent:"${settings.localRepository}"/org/jaco

[PR] [ENHANCEMENT] Smtp technical accounts [james-project]

2025-09-17 Thread via GitHub
chibenwa opened a new pull request, #2810: URL: https://github.com/apache/james-project/pull/2810 My need: Port 25: authentication is disabled Port 465: submission, user auth, strict sender validation Port 26: opened locally with k8s cluster. I want to add auth for it not to be

Re: [PR] [BUILD TIME] james-server-webadmin-mailbox: use lucene instead of Ope… [james-project]

2025-09-17 Thread via GitHub
jeantil commented on PR #2815: URL: https://github.com/apache/james-project/pull/2815#issuecomment-3305522879 sorry i'm a bit tired and don´t quite remember the original numbers, here is what the build scan says about slow tests in this build https://develocity.apache.org/s/sugjiob3gwhpu

Re: [PR] [BUILD] Improve test execution time - batch 2 [james-project]

2025-09-17 Thread via GitHub
jeantil commented on code in PR #2813: URL: https://github.com/apache/james-project/pull/2813#discussion_r2357583622 ## server/apps/cassandra-app/pom.xml: ## @@ -326,6 +326,13 @@ + +org.apache.mave

Re: [PR] [BUILD] Improve test execution time - batch 2 [james-project]

2025-09-17 Thread via GitHub
jeantil commented on code in PR #2813: URL: https://github.com/apache/james-project/pull/2813#discussion_r2357436857 ## server/apps/cassandra-app/pom.xml: ## @@ -326,6 +326,13 @@ + +org.apache.mave

Re: [PR] [BUILD] Improve test execution time - batch 1 [james-project]

2025-09-17 Thread via GitHub
quantranhong1999 commented on PR #2814: URL: https://github.com/apache/james-project/pull/2814#issuecomment-3305325461 Hi @jeantil, I just ran `mvn -B -e -fae clean test -Dorg.slf4j.simpleLogger.showDateTime=true -Ddevelocity.cache.local.enabled=false` on the `james-server-webadmin-m

Re: [PR] [BUILD] Improve test execution time - batch 1 [james-project]

2025-09-17 Thread via GitHub
quantranhong1999 commented on PR #2814: URL: https://github.com/apache/james-project/pull/2814#issuecomment-3305385752 Just an example of `forkCount` result on the CI: https://github.com/apache/james-project/pull/2813#discussion_r2357445005 But I understand this is not representative

Re: [PR] [BUILD] Improve test execution time - batch 1 [james-project]

2025-09-17 Thread via GitHub
quantranhong1999 merged PR #2814: URL: https://github.com/apache/james-project/pull/2814 -- 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: notifications-u

Re: [PR] [BUILD] Improve test execution time - batch 1 [james-project]

2025-09-17 Thread via GitHub
jeantil commented on PR #2814: URL: https://github.com/apache/james-project/pull/2814#issuecomment-3305367985 go ahaead as part of > Other improvements from your side are welcome too (maybe in another PR). I might just try to disable all this forkcount tuning and see how it affects

Re: [PR] [BUILD] Improve test execution time - batch 2 [james-project]

2025-09-17 Thread via GitHub
quantranhong1999 commented on code in PR #2813: URL: https://github.com/apache/james-project/pull/2813#discussion_r2357447847 ## server/apps/cassandra-app/pom.xml: ## @@ -326,6 +326,13 @@ + +org.ap

Re: [PR] [BUILD] Improve test execution time - batch 2 [james-project]

2025-09-17 Thread via GitHub
quantranhong1999 commented on code in PR #2813: URL: https://github.com/apache/james-project/pull/2813#discussion_r2357445005 ## server/apps/distributed-app/pom.xml: ## @@ -501,6 +501,7 @@ -javaagent:"${settings.localRepository}"/org/jacoco/org.jacoco.a

Re: [PR] [BUILD] Improve test execution time - batch 2 [james-project]

2025-09-17 Thread via GitHub
jeantil commented on code in PR #2813: URL: https://github.com/apache/james-project/pull/2813#discussion_r2357437132 ## server/apps/cassandra-app/pom.xml: ## @@ -326,6 +326,13 @@ + +org.apache.mave

Re: [PR] [BUILD] Improve test execution time - batch 1 [james-project]

2025-09-17 Thread via GitHub
jeantil commented on PR #2814: URL: https://github.com/apache/james-project/pull/2814#issuecomment-3305301730 I have been thinking about this again. I would much prefer we merge the changes incrementally and accumulate datapoints before merging the next set of changes. This is especi

Re: [PR] [BUILD] Improve test execution time - batch 1 [james-project]

2025-09-17 Thread via GitHub
quantranhong1999 commented on PR #2814: URL: https://github.com/apache/james-project/pull/2814#issuecomment-3305163378 Hi @jeantil is this work ok for you overall? Other improvements from your side are welcome too (maybe in another PR). -- This is an automated message from the Apache Git

Re: [PR] Jmap filter start-with [james-project]

2025-09-17 Thread via GitHub
chibenwa commented on code in PR #2801: URL: https://github.com/apache/james-project/pull/2801#discussion_r2336297138 ## server/protocols/jmap-rfc-8621/src/test/java/org/apache/james/jmap/mailet/filter/JMAPFilteringTest.java: ## @@ -1129,4 +1130,56 @@ void orShouldNotMatchWhenN

[PR] Netty upgrade 10 09 2025 [james-project]

2025-09-17 Thread via GitHub
chibenwa opened a new pull request, #2802: URL: https://github.com/apache/james-project/pull/2802 ## Why ? https://www.cve.org/CVERecord?id=CVE-2025-58056 https://www.cve.org/CVERecord?id=CVE-2025-58057 -- This is an automated message from the Apache Git Service. To respond t

Re: [PR] JAMES-4135 OpenSearch: make query string usage lenient [james-project]

2025-09-17 Thread via GitHub
quantranhong1999 commented on PR #2809: URL: https://github.com/apache/james-project/pull/2809#issuecomment-3291001854 > 40 minutes seem to stem from only 4 testsuites. Hi @jeantil, I am curious here (sorry, I am not an expert in build caching). I see those "40 minutes" goal(s) are `C

Re: [PR] [BUILD] Improve test execution time - batch 1: consensual part [james-project]

2025-09-17 Thread via GitHub
jeantil commented on code in PR #2814: URL: https://github.com/apache/james-project/pull/2814#discussion_r2355778211 ## mailbox/opensearch/pom.xml: ## @@ -165,4 +165,16 @@ test + + + +org.apache.maven.plugins +

Re: [PR] JAMES-4144 RecipientHasMXRecord + SenderHasMXRecord [james-project]

2025-09-17 Thread via GitHub
chibenwa merged PR #2804: URL: https://github.com/apache/james-project/pull/2804 -- 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: notifications-unsubscr.

Re: [PR] [BUILD] Improve test execution time - batch 1: consensual part [james-project]

2025-09-17 Thread via GitHub
quantranhong1999 commented on code in PR #2814: URL: https://github.com/apache/james-project/pull/2814#discussion_r2356036450 ## server/protocols/jmap-rfc-8621-integration-tests/postgres-jmap-rfc-8621-integration-tests/pom.xml: ## @@ -101,11 +101,37 @@ org.apach

Re: [PR] [BUILD] Improve test execution time - batch 1: consensual part [james-project]

2025-09-17 Thread via GitHub
quantranhong1999 commented on PR #2814: URL: https://github.com/apache/james-project/pull/2814#issuecomment-3303696790 > out of curiosity why aim for forkCount instead of -T2 ? I remember in the past I tried tuning `threadCount` for Maven Surefire and using ` -T xxx` in the maven comm

Re: [PR] [BUILD] Improve test execution time - batch 1: consensual part [james-project]

2025-09-17 Thread via GitHub
quantranhong1999 commented on code in PR #2814: URL: https://github.com/apache/james-project/pull/2814#discussion_r2356031291 ## mailbox/opensearch/pom.xml: ## @@ -165,4 +165,16 @@ test + + + +org.apache.maven.

Re: [PR] [BUILD] Improve test execution time - batch 1: consensual part [james-project]

2025-09-17 Thread via GitHub
jeantil commented on code in PR #2814: URL: https://github.com/apache/james-project/pull/2814#discussion_r2355790241 ## server/protocols/jmap-rfc-8621-integration-tests/postgres-jmap-rfc-8621-integration-tests/pom.xml: ## @@ -101,11 +101,37 @@ org.apache.maven.p

Re: [PR] [BUILD] Improve test execution time [james-project]

2025-09-17 Thread via GitHub
jeantil commented on PR #2813: URL: https://github.com/apache/james-project/pull/2813#issuecomment-3303183129 Yes the build cache is a remote build cache so as long as the inputs have not changed it will apply regardless of the branch being built. If I did this right, it will even be use

Re: [PR] [BUILD] Improve test execution time [james-project]

2025-09-17 Thread via GitHub
jeantil commented on PR #2813: URL: https://github.com/apache/james-project/pull/2813#issuecomment-3302970838 can you clarify what you mean by > 2h master build is definitly hardware dependant, and comes at a real surprise to me. Thanks cloud bee and the foundation! are you sayin

Re: [PR] [ENHANCEMENT] Smtp technical accounts [james-project]

2025-09-17 Thread via GitHub
chibenwa merged PR #2810: URL: https://github.com/apache/james-project/pull/2810 -- 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: notifications-unsubscr.

[PR] [BUILD TIME] james-server-webadmin-mailbox: use lucene instead of Ope… [james-project]

2025-09-17 Thread via GitHub
chibenwa opened a new pull request, #2815: URL: https://github.com/apache/james-project/pull/2815 …nSearch -- 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

Re: [PR] [BUILD] Improve test execution time [james-project]

2025-09-17 Thread via GitHub
jeantil commented on PR #2813: URL: https://github.com/apache/james-project/pull/2813#issuecomment-3302045060 wow thanks, I intended to investigate when I next got some free time and didn't really expect anyone would look into it but that definitly saves me a lot of time. -- This is an

Re: [PR] [BUILD] Improve test execution time [james-project]

2025-09-17 Thread via GitHub
chibenwa commented on PR #2813: URL: https://github.com/apache/james-project/pull/2813#issuecomment-3302130127 Yes there is a topic with BlobStoreDAO contract too ^^ -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

Re: [PR] [BUILD] Improve test execution time [james-project]

2025-09-17 Thread via GitHub
quantranhong1999 commented on code in PR #2813: URL: https://github.com/apache/james-project/pull/2813#discussion_r2354912213 ## server/apps/distributed-app/src/test/java/org/apache/james/CassandraRabbitMQLegacyQuotaJamesServerTest.java: ## @@ -22,8 +22,11 @@ import org.apache.

Re: [PR] [BUILD] Improve test execution time [james-project]

2025-09-17 Thread via GitHub
quantranhong1999 commented on code in PR #2813: URL: https://github.com/apache/james-project/pull/2813#discussion_r2354909711 ## server/apps/distributed-app/pom.xml: ## @@ -501,6 +501,7 @@ -javaagent:"${settings.localRepository}"/org/jacoco/org.jacoco.a

Re: [PR] [BUILD] Improve test execution time [james-project]

2025-09-17 Thread via GitHub
quantranhong1999 commented on PR #2813: URL: https://github.com/apache/james-project/pull/2813#issuecomment-3302144373 > wow thanks, I intended to investigate when I next got some free time and didn't really expect anyone would look into it but that definitly saves me a lot of time.

[PR] [BUILD] Improve test execution time - batch 1: consensual part [james-project]

2025-09-17 Thread via GitHub
quantranhong1999 opened a new pull request, #2814: URL: https://github.com/apache/james-project/pull/2814 Consensual split part of https://github.com/apache/james-project/pull/2813 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitH

Re: [PR] [BUILD] Improve test execution time [james-project]

2025-09-17 Thread via GitHub
quantranhong1999 commented on code in PR #2813: URL: https://github.com/apache/james-project/pull/2813#discussion_r2354860560 ## server/protocols/webadmin/webadmin-mailbox/pom.xml: ## @@ -259,6 +259,7 @@ -javaagent:"${settings.localRepository}"/org/jaco

Re: [PR] [BUILD] Improve test execution time [james-project]

2025-09-17 Thread via GitHub
jeantil commented on PR #2813: URL: https://github.com/apache/james-project/pull/2813#issuecomment-3302075472 you might be interested in this develocity view too : https://develocity.apache.org/scans/tests?search.relativeStartTime=P90D&search.rootProjectNames=*james*&search.tasks=clean,test,

Re: [PR] [BUILD] Improve test execution time [james-project]

2025-09-17 Thread via GitHub
jeantil commented on code in PR #2813: URL: https://github.com/apache/james-project/pull/2813#discussion_r2354846211 ## server/apps/cassandra-app/pom.xml: ## @@ -326,6 +326,13 @@ + +org.apache.mave

Re: [PR] [BUILD] Improve test execution time [james-project]

2025-09-17 Thread via GitHub
chibenwa commented on code in PR #2813: URL: https://github.com/apache/james-project/pull/2813#discussion_r2354793168 ## server/protocols/webadmin/webadmin-mailbox/pom.xml: ## @@ -259,6 +259,7 @@ -javaagent:"${settings.localRepository}"/org/jacoco/org.j

Re: [PR] [BUILD] Improve test execution time [james-project]

2025-09-17 Thread via GitHub
chibenwa commented on code in PR #2813: URL: https://github.com/apache/james-project/pull/2813#discussion_r2354784122 ## server/apps/cassandra-app/pom.xml: ## @@ -326,6 +326,13 @@ + +org.apache.mav

Re: [PR] [BUILD] Improve test execution time [james-project]

2025-09-17 Thread via GitHub
quantranhong1999 commented on code in PR #2813: URL: https://github.com/apache/james-project/pull/2813#discussion_r2354771975 ## server/apps/distributed-app/pom.xml: ## @@ -501,6 +501,7 @@ -javaagent:"${settings.localRepository}"/org/jacoco/org.jacoco.a

Re: [PR] [BUILD] Improve test execution time [james-project]

2025-09-17 Thread via GitHub
chibenwa commented on code in PR #2813: URL: https://github.com/apache/james-project/pull/2813#discussion_r2354773661 ## server/apps/distributed-app/src/test/java/org/apache/james/WithScanningSearchMutableTest.java: ## @@ -19,8 +19,11 @@ package org.apache.james; +import or

[PR] [BUILD Improve test execution time [james-project]

2025-09-17 Thread via GitHub
quantranhong1999 opened a new pull request, #2813: URL: https://github.com/apache/james-project/pull/2813 Effort to improve the test execution time of 5m+ modules: https://github.com/user-attachments/assets/e3bf77e2-51f0-406d-802a-aa758ed1a9c3"; /> cf https://github.com/apache/jame

Re: [PR] JAMES-4057 Align subject search with other implementations [james-project]

2025-09-16 Thread via GitHub
Arsnael commented on code in PR #2812: URL: https://github.com/apache/james-project/pull/2812#discussion_r2354123331 ## mailbox/opensearch/src/main/java/org/apache/james/mailbox/opensearch/query/DefaultCriterionConverter.java: ## @@ -331,21 +330,33 @@ protected Query convertTex

Re: [PR] JAMES-4057 Align subject search with other implementations [james-project]

2025-09-16 Thread via GitHub
quantranhong1999 commented on code in PR #2812: URL: https://github.com/apache/james-project/pull/2812#discussion_r2354104065 ## mailbox/opensearch/src/main/java/org/apache/james/mailbox/opensearch/query/DefaultCriterionConverter.java: ## @@ -331,21 +330,33 @@ protected Query c

Re: [PR] JAMES-4143 Add a custom header Field for JMAP rule matching [james-project]

2025-09-16 Thread via GitHub
Arsnael merged PR #2811: URL: https://github.com/apache/james-project/pull/2811 -- 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: notifications-unsubscr..

Re: [PR] JAMES-4143 Add a custom header Field for JMAP rule matching [james-project]

2025-09-16 Thread via GitHub
chibenwa commented on code in PR #2811: URL: https://github.com/apache/james-project/pull/2811#discussion_r2351549003 ## server/data/data-jmap/src/main/java/org/apache/james/jmap/api/filtering/Rule.java: ## @@ -129,33 +129,52 @@ public String toString() { public static cl

Re: [PR] JAMES-3758 Expiration: specific user and possibility to use saved date [james-project]

2025-09-15 Thread via GitHub
Arsnael merged PR #2806: URL: https://github.com/apache/james-project/pull/2806 -- 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: notifications-unsubscr..

Re: [PR] [ENHANCEMENT] Smtp technical accounts [james-project]

2025-09-15 Thread via GitHub
Arsnael commented on code in PR #2810: URL: https://github.com/apache/james-project/pull/2810#discussion_r2350506424 ## server/protocols/protocols-smtp/src/test/java/org/apache/james/smtpserver/ConfiguredAuthTest.java: ## @@ -0,0 +1,82 @@ +/**

Re: [PR] JAMES-4143 Add a custom header Field for JMAP rule matching [james-project]

2025-09-15 Thread via GitHub
Arsnael commented on code in PR #2811: URL: https://github.com/apache/james-project/pull/2811#discussion_r2350500895 ## server/data/data-jmap/src/main/java/org/apache/james/jmap/api/filtering/Rule.java: ## @@ -129,33 +129,58 @@ public String toString() { public static cla

[PR] JAMES-4143 Add a custom header Field for JMAP rule matching [james-project]

2025-09-15 Thread via GitHub
chibenwa opened a new pull request, #2811: URL: https://github.com/apache/james-project/pull/2811 (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 unsubscrib

Re: [PR] [ENHANCEMENT] Smtp technical accounts [james-project]

2025-09-15 Thread via GitHub
chibenwa commented on code in PR #2810: URL: https://github.com/apache/james-project/pull/2810#discussion_r2350013468 ## server/protocols/protocols-smtp/src/main/java/org/apache/james/smtpserver/NoAuthCmdHandlerLoader.java: ## @@ -56,7 +56,6 @@ public class NoAuthCmdHandlerLoade

Re: [PR] [ENHANCEMENT] Smtp technical accounts [james-project]

2025-09-15 Thread via GitHub
chibenwa commented on code in PR #2810: URL: https://github.com/apache/james-project/pull/2810#discussion_r2350015156 ## docs/modules/servers/partials/configure/smtp-hooks.adoc: ## @@ -423,4 +423,77 @@ Example handlerchain configuration for `smtpserver.xml`: Would allow emails

Re: [PR] [ENHANCEMENT] Smtp technical accounts [james-project]

2025-09-15 Thread via GitHub
jeantil commented on code in PR #2810: URL: https://github.com/apache/james-project/pull/2810#discussion_r2349528320 ## server/protocols/protocols-smtp/src/main/java/org/apache/james/smtpserver/NoAuthCmdHandlerLoader.java: ## @@ -56,7 +56,6 @@ public class NoAuthCmdHandlerLoader

Re: [PR] [ENHANCEMENT] Smtp technical accounts [james-project]

2025-09-15 Thread via GitHub
jeantil commented on PR #2810: URL: https://github.com/apache/james-project/pull/2810#issuecomment-3292998839 Nice ! I find these quite interesting :D -- 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

Re: [PR] [ENHANCEMENT] Smtp technical accounts [james-project]

2025-09-15 Thread via GitHub
jeantil commented on code in PR #2810: URL: https://github.com/apache/james-project/pull/2810#discussion_r2349534318 ## docs/modules/servers/partials/configure/smtp-hooks.adoc: ## @@ -423,4 +423,77 @@ Example handlerchain configuration for `smtpserver.xml`: Would allow emails u

Re: [PR] [ENHACEMENT] Include mailbox storage ids in the audit trail upon rece… [james-project]

2025-09-15 Thread via GitHub
chibenwa merged PR #2796: URL: https://github.com/apache/james-project/pull/2796 -- 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: notifications-unsubscr.

Re: [PR] JAMES-4135 OpenSearch: make query string usage lenient [james-project]

2025-09-15 Thread via GitHub
chibenwa merged PR #2809: URL: https://github.com/apache/james-project/pull/2809 -- 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: notifications-unsubscr.

Re: [PR] JAMES-4135 OpenSearch: make query string usage lenient [james-project]

2025-09-15 Thread via GitHub
chibenwa commented on PR #2809: URL: https://github.com/apache/james-project/pull/2809#issuecomment-3290933294 > 40 minutes seem to stem from only 4 testsuites. Considering the length of execution of these test suites,I wonder if we could reclaim build time by dropping some integration test

Re: [PR] Netty upgrade 10 09 2025 [james-project]

2025-09-15 Thread via GitHub
chibenwa merged PR #2802: URL: https://github.com/apache/james-project/pull/2802 -- 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: notifications-unsubscr.

Re: [PR] Netty upgrade 10 09 2025 [james-project]

2025-09-15 Thread via GitHub
chibenwa commented on PR #2802: URL: https://github.com/apache/james-project/pull/2802#issuecomment-3290925170 > Looks green... still a draft? Nope I wanted a green build for considering this viable -- This is an automated message from the Apache Git Service. To respond to the messa

Re: [PR] JAMES-3758 Expiration: specific user and possibility to use saved date [james-project]

2025-09-14 Thread via GitHub
quantranhong1999 commented on PR #2806: URL: https://github.com/apache/james-project/pull/2806#issuecomment-3290606304 > I restarted a build to see if it was a random failure from the builder and the build job succeeded, so I would say it was just a fluke. Thank you. The build looks g

Re: [PR] JAMES-3758 Expiration: specific user and possibility to use saved date [james-project]

2025-09-14 Thread via GitHub
jeantil commented on PR #2806: URL: https://github.com/apache/james-project/pull/2806#issuecomment-3290552410 @quantranhong1999 I don't think it's related to build cache : - the build cache has been in place for a while now, the recent changes added cache to scala compilation and test ex

Re: [PR] JAMES-4135 OpenSearch: make query string usage lenient [james-project]

2025-09-14 Thread via GitHub
jeantil commented on PR #2809: URL: https://github.com/apache/james-project/pull/2809#issuecomment-3290521909 Slightly offtopic food for thoughts: The build took ~1h30 instead of 4h thanks to build caching which I think is a good thing. The main contributors to the remaining

Re: [PR] DNSJavaService: small code improvments [james-project]

2025-09-14 Thread via GitHub
Arsnael merged PR #2803: URL: https://github.com/apache/james-project/pull/2803 -- 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: notifications-unsubscr..

Re: [PR] JAMES-3758 Expiration: specific user and possibility to use saved date [james-project]

2025-09-14 Thread via GitHub
quantranhong1999 commented on PR #2806: URL: https://github.com/apache/james-project/pull/2806#issuecomment-3290396115 ```java 03:51:20,358 [INFO] 7234 goals, 6476 executed, 758 from cache, saving at least 47s --   |   |   | 03:51:20,932 [INFO]   |   |   | 03:51:20,932 [INFO]

Re: [PR] [JAMES-2124] automatically sort poms when not on CI [james-project]

2025-09-14 Thread via GitHub
Arsnael merged PR #2807: URL: https://github.com/apache/james-project/pull/2807 -- 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: notifications-unsubscr..

Re: [PR] JAMES-4144 RecipientHasMXRecord + SenderHasMXRecord [james-project]

2025-09-14 Thread via GitHub
Arsnael commented on code in PR #2804: URL: https://github.com/apache/james-project/pull/2804#discussion_r2347791676 ## server/mailet/mailets/src/test/java/org/apache/james/transport/matchers/RecipientHasMXRecordTest.java: ## @@ -0,0 +1,189 @@ +/*

Re: [PR] Netty upgrade 10 09 2025 [james-project]

2025-09-14 Thread via GitHub
Arsnael commented on PR #2802: URL: https://github.com/apache/james-project/pull/2802#issuecomment-3290318715 Looks green... still a draft? -- 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 spe

Re: [PR] JAMES-4146: Add support for proxy protocol in managesieve [james-project]

2025-09-14 Thread via GitHub
Arsnael merged PR #2808: URL: https://github.com/apache/james-project/pull/2808 -- 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: notifications-unsubscr..

Re: [PR] feat(managesieve): add XOAUTH2 authentication mechanism [james-project]

2025-09-14 Thread via GitHub
chibenwa commented on PR #2773: URL: https://github.com/apache/james-project/pull/2773#issuecomment-3289869882 https://github.com/user-attachments/assets/f6bb7407-661c-4d9b-9ce7-25a1dcfdb318"; /> Failing tests seems to be related. Can we investigate this? -- This is an automate

Re: [PR] [JAMES-2124] automatically sort poms when not on CI [james-project]

2025-09-14 Thread via GitHub
chibenwa commented on PR #2807: URL: https://github.com/apache/james-project/pull/2807#issuecomment-3289868139 > git config blame.ignoreRevsFile .git-blame-ignore-revs Discovering this: it's awesome! -- This is an automated message from the Apache Git Service. To respond to the mess

Re: [PR] [JAMES-3978] Improves james build cache [james-project]

2025-09-14 Thread via GitHub
jeantil merged PR #2786: URL: https://github.com/apache/james-project/pull/2786 -- 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: notifications-unsubscr..

[PR] feat(managesieve): add support for proxy protocol in managesieve [james-project]

2025-09-14 Thread via GitHub
felixauringer opened a new pull request, #2808: URL: https://github.com/apache/james-project/pull/2808 I found that it is possible to configure proxy protocol for managesieve but the configuration is ignored. This fixes that. I did not find any logic in managesieve that tries to act on t

[PR] [JAMES-2124] automatically sort poms when not on CI [james-project]

2025-09-14 Thread via GitHub
jeantil opened a new pull request, #2807: URL: https://github.com/apache/james-project/pull/2807 You can ignore this commit in git blame by running ``` git config blame.ignoreRevsFile .git-blame-ignore-revs ``` The `.git-blame-ignore-revs` file will be created once this commit is

Re: [PR] Improves james build cache [james-project]

2025-09-14 Thread via GitHub
jeantil commented on PR #2786: URL: https://github.com/apache/james-project/pull/2786#issuecomment-3289597196 [This build](https://ci-builds.apache.org/job/james/job/ApacheJames/job/PR-2786/13/cloudbees-pipeline-explorer/) shows test output successfully loaded from the build cache. [The

Re: [PR] Improves james build cache [james-project]

2025-09-14 Thread via GitHub
jeantil commented on code in PR #2786: URL: https://github.com/apache/james-project/pull/2786#discussion_r2347358578 ## pom.xml: ## @@ -3179,7 +3179,336 @@ +

[PR] JAMES-3758 Expiration: specific user and possibility to use saved date [james-project]

2025-09-13 Thread via GitHub
chibenwa opened a new pull request, #2806: URL: https://github.com/apache/james-project/pull/2806 Refactor the class to choose between the different message listing strategies. We did also review the reactive chain to make it more readable. -- This is an automated message from the Apache

Re: [PR] [JAMES-4145] fixes unnecessary dependencies [james-project]

2025-09-13 Thread via GitHub
jeantil commented on PR #2805: URL: https://github.com/apache/james-project/pull/2805#issuecomment-3288796107 I think my next PR will simply enable the automatic sorting of the pom files by default, I'll run it on all the project's pom and add the commit to the git blame ignore list and be

Re: [PR] [JAMES-4145] fixes unnecessary dependencies [james-project]

2025-09-13 Thread via GitHub
jeantil commented on code in PR #2805: URL: https://github.com/apache/james-project/pull/2805#discussion_r2346928380 ## server/apps/postgres-app/src/main/java/org/apache/james/modules/data/PostgresDLPConfigurationStoreModule.java: ## @@ -31,6 +31,7 @@ import com.google.inject.T

Re: [PR] [JAMES-4145] fixes unnecessary dependencies [james-project]

2025-09-13 Thread via GitHub
chibenwa commented on PR #2805: URL: https://github.com/apache/james-project/pull/2805#issuecomment-3288755606 ``` org.apache.maven.lifecycle.LifecycleExecutionException: Failed to execute goal com.github.ekryd.sortpom:sortpom-maven-plugin:4.0.0:verify (strict-pom-ordering) on project ja

Re: [PR] [JAMES-4145] fixes unnecessary dependencies [james-project]

2025-09-13 Thread via GitHub
chibenwa commented on code in PR #2805: URL: https://github.com/apache/james-project/pull/2805#discussion_r2346905411 ## server/apps/postgres-app/src/main/java/org/apache/james/modules/data/PostgresDLPConfigurationStoreModule.java: ## @@ -31,6 +31,7 @@ import com.google.inject.

[PR] [JAMES-4145] fixes unnecessary dependencies [james-project]

2025-09-13 Thread via GitHub
jeantil opened a new pull request, #2805: URL: https://github.com/apache/james-project/pull/2805 Removes - james-server-postgres-common-guice -> james-server-guice-distributed (which brings cassandra, opensearch and rabbitmq to a module that's suppose to provide pg bindings) - james-

Re: [PR] Improves james build cache [james-project]

2025-09-13 Thread via GitHub
jeantil commented on code in PR #2786: URL: https://github.com/apache/james-project/pull/2786#discussion_r2345938431 ## pom.xml: ## @@ -3196,6 +3196,10 @@ true

Re: [PR] Improves james build cache [james-project]

2025-09-13 Thread via GitHub
jeantil commented on PR #2786: URL: https://github.com/apache/james-project/pull/2786#issuecomment-3287708130 with build cache enabled for scala compilation, a rebuild on an unchanged james codebase unsing `mvn clean test-compile` goes from ``` [INFO]

[PR] JAMES-4144 RecipientHasMXRecord + SenderHasMXRecord [james-project]

2025-09-12 Thread via GitHub
chibenwa opened a new pull request, #2804: URL: https://github.com/apache/james-project/pull/2804 (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 unsubscrib

  1   2   3   4   5   6   7   8   9   10   >