You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@james.apache.org by "woj-tek (via GitHub)" <gi...@apache.org> on 2023/04/28 00:19:38 UTC

[GitHub] [james-project] woj-tek opened a new pull request, #1546: Don't add gateway if no gateway is configured (empty value)

woj-tek opened a new pull request, #1546:
URL: https://github.com/apache/james-project/pull/1546

   If there is an empty `<gateway/>` element in configuration then RemoteDelivery adds empty values to collection resulting in entry `[:]` which then fails to deliver. I'd argue that in that case James should skip such configuration and use `serviceNoGateway()` method.
   
   Example configuration:
   ```xml
   <processor state="relay" enableJmx="true">
       <mailet match="All" class="RemoteDelivery">
           <outgoingQueue>outgoing</outgoingQueue>
           <delayTime>5000, 100000, 500000</delayTime>
           <maxRetries>3</maxRetries>
           <maxDnsProblemRetries>0</maxDnsProblemRetries>
           <deliveryThreads>10</deliveryThreads>
           <sendpartial>true</sendpartial>
           <bounceProcessor>bounces</bounceProcessor>
           <debug>true</debug>
           <gateway></gateway>
           <gatewayPort></gatewayPort>
           <gatewayUsername></gatewayUsername>
           <gatewayPassword></gatewayPassword>
       </mailet>
   </processor>
   ```


-- 
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-unsubscribe@james.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@james.apache.org
For additional commands, e-mail: notifications-help@james.apache.org


[GitHub] [james-project] chibenwa merged pull request #1546: Don't add gateway if no gateway is configured (empty value)

Posted by "chibenwa (via GitHub)" <gi...@apache.org>.
chibenwa merged PR #1546:
URL: https://github.com/apache/james-project/pull/1546


-- 
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-unsubscribe@james.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@james.apache.org
For additional commands, e-mail: notifications-help@james.apache.org