search

sympa-community / sympa

Sympa, Mailing List Management Software
https://www.sympa.community/sympa
GNU General Public License v2.0
237 stars 95 forks source link

Missing check on "Digest frequency" #1742

Closed fchaumontet closed 7 months ago

fchaumontet commented 7 months ago

Users can defined digest frequency with minutes = 99. It make sympa process crash. Attached files :

Version

Sympa 6.2.68

Installation method

source package

Expected behavior

Integrate a value check in new release in the configuration screen Add a value check in ProcessDigest.pm that log the list name

Steps to reproduce

Go to "Edit list Config" > "Sending/receiving setup" Enter hour or/and minute out of range (27h99m for example) Start sympa.

ListConfig-InvalidDigestFrequency SympaLogExtract-hide.log

ikedas commented 7 months ago

@fchaumontet , thank you for reporting bug!

Could you please apply this patch and check if the problem will be solved?

fchaumontet commented 7 months ago

Hi, We just try the patch. It work for minutes but not for hours image

ikedas commented 7 months ago

Hi @fchaumontet , Sorry. Could you please apply additional patch and check again?

fchaumontet commented 7 months ago

Hello, It looks good. Thanks for the patch. Regards