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

2024-02-25 Thread via GitHub
chibenwa commented on PR #2049: URL: https://github.com/apache/james-project/pull/2049#issuecomment-1963499656 > I wasn't aware that the James project build used git submodules. Not in james but the use linagora uses James. TLDR: it allows repackaging james, cross repository

(james-project) branch compress-cves-2 updated (26408466b3 -> 25de96ea4f)

2024-02-25 Thread btellier
This is an automated email from the ASF dual-hosted git repository. btellier pushed a change to branch compress-cves-2 in repository https://gitbox.apache.org/repos/asf/james-project.git omit 26408466b3 [UPDATE] commons-compress 1.24.0 -> 1.26.0 add 8a9a9d38de [SITE] Blog post for

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

2024-02-25 Thread via GitHub
jeantil commented on PR #2049: URL: https://github.com/apache/james-project/pull/2049#issuecomment-1963493574 Hi, I'm on limited connectivity this week. I wasn't aware that the James project build used git submodules. There are several limitations in the plugin with regard to

Re: [PR] JAMES-4007 Improve IMAP literal lifecycle [james-project]

2024-02-25 Thread via GitHub
chibenwa merged PR #2035: URL: https://github.com/apache/james-project/pull/2035 -- 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: JAMES-4007 ImapChannelUpstreamHandler should always cleanup INBOUND message

2024-02-25 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 commit ba34e7242c95fa215b871a2af133e75ee14952ad Author: Benoit TELLIER AuthorDate: Wed Feb 21 14:21:22 2024 +0100

(james-project) 02/02: JAMES-4007 Prevent channel inactivity to clear literal while IMAP APPEND is in progress

2024-02-25 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 commit 8b9cbf138555b9c3c5027f9a0872323da511486a Author: Benoit TELLIER AuthorDate: Wed Feb 21 14:28:29 2024 +0100

(james-project) branch master updated (a2583d4e8a -> 8b9cbf1385)

