Skip to content

Replace constants in the bounce handling code by config.php settings#963

Merged
aulona1 merged 1 commit intophpList:release-3.6.14from
bramley:bounce_config
Sep 13, 2023
Merged

Replace constants in the bounce handling code by config.php settings#963
aulona1 merged 1 commit intophpList:release-3.6.14from
bramley:bounce_config

Conversation

@bramley
Copy link
Contributor

@bramley bramley commented Jun 3, 2023

Description

Allow some hard-coded settings to be specifed in config.php

  • maximum number of bounces to retrieve from the mailbox, default is 100,000
  • the number of bounces to process in one batch for bounce rules, default is 500
  • the name of the mailbox, default is INBOX

Further, allow more than one mailbox to be specified. This is useful to be able to retrieve bounces that have been put in a Junk folder on the server.

Related Issue

Screenshots (if appropriate):

@bramley bramley changed the title Replace constants in the code by config.php settings Replace constants in the bounce handling code by config.php settings Jun 3, 2023
@michield michield self-requested a review July 27, 2023 12:43
@michield
Copy link
Member

Haven't tested it, but reading the changes looks fine to me.

@aulona1 aulona1 changed the base branch from main to release-3.6.14 September 13, 2023 15:11
@aulona1 aulona1 self-requested a review September 13, 2023 15:12
@aulona1 aulona1 merged commit 118f390 into phpList:release-3.6.14 Sep 13, 2023
@phpListDockerBot
Copy link
Contributor

This pull request has been mentioned on phpList Discuss. There might be relevant details there:

https://discuss.phplist.org/t/3-6-14-release-candidate-ready-for-testing/9109/1

@bramley bramley deleted the bounce_config branch September 18, 2023 22:25
@phpListDockerBot
Copy link
Contributor

This pull request has been mentioned on phpList Discuss. There might be relevant details there:

https://discuss.phplist.org/t/phplist-3-6-14-released-security-release/9158/1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants