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

Testing : version 1.23.1 #176

Merged
merged 3 commits into from Jan 14, 2024
Merged

Testing : version 1.23.1 #176

merged 3 commits into from Jan 14, 2024

Conversation

lapineige
Copy link
Member

@lapineige lapineige commented Dec 24, 2023

ericgaspar and others added 2 commits December 9, 2023 21:36
* Upgrade to v1.23.0

* Auto-update README
@lapineige
Copy link
Member Author

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

This require manual testing...

@OniriCorpe
Copy link
Member

maybe related to FreshRSS/FreshRSS#5976

@Alkarex
Copy link

Alkarex commented Dec 25, 2023

@lapineige Any logs?

@OniriCorpe
Copy link
Member

I put this here:

Environment information:

  • Device:
  • OS: Debian Bullseye
  • Browser:
  • FreshRSS version: 1.23.0
  • Database version: MariaDB 10.5.21-0+deb11u1
  • PHP version: PHP 8.2
  • Installation type: Yunohost
  • Installed dependencies: see here

@lapineige
Copy link
Member Author

@lapineige Any logs?

Not that I could find.

@lapineige
Copy link
Member Author

@Alkarex
That file FreshRSS/FreshRSS#5976 (comment) content :

'extensions_enabled' => 
  array (
    'Google-Groups' => false,
    'Tumblr-GDPR' => false,
    'Invidious Video Feed' => true,
  ),
  'extensions' => 
  array (
  ),

@Alkarex
Copy link

Alkarex commented Dec 26, 2023

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

@lapineige
Copy link
Member Author

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
Copy link

Alkarex commented Dec 26, 2023

Yes, same than FreshRSS/FreshRSS#5976

@Alkarex
Copy link

Alkarex commented Dec 26, 2023

FreshRSS/FreshRSS#5979

Alkarex added a commit to FreshRSS/FreshRSS that referenced this pull request Dec 26, 2023
@Alkarex
Copy link

Alkarex commented Dec 29, 2023

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

@lapineige
Copy link
Member Author

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

@Alkarex
Copy link

Alkarex commented Dec 29, 2023

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
Copy link
Member Author

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

* Upgrade to v1.23.1

* Auto-update README
@lapineige lapineige changed the title Testing : version 1.23.0 Testing : version 1.23.1 Dec 31, 2023
@lapineige
Copy link
Member Author

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

@alexAubin alexAubin merged commit 38e8cfe into master Jan 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants