Email Reports PHP warning -> report not created

patient0

Hello,

I'm on pfSense+ 24.11 and setup email notification today in System / Advanced / Notifications, "Test SMTP Settings" is successful and the I get the test email.

After that I wanted to create a report under Status / Email Reports. No errors in the GUI when creating the report but no report showed up. In the System / General log there are some php-fm warnings related to it:

2025-02-27 19:19:01.267297+01:00 	php-fpm 	16844 	/status_mail_report_edit.php: Configuration Change: admin@172.16.16.4 (Local Database): mailreport: Settings updated
2025-02-27 19:19:01.257450+01:00 	php-fpm 	16844 	/status_mail_report_edit.php: config warning: invalid path "mailreports/schedule//" at /usr/local/www/status_mail_report_edit.php:210

Am I missing a vital step?

Gertjan

@patient0 said in Email Reports PHP warning -> report not created:

Am I missing a vital step?

Look 3 posts lower.
Email Reports WebGUI crashes February 2025,
there is a patch proposed over there. With some luck it's the same issue.

patient0

@Gertjan Thank you, I did see it but did not fetch.
Now I fetched it but there is not apply button.

Gertjan

@patient0

I don't know (saw) what you did.
I can't apply neither, as I don't have that package.

If Apply isn't there, you can see the reason.
Use "Debug".
You probably see :

which means the patch is based upon another source version.

Was that packet upgraded recently ?
Maybe to solution is now different ?

patient0

I now tested it on a pristine 24.11 in a VM and the issue is the same. There were no other packages installed beside System_Patches, which I only installed afterward.

Applying all patches and reboot didn't change a thing. Still the smae error at the same position (line 210). The error does not occur on 25.03-BETA

207         if ($a_mailreports) {
208                 config_set_path("mailreports/schedule/{$id}", $pconfig);
209         } else {
210                 config_set_path("mailreports/schedule/{$id}/", $pconfig);
211         }

Update: Well, since the difference between line 208 and 210 is only the '/', I removed it in 210 and now it does work.
The complete if() seems not to be necessary anymore. At least for me.

Last Update: I compared it to 25.03-BETA and went with that option:

207         if ($a_mailreports) {
208                 config_set_path("mailreports/schedule/{$id}", $pconfig);
209         } else {
210                 config_set_path("mailreports/schedule/", $pconfig);
211         }

Gertjan

@patient0 said in Email Reports PHP warning -> report not created:

mailreports

is a pfSense package, right ?
Normally (imho, afaik), these don't receive (official) "system patches", the package is updated and you have to upgrade it.
Is the version number the same between 24.11 and 25.03-beta ?

patient0

@Gertjan said in Email Reports PHP warning -> report not created:

Is the version number the same between 24.11 and 25.03-beta ?

Nope, it's a minor minor version change:
24.11 3.6.4_4
25.03-BETA 3.6.4_5

I did search on Redmine for issues with the package but my searching skills didn't show anything.

Gertjan

@patient0

This one : https://redmine.pfsense.org/issues/15872 ?

patient0

@Gertjan said in Email Reports PHP warning -> report not created:

@patient0

This one : https://redmine.pfsense.org/issues/15872 ?

I don't think so, it's on line 44 and my issue was on line 210 and not an PHP error but ${id} not set. But not very important, it's fixed and that's good.