2024-02-25 Thread btellier
This is an automated email from the ASF dual-hosted git repository. btellier pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/james-project.git from a2583d4e8a JAMES-4008 JMAP - Email/set - Should be able to save a draft with invalid email address (#2040)

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

2024-02-25 Thread via GitHub
chibenwa commented on PR #2049: URL: https://github.com/apache/james-project/pull/2049#issuecomment-1963490182 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

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

2024-02-25 Thread via GitHub
chibenwa opened a new pull request, #2050: URL: https://github.com/apache/james-project/pull/2050 (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

(james-project) branch master updated: JAMES-4008 JMAP - Email/set - Should be able to save a draft with invalid email address (#2040)

2024-02-25 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 a2583d4e8a JAMES-4008 JMAP - Email/set

Re: [PR] JAMES-4008 JMAP - Email/set - Should be able to save a draft with invalid email address [james-project]

2024-02-25 Thread via GitHub
chibenwa merged PR #2040: URL: https://github.com/apache/james-project/pull/2040 -- 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] Revert git commit id maven plugin [james-project]

2024-02-25 Thread via GitHub
chibenwa commented on PR #2049: URL: https://github.com/apache/james-project/pull/2049#issuecomment-1963472928 To be honnest I'd be more in favour of removing that plugin from global build and configuring it only in `server/apps` modules. -- This is an automated message from the Apache

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

2024-02-25 Thread via GitHub
Arsnael commented on PR #2029: URL: https://github.com/apache/james-project/pull/2029#issuecomment-1963463897 No sorry didnt realize that the faulty tests in question were not even part of this PR. Thinking more of a bad test isolation here..? -- This is an automated message from the

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

2024-02-25 Thread via GitHub
Arsnael commented on PR #2049: URL: https://github.com/apache/james-project/pull/2049#issuecomment-1963452919 > Have we tried disactivating it in tmail build? I tried but wasn't successful. You are free to give it a shot though :) -- This is an automated message from the Apache Git

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

2024-02-25 Thread via GitHub
chibenwa commented on PR #2022: URL: https://github.com/apache/james-project/pull/2022#issuecomment-1963409069 I suspect the recent build change to not retrigger the test and report it as failed... -- This is an automated message from the Apache Git Service. To respond to the message,

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

2024-02-25 Thread via GitHub
chibenwa commented on PR #2049: URL: https://github.com/apache/james-project/pull/2049#issuecomment-1963406487 Have we tried disactivating it in tmail build? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

(james-project) branch master updated: JAMES-4002 AES blob store tuning to reduce memory pressure (#2021)

2024-02-25 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 66426a8005 JAMES-4002 AES blob store

Re: [PR] JAMES-4002 AES blob store tuning to reduce memory pressure [james-project]

2024-02-25 Thread via GitHub
chibenwa merged PR #2021: URL: https://github.com/apache/james-project/pull/2021 -- 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-02-25 Thread via GitHub
vttranlina commented on code in PR #2048: URL: https://github.com/apache/james-project/pull/2048#discussion_r1502032559 ## mailbox/postgres/src/main/java/org/apache/james/mailbox/postgres/mail/dao/PostgresAttachmentDAO.java: ## @@ -80,9 +84,21 @@ public Mono>

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

2024-02-25 Thread via GitHub
Arsnael opened a new pull request, #2049: URL: https://github.com/apache/james-project/pull/2049 @jeantil I'm sorry to ask for this, but do you mind if we revert the upgrade of the git-commit-id-maven-plugin for now? It seems tocreate issues with projects using James as a submodule

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

2024-02-25 Thread via GitHub
Arsnael commented on PR #2022: URL: https://github.com/apache/james-project/pull/2022#issuecomment-1963284254 This test failed on the last two builds:

(james-project) branch master updated: JAMES-3995 Optimize Email/get

2024-02-25 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 aee6550277 JAMES-3995 Optimize

Re: [PR] JAMES-3995 Optimize Email/get [james-project]

2024-02-25 Thread via GitHub
Arsnael merged PR #2005: URL: https://github.com/apache/james-project/pull/2005 -- 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 Integration tests for JMAP postgres [james-project]

2024-02-25 Thread via GitHub
Arsnael commented on PR #2029: URL: https://github.com/apache/james-project/pull/2029#issuecomment-1963275402 Potentially racing issue? -- 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] JAMES-2586 Fix some jmap postgres integration tests [james-project]

2024-02-25 Thread via GitHub
hungphan227 commented on code in PR #2048: URL: https://github.com/apache/james-project/pull/2048#discussion_r1502017082 ## mailbox/postgres/src/main/java/org/apache/james/mailbox/postgres/mail/PostgresAttachmentModule.java: ## @@ -35,20 +35,22 @@ public interface

(james-project) branch master updated: JAMES-4009 StripAttachment should explicitly handle duplicates

2024-02-25 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 7a300ed824 JAMES-4009 StripAttachment

Re: [PR] JAMES-4009 StripAttachment should explicitly handle duplicates [james-project]

2024-02-25 Thread via GitHub
Arsnael merged PR #2043: URL: https://github.com/apache/james-project/pull/2043 -- 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-02-25 Thread via GitHub
Arsnael commented on code in PR #2048: URL: https://github.com/apache/james-project/pull/2048#discussion_r1502013047 ## mailbox/postgres/src/main/java/org/apache/james/mailbox/postgres/mail/PostgresAttachmentModule.java: ## @@ -35,20 +35,22 @@ public interface

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

2024-02-25 Thread via GitHub
hungphan227 commented on code in PR #2048: URL: https://github.com/apache/james-project/pull/2048#discussion_r1502005171 ##

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

2024-02-25 Thread via GitHub
quantranhong1999 commented on code in PR #2048: URL: https://github.com/apache/james-project/pull/2048#discussion_r1501987659 ##

Re: [PR] JAMES-4008 JMAP - Email/set - Should be able to save a draft with invalid email address [james-project]

2024-02-25 Thread via GitHub
chibenwa commented on PR #2040: URL: https://github.com/apache/james-project/pull/2040#issuecomment-1963077030 org.apache.james.jmap.rfc8621.memory.MemoryEmailSubmissionSetMethodTest.emailSubmissionSetCanBeChainedAfterEmailSet(GuiceJamesServer) Failing for the past 1 build (Since

Re: [PR] JAMES-4008 JMAP - Email/set - Should be able to save a draft with invalid email address [james-project]

2024-02-25 Thread via GitHub
chibenwa commented on PR #2040: URL: https://github.com/apache/james-project/pull/2040#issuecomment-1963077075 Related? -- 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: JAMES-4010 Do not index body (#2018)

2024-02-25 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 667c27e6b9 JAMES-4010 Do not index

Re: [PR] Do not index body [james-project]

2024-02-25 Thread via GitHub
chibenwa merged PR #2018: URL: https://github.com/apache/james-project/pull/2018 -- 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] Do not index body [james-project]

2024-02-25 Thread via GitHub
chibenwa commented on PR #2018: URL: https://github.com/apache/james-project/pull/2018#issuecomment-1963076095 https://issues.apache.org/jira/browse/JAMES-4010 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

(james-project) branch master updated: JAMES-3997 Netty backpressure for IMAP FETCH (#2031)

2024-02-25 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 915bcc33f2 JAMES-3997 Netty

Re: [PR] JAMES-3997 Netty backpressure for IMAP FETCH [james-project]

2024-02-25 Thread via GitHub
chibenwa commented on PR #2031: URL: https://github.com/apache/james-project/pull/2031#issuecomment-1963074605 Running in prod like wonder, no more OOMs since then. Merging... -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [PR] JAMES-3997 Netty backpressure for IMAP FETCH [james-project]

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

2024-02-25 Thread via GitHub
hungphan227 opened a new pull request, #2048: URL: https://github.com/apache/james-project/pull/2048 (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