DigitalDJ / tinytinyrss-fever-plugin

Tiny Tiny RSS Fever API Plugin
https://github.com/DigitalDJ/tinytinyrss-fever-plugin
GNU General Public License v3.0
207 stars 28 forks source link

Recent tt-rss update broke the plugin #28

Closed shantheone closed 2 years ago

shantheone commented 2 years ago

I receive the following messages in the tt-rss error log since the 2021.11.18. updates:

E_COMPILE_ERROR (64)
plugins.local/fever/fever_api.php:113

Declaration of FeverAPI::before($method) must be compatible with Handler::before(string $method): bool
Real IP: <omitted>
Forwarded For: <omitted>
Forwarded Protocol: https
Remote IP: <omitted>
Request URI: /tt-rss/plugins.local/fever/?api&feeds
User agent: Reeder/5001.29.01 CFNetwork/1325.0.1 Darwin/21.1.0

I'm running tt-rss in docker and using Reeder 5 as client if that's relevant. Please let me know if you need more information to help solve the issue!

phenomeus commented 2 years ago

same here, unfortunately

phenomeus commented 2 years ago

because of this: https://community.tt-rss.org/t/heads-up-ongoing-attempt-to-comply-with-phpstan-rule-level-6/5046

eric-pierce commented 2 years ago

I updated the API earlier today to fix this and opened #27 to merge the updates into this repo. If you don’t want to wait for the pull request to be merged you can use my fork of the repo: https://github.com/eric-pierce/tinytinyrss-fever-plugin

eric-pierce commented 2 years ago

The update is now merged into master - @shantheone after confirming that everything is working would you close this issue?

shantheone commented 2 years ago

I've updated the plugin and tested with Reeder. The error messages are gone and Reeder is working fine.

Thank you for the quick response!

phenomeus commented 2 years ago

thank you for the fast reaction.

markwaters commented 2 years ago

I have upgraded the fever plugin to version 2.3.0 but if I enable it then I start seeing a warning message showing - "Tiny Tiny RSS is running in Safe Mode. All themes and plugins are disabled. You will need to logout and back into to disable it" I have even removed the whole /fever directory and installed from github , same error. Running on latest baremetal debian. Thanks. Mark.

markwaters commented 2 years ago

I have removed , downloaded the latest fever source again. When I add the fever plugin to plugins.local/ I see this error when visiting the plugins page. "Error while loading plugins list: RequestError: Unable to load backend.php status: 500." When I remove the fuever directory everything works again as expected.

eric-pierce commented 2 years ago

@markwaters I'm not able to re-create the issue you're seeing, with the latest version of TT-RSS using docker containers all is working fine with my install. Are you running the latest version of TT-RSS using the docker based installation method?

markwaters commented 2 years ago

Hey @eric-pierce ,no I am not using docker ,it is baremetal debian with php8.