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

25.2.0 is temporarily broken after update #3043

Closed
3 tasks done
provokateurin opened this issue Jan 10, 2025 · 2 comments
Closed
3 tasks done

25.2.0 is temporarily broken after update #3043

provokateurin opened this issue Jan 10, 2025 · 2 comments
Labels

Comments

@provokateurin
Copy link
Member

IMPORTANT

Read and tick the following checkbox after you have created the issue or place an x inside the brackets ;)

  • I have read the CONTRIBUTING.md and followed the provided tips
  • I accept that the issue will be closed without comment if I do not check here
  • I accept that the issue will be closed without comment if I do not fill out all items in the issue template.

Explain the Problem

I just upgraded to 25.2.0 and now I get an error in the logs (see below) and the web frontend doesn't work anymore (only 500 errors).
After about 2 minutes this issue is gone.
I guess some internal changes were made and the time period where the changes are not fully applied yet needs to be handled more gracefully.

Steps to Reproduce

Explain what you did to encounter the issue

  1. Be on 25.1.2 and have a bunch of feeds
  2. Update to 25.2.0
  3. See that the errors are thrown and the web frontend is broken
  4. Wait a few minutes, then everything will work again.

System Information

  • News app version: 25.2.0
  • Nextcloud version: 30.0.4
  • Cron type: cron
  • PHP version: 8.3
  • Database and version: PostgresSQL 15
  • Browser and version: Firefox 134
  • OS and version: Arch Linux
Contents of nextcloud/data/nextcloud.log
{"reqId":"vZSNjhLXKNsQj5fxbOUK","level":3,"time":"2025-01-10T11:32:36+00:00","remoteAddr":"10.42.0.1","user":"kate","app":"index","method":"GET","url":"/apps/news/items?limit=40&oldestFirst=false&search=&showAll=true&type=2","message":"nextUpdateTime is not a valid attribute","userAgent":"Mozilla/5.0 (X11; Linux x86_64; rv:134.0) Gecko/20100101 Firefox/134.0","version":"30.0.1.2","exception":{"Exception":"BadFunctionCallException","Message":"nextUpdateTime is not a valid attribute","Code":0,"Trace":[{"file":"/var/www/html/lib/public/AppFramework/Db/Entity.php","line":56,"function":"setter","class":"OCP\\AppFramework\\Db\\Entity","type":"->","args":["nextUpdateTime",[null]]},{"file":"/var/www/html/lib/public/AppFramework/Db/QBMapper.php","line":299,"function":"fromRow","class":"OCP\\AppFramework\\Db\\Entity","type":"::","args":[[170,10,"kate",1733025029023147,"6bc048d9c13a184e1e432fefc7cdb972","And 20 more entries, set log level to debug to see all entries"]]},{"file":"/var/www/html/lib/public/AppFramework/Db/QBMapper.php","line":317,"function":"mapRowToEntity","class":"OCP\\AppFramework\\Db\\QBMapper","type":"->","args":[[170,10,"kate",1733025029023147,"6bc048d9c13a184e1e432fefc7cdb972","And 20 more entries, set log level to debug to see all entries"]]},{"file":"/var/www/html/custom_apps/news/lib/Db/FeedMapperV2.php","line":70,"function":"findEntities","class":"OCP\\AppFramework\\Db\\QBMapper","type":"->","args":[["OC\\DB\\QueryBuilder\\QueryBuilder"]]},{"file":"/var/www/html/custom_apps/news/lib/Service/FeedServiceV2.php","line":101,"function":"findAllFromUser","class":"OCA\\News\\Db\\FeedMapperV2","type":"->","args":["kate",[]]},{"file":"/var/www/html/custom_apps/news/lib/Controller/ItemController.php","line":124,"function":"findAllForUser","class":"OCA\\News\\Service\\FeedServiceV2","type":"->","args":["kate"]},{"file":"/var/www/html/lib/private/AppFramework/Http/Dispatcher.php","line":208,"function":"index","class":"OCA\\News\\Controller\\ItemController","type":"->","args":[2,0,40,0,true,false,""]},{"file":"/var/www/html/lib/private/AppFramework/Http/Dispatcher.php","line":114,"function":"executeController","class":"OC\\AppFramework\\Http\\Dispatcher","type":"->","args":[["OCA\\News\\Controller\\ItemController"],"index"]},{"file":"/var/www/html/lib/private/AppFramework/App.php","line":161,"function":"dispatch","class":"OC\\AppFramework\\Http\\Dispatcher","type":"->","args":[["OCA\\News\\Controller\\ItemController"],"index"]},{"file":"/var/www/html/lib/private/Route/Router.php","line":302,"function":"main","class":"OC\\AppFramework\\App","type":"::","args":["OCA\\News\\Controller\\ItemController","index",["OC\\AppFramework\\DependencyInjection\\DIContainer"],["news.item.index"]]},{"file":"/var/www/html/lib/base.php","line":1001,"function":"match","class":"OC\\Route\\Router","type":"->","args":["/apps/news/items"]},{"file":"/var/www/html/index.php","line":24,"function":"handleRequest","class":"OC","type":"::","args":[]}],"File":"/var/www/html/lib/public/AppFramework/Db/Entity.php","Line":119,"message":"nextUpdateTime is not a valid attribute","exception":{},"CustomMessage":"nextUpdateTime is not a valid attribute"}}
@SMillerDev
Copy link
Contributor

Do you have caching set up in php?

@provokateurin
Copy link
Member Author

Yes, that could indeed be the source of the problem 🤔

I didn't look closer at the error message, but given that this is a PHP syntax error that fixed itself, it must be related to caching.

Sorry for the noise.

@provokateurin provokateurin closed this as not planned Won't fix, can't repro, duplicate, stale Jan 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants