Univention Bugzilla – Bug 51124
add UCRV for Postfix settings delay_warning_time, maximal_queue_lifetime and
Last modified: 2020-09-30 15:37:46 CEST
The setting http://www.postfix.org/postconf.5.html#delay_warning_time defaults to 0 by default. As a result no warnings for temporarily undeliverable messages are sent until http://www.postfix.org/postconf.5.html#maximal_queue_lifetime (or http://www.postfix.org/postconf.5.html#bounce_queue_lifetime) is reached. Those default to 5 days. Even if it is possible to define all these settings in main.cf.local it would make sense to make them controllable by UCR and - for larger installations - by a UCR-policy.
I added the UCRVs... mail/postfix/maximal_queue_lifetime mail/postfix/bounce_queue_lifetime mail/postfix/delay_warning_time in commit commit fef58444c3de07f49cd9b0126d7f0008924c6e73 Bug #51124: added postfix UCRVs on branch troehmey/bug51124_add_postfix_ucrv
QA: looks Ok, but I have some remarks: mail/postfix/maximal_queue_lifetime -> UCRV text is ok, but the english translation is slightly different than the german text. mail/postfix/delay_warning_time -> Bei gescheiterter Maileinlieferung, -> comma is not needed. 10_general -> You can simplefy your code a little bit: delay = configRegistry.get('mail/postfix/delay_warning_time', '') if delay: print('delay_warning_time = %s' % delay) I would also add the default like in mail/postfix/postscreen/greet/ttl and add the default value to be more explicit (e.g.): delay = configRegistry.get('mail/postfix/delay_warning_time', '0h') if delay: print('delay_warning_time = %s' % delay)
Applied suggested fixes with commit commit db9585f31d71a39e7e83975a04914eaa27a206c0 Bug #51124: improved description texts and code clarity
REOPEN Please add the Type & Categories in the UCR-V file. Otherwise ucr info mail/postfix/delay_warning_time can't be displayed. Other than that the file is updated, when the ucr-vs change.
Sorry, totally overlooked that... Fixed with... commit c0c8fb27e5cf15fa0b8bd56c625f495436e5ed87 Bug #51124: add type and category to UCRVs
Thanks for the quick response! Now the info text is displayed as expected. REOPEN for merge to 4.4-6 & build.
Merged with commits eb8254e2e4 Bug #51124: added yaml 14366ea37f Bug #51124: added changelog entry c3cbfbfa95 Bug #51124: Merge branch 'troehmey/bug51124_add_postfix_ucrv' into 4.4-6 successful build: Package: univention-mail-postfix Version: 13.0.2-2A~4.4.0.202009251042 Branch: ucs_4.4-0 Scope: errata4.4-6
QA -> All OK -> VERIFY Changelog -> OK YAML -> OK Merge without conflicts -> OK
<https://errata.software-univention.de/#/?erratum=4.4x758>