Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

How can i fix the mail issue? #483

Closed
n-for-all opened this issue Mar 22, 2024 · 1 comment
Closed

How can i fix the mail issue? #483

n-for-all opened this issue Mar 22, 2024 · 1 comment
Assignees
Labels

Comments

@n-for-all
Copy link

n-for-all commented Mar 22, 2024

I am using the default configuration, even if i remove the mail rule in the default rules, i still get the logs below:

[snuffleupagus][0.0.0.0][config][log]   - 3 parameter's name: 'additional_headers'-/public_html/vendor/phpmailer/phpmailer/src/PHPMailer.php-874
[snuffleupagus][0.0.0.0][config][log]   - 4 parameter's name: 'additional_params'-/public_html/vendor/phpmailer/phpmailer/src/PHPMailer.php-874
[snuffleupagus][0.0.0.0][config][log]   - 0 parameter's name: 'to'-/public_html/vendor/phpmailer/phpmailer/src/PHPMailer.php-874
[snuffleupagus][0.0.0.0][config][log]   - 1 parameter's name: 'subject'-/public_html/vendor/phpmailer/phpmailer/src/PHPMailer.php-874
[snuffleupagus][0.0.0.0][config][log]   - 2 parameter's name: 'message'-/public_html/vendor/phpmailer/phpmailer/src/PHPMailer.php-874
[snuffleupagus][0.0.0.0][config][log] It seems that you are filtering on a parameter 'additional_parameters' of the function 'mail', but the parameter does not exists.-/public_html/vendor/phpmailer/phpmailer/src/PHPMailer.php-874

I am using php8.3

@jvoisin
Copy link
Owner

jvoisin commented Mar 24, 2024

You can change the rule sp.disable_function.function("mail").param("additional_parameters").value_r("\\-").drop(); to sp.disable_function.function("mail").param("additional_params").value_r("\\-").drop();.

The root cause is php changing parameter names from time to time, this time from additional_parameters to additional_params in mail.

This should be fixed by 6fe3815

@jvoisin jvoisin closed this as completed Mar 24, 2024
@jvoisin jvoisin self-assigned this Mar 24, 2024
@jvoisin jvoisin added the rules label Mar 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants