(james-project) branch master updated (008923d001 -> bbe97c5c6f)

2024-03-05 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 008923d001 [ENHANCEMENT] JsoupHtmlTextExtractor: The use of concurrent structures is not needed add

Re: [PR] [FIX] IMAP execution management [james-project]

2024-03-05 Thread via GitHub
Arsnael merged PR #2090: URL: https://github.com/apache/james-project/pull/2090 -- 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] JAMES-2586 Fix some jmap postgres integration tests [james-project]

2024-03-05 Thread via GitHub
vttranlina commented on PR #2048: URL: https://github.com/apache/james-project/pull/2048#issuecomment-1980085869 > Related to the ci failed bodyStructureShouldSupportSpecificHeaders(GuiceJamesServer) we can cherry-pick fixup commit:

[PR] JAMES-3419 JMAP EmailBodyPart individual headers - Fix test cases [james-project]

2024-03-05 Thread via GitHub
vttranlina opened a new pull request, #2093: URL: https://github.com/apache/james-project/pull/2093 Fix test cases `bodyStructureShouldSupportSpecificHeaders` ![image](https://github.com/apache/james-project/assets/81145350/0625587b-c6f5-4fab-b44e-93a3fcba191e) it fails when run

Re: [PR] JAMES-2586 Fix some jmap postgres integration tests [james-project]

2024-03-05 Thread via GitHub
vttranlina commented on code in PR #2048: URL: https://github.com/apache/james-project/pull/2048#discussion_r1513805508 ## mailbox/postgres/src/main/java/org/apache/james/mailbox/postgres/mail/PostgresMessageIdMapper.java: ## @@ -130,18 +142,44 @@ public Publisher

Re: [PR] JAMES-2586 Fix some jmap postgres integration tests [james-project]

2024-03-05 Thread via GitHub
hungphan227 commented on code in PR #2048: URL: https://github.com/apache/james-project/pull/2048#discussion_r1513797407 ## mailbox/postgres/src/main/java/org/apache/james/mailbox/postgres/mail/PostgresMessageIdMapper.java: ## @@ -130,18 +142,44 @@ public Publisher

Re: [PR] JAMES-2586 Fix some jmap postgres integration tests [james-project]

2024-03-05 Thread via GitHub
vttranlina commented on PR #2048: URL: https://github.com/apache/james-project/pull/2048#issuecomment-1980061620 Related to the ci failed ``` bodyStructureShouldSupportSpecificHeaders(GuiceJamesServer) – org.apache.james.jmap.rfc8621.postgres.PostgresEmailGetMethodTest JSON

Re: [PR] JAMES-2586 Fix some jmap postgres integration tests [james-project]

2024-03-05 Thread via GitHub
vttranlina commented on code in PR #2048: URL: https://github.com/apache/james-project/pull/2048#discussion_r1513787434 ## mailbox/postgres/src/main/java/org/apache/james/mailbox/postgres/mail/PostgresMessageIdMapper.java: ## @@ -130,18 +142,44 @@ public Publisher

Re: [PR] DRAFT JAMES-2586 Index for RLS column [james-project]

2024-03-05 Thread via GitHub
Arsnael closed pull request #2082: DRAFT JAMES-2586 Index for RLS column URL: https://github.com/apache/james-project/pull/2082 -- 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.

Re: [PR] DRAFT JAMES-2586 Index for RLS column [james-project]

2024-03-05 Thread via GitHub
Arsnael commented on PR #2082: URL: https://github.com/apache/james-project/pull/2082#issuecomment-1980028284 After discussions in https://github.com/linagora/james-project/issues/5054, it's been proven that this does not really give much benefits. Closing it -- This is an

Re: [PR] JAMES-2586 Fix some jmap postgres integration tests [james-project]

2024-03-05 Thread via GitHub
vttranlina commented on code in PR #2048: URL: https://github.com/apache/james-project/pull/2048#discussion_r1513757850 ## mailbox/postgres/src/main/java/org/apache/james/mailbox/postgres/mail/dao/PostgresAttachmentDAO.java: ## @@ -80,9 +83,20 @@ public Mono>

Re: [PR] JAMES-2586 Fix some jmap postgres integration tests [james-project]

2024-03-05 Thread via GitHub
vttranlina commented on code in PR #2048: URL: https://github.com/apache/james-project/pull/2048#discussion_r1513738873 ## mailbox/postgres/src/main/java/org/apache/james/mailbox/postgres/mail/PostgresMessageIdMapper.java: ## @@ -130,18 +142,44 @@ public Publisher

Re: [PR] JAMES-2586 Fix some jmap postgres integration tests [james-project]

2024-03-05 Thread via GitHub
hungphan227 commented on code in PR #2048: URL: https://github.com/apache/james-project/pull/2048#discussion_r1513733102 ## mailbox/postgres/src/main/java/org/apache/james/mailbox/postgres/mail/PostgresMessageIdMapper.java: ## @@ -130,18 +142,44 @@ public Publisher

(james-project) 05/05: [ENHANCEMENT] JsoupHtmlTextExtractor: The use of concurrent structures is not needed

2024-03-05 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 008923d001157789aaadea7b8fae99cac67adec5 Author: Benoit TELLIER AuthorDate: Tue Mar 5 09:27:34 2024 +0100

(james-project) 02/05: [ENHANCEMENT] AESBlobStoreDAO: carry over the size

2024-03-05 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 c0ecbcbacac4563b9086a5c00b38d718e80504a4 Author: Benoit TELLIER AuthorDate: Tue Mar 5 09:23:19 2024 +0100

(james-project) 03/05: [ENHANCEMENT] Computing preview: avoid decoding base64

2024-03-05 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 9785c573fb4b5be24f51019d9e5aa7c11a6de5f7 Author: Benoit TELLIER AuthorDate: Tue Mar 5 09:24:04 2024 +0100

(james-project) 01/05: [ENHANCEMENT] Fix warning in AvoidBinaryBodyBufferingBodyFactory

2024-03-05 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 23c4ef489c058caa7599b4f33f189725d8590358 Author: Benoit TELLIER AuthorDate: Tue Mar 5 07:40:16 2024 +0100

(james-project) branch master updated (aba90e1b8b -> 008923d001)

2024-03-05 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 aba90e1b8b Update JamesServerExtension new 23c4ef489c [ENHANCEMENT] Fix warning in

(james-project) 04/05: [ENHANCEMENT] Message content extractor: pre-size strings

2024-03-05 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 3c86f6dc0d34984c962fe3f0386f1ef6526e0425 Author: Benoit TELLIER AuthorDate: Tue Mar 5 09:24:33 2024 +0100

Re: [PR] [ENHANCEMENT] Collection of minor performance enhancements [james-project]

2024-03-05 Thread via GitHub
Arsnael merged PR #2088: URL: https://github.com/apache/james-project/pull/2088 -- 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] JAMES-2586 Fix some jmap postgres integration tests [james-project]

2024-03-05 Thread via GitHub
hungphan227 commented on code in PR #2048: URL: https://github.com/apache/james-project/pull/2048#discussion_r1513733102 ## mailbox/postgres/src/main/java/org/apache/james/mailbox/postgres/mail/PostgresMessageIdMapper.java: ## @@ -130,18 +142,44 @@ public Publisher

(james-project) branch master updated: Update JamesServerExtension

2024-03-05 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 aba90e1b8b Update JamesServerExtension

Re: [PR] JAMES-2586 Fix PostgresAuthenticationTest [james-project]

2024-03-05 Thread via GitHub
Arsnael merged PR #2072: URL: https://github.com/apache/james-project/pull/2072 -- 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 postgresql updated: JAMES-2586 Fix PostgresAuthenticationTest

2024-03-05 Thread rcordier
This is an automated email from the ASF dual-hosted git repository. rcordier pushed a commit to branch postgresql in repository https://gitbox.apache.org/repos/asf/james-project.git The following commit(s) were added to refs/heads/postgresql by this push: new 159806c86a JAMES-2586 Fix

Re: [PR] Update JamesServerExtension [james-project]

2024-03-05 Thread via GitHub
Arsnael merged PR #2086: URL: https://github.com/apache/james-project/pull/2086 -- 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:

[PR] [NO_REVIEW] Avoid sorting PG messages [james-project]

2024-03-05 Thread via GitHub
vttranlina opened a new pull request, #2092: URL: https://github.com/apache/james-project/pull/2092 The clone pr of https://github.com/apache/james-project/pull/1964 for see the ci test already: - rebased master - squash fixup -- This is an automated message from the

(james-project) branch master updated: Upgrade git-commit-id-maven-plugin to 8.0.0

2024-03-05 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 1bf699dc92 Upgrade

Re: [PR] Upgrade git-commit-id-maven-plugin to 8.0.0 [james-project]

2024-03-05 Thread via GitHub
Arsnael merged PR #2085: URL: https://github.com/apache/james-project/pull/2085 -- 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] Upgrade git-commit-id-maven-plugin to 8.0.0 [james-project]

2024-03-05 Thread via GitHub
jeantil commented on PR #2085: URL: https://github.com/apache/james-project/pull/2085#issuecomment-1979654577 nice and we even got https://github.com/git-commit-id/git-commit-id-maven-plugin/pull/700: Only injects properties into every reactor project once as a small bonus :D

Re: [PR] MIME4J-326 Adopt Mime4J 0.8.11-SNAPSHOT [james-project]

2024-03-05 Thread via GitHub
chibenwa commented on PR #2073: URL: https://github.com/apache/james-project/pull/2073#issuecomment-1979067464 I just called a vote on this. -- 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] MIME4J release 0.8.11 [james-project]

2024-03-05 Thread via GitHub
chibenwa opened a new pull request, #2091: URL: https://github.com/apache/james-project/pull/2091 (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] Avoid sorting PG messages [james-project]

2024-03-05 Thread via GitHub
chibenwa commented on PR #1964: URL: https://github.com/apache/james-project/pull/1964#issuecomment-1978836912 Please team care about this as this is a potential production-killer. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[PR] [FIX] IMAP execution management [james-project]

2024-03-05 Thread via GitHub
chibenwa opened a new pull request, #2090: URL: https://github.com/apache/james-project/pull/2090 That is my best shot at explaining: ``` java.lang.NullPointerException: Cannot invoke "org.apache.james.imap.api.process.SelectedMailbox.existsCount()" because "selected" is null

(james-project) branch master updated: [FIX] Modified server set identities may not be deletes (#2083)

2024-03-05 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 1dcfac3332 [FIX] Modified server set

Re: [PR] [FIX] Modified server set identities may not be deletes [james-project]

2024-03-05 Thread via GitHub
chibenwa merged PR #2083: URL: https://github.com/apache/james-project/pull/2083 -- 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] JAMES-2586 Fix some jmap postgres integration tests [james-project]

2024-03-05 Thread via GitHub
vttranlina commented on code in PR #2048: URL: https://github.com/apache/james-project/pull/2048#discussion_r1512598925 ## mailbox/postgres/src/main/java/org/apache/james/mailbox/postgres/mail/dao/PostgresAttachmentDAO.java: ## @@ -80,9 +83,20 @@ public Mono>

Re: [PR] [FIX] Modified server set identities may not be deletes [james-project]

2024-03-05 Thread via GitHub
vttranlina commented on PR #2083: URL: https://github.com/apache/james-project/pull/2083#issuecomment-197841 > So we merge this along with #2087 is that ok to you @vttranlina @quantranhong1999 ? +1 -- This is an automated message from the Apache Git Service. To respond to the

Re: [PR] [UPDATE] commons-compress 1.24.0 -> 1.26.0 [james-project]

2024-03-05 Thread via GitHub
chibenwa closed pull request #2055: [UPDATE] commons-compress 1.24.0 -> 1.26.0 URL: https://github.com/apache/james-project/pull/2055 -- 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

[PR] [UPDATE] commons-compress 1.24.0 -> 1.26.0 [james-project]

2024-03-05 Thread via GitHub
chibenwa opened a new pull request, #2089: URL: https://github.com/apache/james-project/pull/2089 Fixes: - CVE-2024-25710: Denial of service caused by an infinite loop for a corrupted DUMP file - CVE-2024-26308: OutOfMemoryError unpacking broken Pack200 file -- This is an

Re: [PR] [FIX] Modified server set identities may not be deletes [james-project]

2024-03-05 Thread via GitHub
quantranhong1999 commented on PR #2083: URL: https://github.com/apache/james-project/pull/2083#issuecomment-1978360015 > So we merge this along with https://github.com/apache/james-project/pull/2087 is that ok to you @vttranlina @quantranhong1999 ? +1 -- This is an automated

Re: [PR] [FIX] Modified server set identities may not be deletes [james-project]

2024-03-05 Thread via GitHub
chibenwa commented on PR #2083: URL: https://github.com/apache/james-project/pull/2083#issuecomment-1978356845 So we merge this along with #2087 is that ok to you @vttranlina @quantranhong1999 ? -- This is an automated message from the Apache Git Service. To respond to the message,

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

2024-03-05 Thread via GitHub
chibenwa closed pull request #2063: [BUILD] Use git-commit-id-maven-plugin in ci-test URL: 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 above to go to the

Re: [PR] Update JamesServerExtension [james-project]

2024-03-05 Thread via GitHub
hungphan227 commented on PR #2086: URL: https://github.com/apache/james-project/pull/2086#issuecomment-1978354019 > Can you please better explain why this change is needed? The old code of JamesServerExtension: ``` PER_ENCLOSING_CLASS( (extension, context) ->

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

2024-03-05 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 d716ff688 Site checkin for

Re: [PR] [FIX] Calendar parsing: be less verbose [james-project]

2024-03-05 Thread via GitHub
Arsnael merged PR #2084: URL: https://github.com/apache/james-project/pull/2084 -- 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: [FIX] Calendar parsing: be less verbose

2024-03-05 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 d13aab4696 [FIX] Calendar parsing: be

Re: [PR] [FIX] Modified server set identities may not be deletes [james-project]

2024-03-05 Thread via GitHub
vttranlina commented on code in PR #2083: URL: https://github.com/apache/james-project/pull/2083#discussion_r1512377259 ##

[PR] [ENHANCEMENT] Collection of minor performance enhancements [james-project]

2024-03-05 Thread via GitHub
chibenwa opened a new pull request, #2088: URL: https://github.com/apache/james-project/pull/2088 (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-3534 Fixbug - Identity response alway return mayDelete=false when server set identity [james-project]

2024-03-05 Thread via GitHub
vttranlina commented on PR #2087: URL: https://github.com/apache/james-project/pull/2087#issuecomment-1978213045 > Duplicated #2083 or I miss something? as my commented on 2083 pr 2083 fix logic "after" storage database, this pr: > will fix logic "before" storage database