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

Nextcloud 23 #195

Closed
MiRei777 opened this issue Dec 15, 2021 · 7 comments
Closed

Nextcloud 23 #195

MiRei777 opened this issue Dec 15, 2021 · 7 comments
Labels
bug needs investigation Needs an investigation into the technichal reasons or feasibility

Comments

@MiRei777
Copy link

In NC23 the errorlog generate permanaent massive warnings:

"app":"keeweb","method":"POST","url":"/settings/apps/disable","message":"/appinfo/app.php is deprecated, use \OCP\AppFramework\Bootstrap\IBootstrap on the application class instead.","userAgent":"Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/96.0.4664.110 Safari/537.36","version":"23.0.0.10","id":"61b9d873cd6c7"}

@Potemkinkuliss
Copy link

It is the same for me:
"app":"keeweb","method":"GET","url":"/ocs/v2.php/search/providers/settings_apps/search?term=mail&from=%2Fsettings%2Fadmin%2Flogging","message":"/appinfo/app.php is deprecated, use \OCP\AppFramework\Bootstrap\IBootstrap on the application class instead.","userAgent":"Mozilla/5.0 (X11; Linux x86_64; rv:96.0) Gecko/20100101 Firefox/96.0","version":"23.0.0.10"}

@arnowelzel arnowelzel added bug needs investigation Needs an investigation into the technichal reasons or feasibility labels Jan 29, 2022
@arnowelzel
Copy link
Collaborator

arnowelzel commented Feb 26, 2022

It seems this is a warning message which is only logged when debug messages are enabled in the log. The application still works fine though - Nextcloud just warns about using a deprecated implementation.

The reason is, that this was the implementation needed for older Nextcloud versions (<20) - and Keeweb started long before Nextcloud 20.

I'll work migrating the code newer Application class structure as it is recommended for Nextcloud 20 or newer, but this may take some time. Until then you may just disable the debug messages to get rid of those warnings.

@Webxorcist
Copy link

As of Nextcloud 23.0.3 this add on seems to be completely broken. Once it is loaded, every Nextcloud page shows "Bad Gateway" until I disable it using the occ tool.

@arnowelzel
Copy link
Collaborator

Sorry to hear that - but for me it works fine even with NC 23.0.3. Can you provide server logs for this?

@Webxorcist
Copy link

I have the loglevel on 0, according to the documentation this is the highest possible level, but all I am getting in the log is this:

{"reqId":"iEUr61Bh1J4M1uykep9w","level":0,"time":"2022-03-28T13:40:01+00:00","remoteAddr":"","user":"--","app":"keeweb","method":"","url":"--","message":"/appinfo/app.php is deprecated, use \\OCP\\AppFramework\\Bootstrap\\IBootstrap on the application class instead.","userAgent":"--","version":"23.0.3.2"}

But I think I found something, and if that is true you can ignore my post. It seems I had a rights problem in the appdata cache for the /appinfo/app.php. Somehow this triggers way more when keeweb is loaded. I fixed this and now I can't reproduce.

Maybe this is also a cache issue, it seems Keeweb is ignoring the dark mode theme.

Thanks for your fast reply.

@arnowelzel
Copy link
Collaborator

arnowelzel commented May 17, 2022

Please do not use log level 0 for production systems! This means to log everything including debug and info messages.

For regular use you should only use level 2 to get warnings, errors and fatal errors.

In config/config.php of your Nextcloud installation make sure that the loglevel is set to 2:

  'loglevel' => 2,

Also see https://docs.nextcloud.com/server/latest/admin_manual/configuration_server/logging_configuration.html.

Quote:

By default the log level is set to 2 (WARN). Use DEBUG when you have a problem to diagnose, and then reset your log level to a less-verbose level as DEBUG outputs a lot of information, and can affect your server performance.

@arnowelzel
Copy link
Collaborator

Closing this issue since it is not relevant any longer. NC 24 is not supported any more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug needs investigation Needs an investigation into the technichal reasons or feasibility
Projects
None yet
Development

No branches or pull requests

4 participants