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

2024-03-10 Thread via GitHub
vttranlina commented on PR #2048: URL: https://github.com/apache/james-project/pull/2048#issuecomment-1987633242 > Thanks. > > Can you test guys locally on those tests and make sure it passes with those fixes? =>

[PR] JAMES-2586 [Postgres] FIXUP when query with IN - should pre-check collection size [james-project]

2024-03-10 Thread via GitHub
vttranlina opened a new pull request, #2103: URL: https://github.com/apache/james-project/pull/2103 When debugging why CI failed at https://github.com/apache/james-project/pull/2048#issuecomment-1987593874 I realized that Jooq will hang when the input to `IN` clause is empty. =>

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

2024-03-10 Thread via GitHub
Arsnael commented on PR #2048: URL: https://github.com/apache/james-project/pull/2048#issuecomment-1987619546 Thanks. Can you test guys locally on those tests and make sure it passes with those fixes? =>

Re: [PR] Postgres integration tests fix build [james-project]

2024-03-10 Thread via GitHub
Arsnael commented on PR #2095: URL: https://github.com/apache/james-project/pull/2095#issuecomment-1987612629 https://ci-builds.apache.org/job/james/job/ApacheJames/job/PR-2095/3/testReport/ I can reproduce two of those 3 locally (fails everytime): -

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

2024-03-10 Thread via GitHub
vttranlina commented on PR #2048: URL: https://github.com/apache/james-project/pull/2048#issuecomment-1987593874 > fixup! JAMES-2586 Optimize AttachmentLoader Fixed https://github.com/apache/james-project/pull/2048/commits/144ac90c6483eda966301297afcf9c5cc73e55d6 Reason:

(james-project) branch master updated: [UPDATE] commons-compress 1.24.0 -> 1.26.0 (#2089)

2024-03-10 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 e858bad711 [UPDATE] commons-compress

Re: [PR] [FIX] Try recover invalid ascii character in filename [james-project]

2024-03-10 Thread via GitHub
Arsnael merged PR #2097: URL: https://github.com/apache/james-project/pull/2097 -- 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 (09e3803613 -> ed1c2e3d52)

2024-03-10 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 09e3803613 [FIX] RSpamD should not fail when reporting empty body add ed1c2e3d52 [FIX] Try recover

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

2024-03-10 Thread via GitHub
Arsnael merged PR #2089: URL: https://github.com/apache/james-project/pull/2089 -- 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] RSpamD should not fail when reporting empty body

2024-03-10 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 09e3803613 [FIX] RSpamD should not

Re: [PR] [FIX] RSpamD should not fail when reporting empty body [james-project]

2024-03-10 Thread via GitHub
Arsnael merged PR #2098: URL: https://github.com/apache/james-project/pull/2098 -- 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 (d1bfeddc7e -> 35fa8040ff)

2024-03-10 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 d1bfeddc7e JAMES-2644 ContentTypeCleaner: add a test with a parameter add 35fa8040ff JAMES-3534 Fixbug

Re: [PR] JAMES-3534 Fixbug - Identity response alway return mayDelete=false when server set identity [james-project]

2024-03-10 Thread via GitHub
Arsnael merged PR #2087: URL: https://github.com/apache/james-project/pull/2087 -- 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-2644 Various COntentTypeCleaner enhancements [james-project]

2024-03-10 Thread via GitHub
Arsnael merged PR #2101: URL: https://github.com/apache/james-project/pull/2101 -- 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 (956cf750ba -> d1bfeddc7e)

2024-03-10 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 956cf750ba [FIX] Enforce a separate keyspace for blob store cache (#2060) add 831c6c7433 JAMES-2644

Re: [PR] [FIX] JMAP: Set configurable limits for /get /set objects [james-project]

2024-03-10 Thread via GitHub
Arsnael commented on code in PR #2096: URL: https://github.com/apache/james-project/pull/2096#discussion_r1519084111 ## server/protocols/jmap-rfc-8621/src/main/scala/org/apache/james/jmap/method/Method.scala: ## @@ -56,6 +56,35 @@ trait Method { trait WithAccountId { def

Re: [PR] [FIX] JMAP: Set configurable limits for /get /set objects [james-project]

2024-03-10 Thread via GitHub
quantranhong1999 commented on code in PR #2096: URL: https://github.com/apache/james-project/pull/2096#discussion_r1519080682 ## server/protocols/jmap-rfc-8621/src/main/scala/org/apache/james/jmap/core/JmapRfc8621Configuration.scala: ## @@ -46,6 +46,8 @@ object

Re: [PR] [ENHANCEMENT] Decrease log level for "NotSslRecordException" [james-project]

2024-03-10 Thread via GitHub
quantranhong1999 commented on code in PR #2102: URL: https://github.com/apache/james-project/pull/2102#discussion_r1519074376 ## server/protocols/protocols-imap4/src/main/java/org/apache/james/imapserver/netty/ImapChannelUpstreamHandler.java: ## @@ -30,6 +30,7 @@ import

[PR] [ENHANCEMENT] Decrease log level for "NotSslRecordException" [james-project]

2024-03-10 Thread via GitHub
chibenwa opened a new pull request, #2102: URL: https://github.com/apache/james-project/pull/2102 This pollutes the logs. ERROR logs are for really bad thing needing immediate admin attention, ideally triggering an alarm. Those are not to be triggered when a guy try telnet on

[PR] JAMES-2644 Various COntentTypeCleaner enhancements [james-project]

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

[PR] JAMES-4018 SMTP RCPT TO should parse parameters once [james-project]

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

[PR] [FIX] Ship mail over web webadmin routes into more apps [james-project]

2024-03-10 Thread via GitHub
chibenwa opened a new pull request, #2099: URL: https://github.com/apache/james-project/pull/2099 Reported from the matric channel by البهي @muhammadbahey_twitter:gitter.im > I tried to james:jpa-3.8.0 and tried to call curl -XPOST /mail-transfer-service but it gives 404 NOT

[PR] [FIX] RSpamD should not fail when reporting empty body [james-project]

2024-03-10 Thread via GitHub
chibenwa opened a new pull request, #2098: URL: https://github.com/apache/james-project/pull/2098 Technicly this can be caused by buggy IMAP clients When that happens then this causes a cascade of failures with dead letter as this gets retried. Better ignore... -- This is

(james-project) branch postgresql updated: JAMES 2586 PostgresPushSubscriptionRepository: rely on Postgres unique constraint for deviceClientId (#2094)

2024-03-10 Thread btellier
This is an automated email from the ASF dual-hosted git repository. btellier 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 29c85f3fe2 JAMES 2586

Re: [PR] JAMES 2586 PostgresPushSubscriptionRepository: rely on Postgres unique constraint for deviceClientId [james-project]

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

2024-03-10 Thread via GitHub
chibenwa commented on PR #2089: URL: https://github.com/apache/james-project/pull/2089#issuecomment-1987351633 Finally! Review please ;-) -- 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

[PR] [FIX] Try recover invalid ascii character in filename [james-project]

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