YunoHost-Apps / freshrss_ynh

FreshRSS package for YunoHost
https://freshrss.org
GNU Affero General Public License v3.0
28 stars 20 forks source link

Testing : version 1.23.1 #176

Closed lapineige closed 5 months ago

lapineige commented 6 months ago

175

lapineige commented 6 months ago

Upgrading didn't work for me, I have a white screen afterwards.

This require manual testing...

OniriCorpe commented 6 months ago

maybe related to https://github.com/FreshRSS/FreshRSS/issues/5976

Alkarex commented 6 months ago

@lapineige Any logs?

OniriCorpe commented 6 months ago

I put this here:

Environment information:

lapineige commented 6 months ago

@lapineige Any logs?

Not that I could find.

lapineige commented 6 months ago

@Alkarex That file https://github.com/FreshRSS/FreshRSS/issues/5976#issuecomment-1869132178 content :

'extensions_enabled' => 
  array (
    'Google-Groups' => false,
    'Tumblr-GDPR' => false,
    'Invidious Video Feed' => true,
  ),
  'extensions' => 
  array (
  ),
Alkarex commented 6 months ago

@lapineige Your problem does not seem to be the same than https://github.com/FreshRSS/FreshRSS/issues/5976 A white screen most likely creates a log in your Web server logs and/or FreshRSS logs.

lapineige commented 6 months ago

Oh yes, this time (it wasn't the case before) I have this:

PHP Fatal error:  Uncaught TypeError: Minz_ExtensionManager::enable(): Argument #1 ($ext_name) must be of type string, int given, called in /var/www/freshrss/lib/Minz/ExtensionManager.php on line 279 and defined in /var/www/freshrss/lib/Minz/ExtensionManager.php:248
Stack trace:
#0 /var/www/freshrss/lib/Minz/ExtensionManager.php(279): Minz_ExtensionManager::enable()
#1 /var/www/freshrss/app/FreshRSS.php(63): Minz_ExtensionManager::enableByList()
#2 /var/www/freshrss/app/actualize_script.php(97): FreshRSS->init()
#3 {main}
  thrown in /var/www/freshrss/lib/Minz/ExtensionManager.php on line 248
Results:
Alkarex commented 6 months ago

Yes, same than https://github.com/FreshRSS/FreshRSS/issues/5976

Alkarex commented 6 months ago

https://github.com/FreshRSS/FreshRSS/pull/5979

Alkarex commented 6 months ago

Would you be able to try FreshRSS edge version before we release version 1.23.1?

lapineige commented 6 months ago

Where is it available ? Could we have a branch pointing to that specific release ?

Alkarex commented 6 months ago

That is just the edge branch of https://github.com/FreshRSS/FreshRSS Available by git, ZIP, tar... ZIP: https://github.com/FreshRSS/FreshRSS/archive/edge.zip You can also pick a specific commit if you like

lapineige commented 6 months ago

I see 1.3.1 is out, so we will wait for it :)

lapineige commented 6 months ago

Upgrade went fine in my case. Ready for merge. ping @plopoyop