FS#69462 - [tt-rss] Doesn't work after latest php 8.0.1 update

Attached to Project: Community Packages
Opened by Kurnevsky Evgeny (kurnevsky) - Thursday, 28 January 2021, 19:23 GMT
Last edited by Anatol Pomozov (anatolik) - Tuesday, 02 February 2021, 21:25 GMT
Task Type Bug Report
Category Packages
Status Closed
Assigned To Anatol Pomozov (anatolik)
Architecture All
Severity High
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 5
Private No

Details

Description:
tt-rss doesn't work after latest php 8.0.1 update. It just writes "Could not update headlines (invalid object received - see error console for details)". In the php logs I see:

2021/01/28 19:19:55 [error] 374#374: *108 FastCGI sent in stderr: "PHP message: PHP Warning: Array to string conversion in /usr/share/webapps/tt-rss/include/errorhandler.php on line 24PHP message: PHP Warning: Array to string conversion in /usr/share/webapps/tt-rss/include/errorhandler.php on line 24PHP message: PHP Warning: Array to string conversion in /usr/share/webapps/tt-rss/include/errorhandler.php on line 24PHP message: PHP Warning: Array to string conversion in /usr/share/webapps/tt-rss/include/errorhandler.php on line 24PHP message: PHP Warning: Array to string conversion in /usr/share/webapps/tt-rss/include/errorhandler.php on line 24PHP message: PHP Warning: Array to string conversion in /usr/share/webapps/tt-rss/include/errorhandler.php on line 24PHP message: PHP Warning: Array to string conversion in /usr/share/webapps/tt-rss/include/errorhandler.php on line 24PHP message: PHP Warning: Array to string conversion in /usr/share/webapps/tt-rss/include/errorhandler.php on line 24PHP message: PHP Warning: Array to string conversion in /usr/share/webapps/tt-rss/include/errorhandler.php on line 24PHP message: PHP Warning: Array to string conversion in /usr/share/webapps/tt-rss/include/errorhandler.php on line 24PHP message: PHP Warning: Array to string conversion in /usr/share/webapps/tt-rss/include/errorhandler.php on line 24PHP message: PHP Warning: Array to string conversion in /usr/share/webapps/tt-rss/include/errorhandler.php on line 24PHP message: PHP Warning: Array to string conversion in /usr/share/webapps/tt-rss/include/errorhandler.php on line 24PHP message: PHP Warning: Array to string conversion in /usr/share/webapps/tt-rss/include/errorhandler.php on line 24PHP message: PHP Warning: Array to string conversion in /usr/share/webapps/tt-rss/include/errorhandler.php on line 24PHP message: PHP Warning: Array to string conversion in /usr/share/webapps/tt-rss/include/errorhandler.php on line 24PHP message: PHP Warning: Array to string conversion in /usr/share/webapps/tt-rss/include/e

Additional info:
tt-rss version: 2:r10185.41bde84a9-1
This task depends upon

Closed by  Anatol Pomozov (anatolik)
Tuesday, 02 February 2021, 21:25 GMT
Reason for closing:  Fixed
Additional comments about closing:  2:r10198.6d8f2221b-2
Comment by Anatol Pomozov (anatolik) - Thursday, 28 January 2021, 21:19 GMT
Upstream says the master has PHP8 support https://community.tt-rss.org/t/current-version-can-support-php8/4108
Comment by Anatol Pomozov (anatolik) - Thursday, 28 January 2021, 21:21 GMT
I increased fastcgi buffer in the /etc/nginx/nginx.conf file and it seems helped tt-rss.


http {
.....
fastcgi_buffers 16 16k;
fastcgi_buffer_size 32k;
.....
}


My guess is that tt-rss spews a lot of PHP warnings back to nginx and it requires more buffer space.
Comment by Chih-Hsuan Yen (yan12125) - Friday, 29 January 2021, 19:25 GMT
>> I just updated my Arch Linux’s PHP to version 8

>lmao

Looks like upstream does not really support PHP 8. If you decide to go back to php7 for tt-rss, at least the following changes are needed:

* Change dependencies in PKGBUILD
* Change the command in tt-rss.service
* Patch sources so that the default value for PHP_EXECUTABLE is /usr/bin/php7
Comment by Oleksandr Natalenko (post-factum) - Sunday, 31 January 2021, 20:19 GMT
Playing with fastcgi_buffers doesn't make any difference. The package should use php7 for the time being.
Comment by Anatol Pomozov (anatolik) - Monday, 01 February 2021, 01:55 GMT
Chih-Hsuan Yen, don't you mind to share the PKGBUILD patch?

Oleksandr, the workaround definitely works for me. Make sure you add the fastcgi configuration into http{} block.
Comment by Anatol Pomozov (anatolik) - Monday, 01 February 2021, 02:11 GMT
Chih-Hsuan Yen, nevermind, I just modified PKGBUILD for php7 and uploaded the package to [community-testing]. Please test it.

One needs to:
- merge changes for /etc/webapps/tt-rss/config.php
- update /etc/php7/php.ini and enable extensions needed for tt-rss
- start/enable php-fpm7 service
Comment by Oleksandr Natalenko (post-factum) - Monday, 01 February 2021, 07:10 GMT
> Oleksandr, the workaround definitely works for me. Make sure you add the fastcgi configuration into http{} block.

No, it doesn't. But your recent changes that move the package to php7 do work.
Comment by Chih-Hsuan Yen (yan12125) - Monday, 01 February 2021, 09:57 GMT
> don't you mind to share the PKGBUILD patch?

Sorry if my comment was misleading. I didn't actually modify PKGBUILD but go with quick-and-dirty hacks and directly sudoedit various files.

I can also confirm after I revert my changes and install community-testing/tt-rss, it works like a charm again. Also signed-off the package:D
Comment by Anatol Pomozov (anatolik) - Tuesday, 02 February 2021, 01:14 GMT
Alright, a new tt-rss with dependency to PHP7 has been moved to [community]. It also has an INSTALL notice to make migration to php7 less painful. Please check it out and let me know if you have any issues.
Comment by Anatol Pomozov (anatolik) - Tuesday, 02 February 2021, 01:19 GMT
I also updated instructions at https://wiki.archlinux.org/index.php/TT-RSS But some things might be missing. Please review the documentation and change it if needed.
Comment by Oleksandr Natalenko (post-factum) - Tuesday, 02 February 2021, 20:07 GMT
Thanks for taking care of this. Looks like it is working fine now.

Loading...