OPNsense Forum

Archive => 20.1 Legacy Series => Topic started by: dinguz on May 22, 2020, 09:10:09 PM

Title: os-acme-client error in 20.1.7
Post by: dinguz on May 22, 2020, 09:10:09 PM
After the upgrade, I get this warning every time the script is being run from cron:

20 00:05:00 Europe/Amsterdam] PHP Warning:  "continue" targeting switch is equivalent to "break". Did you mean to use "continue 2"? in /usr/local/opnsense/scripts/OPNsense/AcmeClient/certhelper.php on line 1421
[22-May-2020 00:05:00 Europe/Amsterdam] PHP Warning:  "continue" targeting switch is equivalent to "break". Did you mean to use "continue 2"? in /usr/local/opnsense/scripts/OPNsense/AcmeClient/certhelper.php on line 1428
Title: Re: os-acme-client error in 20.1.7
Post by: Maurice on May 22, 2020, 10:27:03 PM
Same here. Probably related to PHP 7.3 update?
But it's just a warning, I haven't observed any actual malfunction. Did you?
Title: Re: os-acme-client error in 20.1.7
Post by: STRUBartacus on May 23, 2020, 07:19:49 PM
Same here, but no problems observed...
Title: Re: os-acme-client error in 20.1.7
Post by: Maurice on May 23, 2020, 07:50:25 PM
Already fixed: https://github.com/opnsense/plugins/commit/ee799d8c75f7aaffaee97439c95fd98263eb1b38
Title: Re: os-acme-client error in 20.1.7
Post by: franco on May 24, 2020, 10:57:52 AM
Slipped through our screening process, but at least has no averse affects on operation.


Cheers,
Franco
Title: Re: os-acme-client error in 20.1.7
Post by: cpw on May 24, 2020, 04:02:42 PM
It generated an "issue has occurred, report it" warning for me overnight. Glad it's been fixed..
Title: Re: os-acme-client error in 20.1.7
Post by: STRUBartacus on May 24, 2020, 11:17:06 PM
Quote from: cpw on May 24, 2020, 04:02:42 PM
It generated an "issue has occurred, report it" warning for me overnight. Glad it's been fixed..

Just disable the "AcmeClient Cronjob" until the next update... (do not forget to renew the certificates before)
Title: Re: os-acme-client error in 20.1.7
Post by: franco on May 25, 2020, 08:35:26 AM
Why o why, it's just a warning pointing to bad programming style and everything keeps working. Disabling your cron job will disable your renew...


Cheers,
Franco