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

php 8.0 Fatal Error when allowing images in an email #7968

Closed
reubenfarrelly opened this issue Mar 23, 2021 · 3 comments
Closed

php 8.0 Fatal Error when allowing images in an email #7968

reubenfarrelly opened this issue Mar 23, 2021 · 3 comments

Comments

@reubenfarrelly
Copy link

Triggered when bring up an email that has images, and clicking on "Always allow from ".

The UI displays a red Internal Error message and the log show the following fatal error:

[24-Mar-2021 09:11:37 Australia/Sydney] PHP Fatal error: Uncaught TypeError: Illegal offset type in isset or empty in /var/www/roundcube/htdocs/program/lib/Roundcube/rcube.php:650
Stack trace:
#0 /var/www/roundcube/htdocs/program/include/rcmail_output_json.php(141): rcube->text_exists()
#1 /var/www/roundcube/htdocs/program/actions/mail/addcontact.php(93): rcmail_output_json->show_message()
#2 /var/www/roundcube/htdocs/program/include/rcmail.php(275): rcmail_action_mail_addcontact->run()
#3 /var/www/roundcube/htdocs/index.php(283): rcmail->action_handler()
#4 {main}
thrown in /var/www/roundcube/htdocs/program/lib/Roundcube/rcube.php on line 650

This is with php 8.0.3 and git -head up to date as of about a week ago.

@alecpl
Copy link
Member

alecpl commented Mar 24, 2021

Looks like it might happen if the email address is invalid. Is that the case?

@reubenfarrelly
Copy link
Author

I can't be 100% sure - but that may be the case. Most addresses work just fine, just a few that fail like that. II'll pay attention going forward and report back here in a few days if there hasn't been any other comments or commits related to it.

@alecpl
Copy link
Member

alecpl commented Mar 26, 2021

I wasn't able to reproduce, but it should be fixed now.

@alecpl alecpl closed this as completed Mar 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants