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

constant log flooding - Cannot retrieve value of undefined directive Core.RemoveBlanks #2883

Open
FadeFx opened this issue Nov 16, 2024 · 16 comments

Comments

@FadeFx
Copy link

FadeFx commented Nov 16, 2024

IMPORTANT

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

  • [x ] I have read the CONTRIBUTING.md and followed the provided tips
  • [x ] I accept that the issue will be closed without comment if I do not check here
  • [x ] 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

no visual issue, but constant log flooding with
{"reqId":"S9A6V7FoXrzh4bT1hs1U","level":2,"time":"2024-11-16T08:50:24+00:00","remoteAddr":"","user":"--","app":"PHP","method":"","url":"--","message":"Cannot retrieve value of undefined directive Core.RemoveBlanks invoked on line 75 in file /var/www/nextcloud/apps/news/vendor/ezyang/htmlpurifier/library/HTMLPurifier/Lexer/DOMLex.php at /var/www/nextcloud/apps/news/vendor/ezyang/htmlpurifier/library/HTMLPurifier/Config.php#901","userAgent":"--","version":"30.0.2.2","data":{"app":"PHP"},"id":"67385db8a6022"}

Steps to Reproduce

install News Alpha 14

System Information

  • News app version: Alpha 14
  • Nextcloud version:30.0.2 (NCP)
  • Cron type: sy.stem cron
  • PHP version: 8.3
  • Database and version: 10.11.6-MariaDB-0+deb12u1
  • Browser and version: any
  • OS and version: Debian 12
{"reqId":"S9A6V7FoXrzh4bT1hs1U","level":2,"time":"2024-11-16T08:50:24+00:00","remoteAddr":"","user":"--","app":"PHP","method":"","url":"--","message":"Cannot retrieve value of undefined directive Core.RemoveBlanks invoked on line 75 in file /var/www/nextcloud/apps/news/vendor/ezyang/htmlpurifier/library/HTMLPurifier/Lexer/DOMLex.php at /var/www/nextcloud/apps/news/vendor/ezyang/htmlpurifier/library/HTMLPurifier/Config.php#901","userAgent":"--","version":"30.0.2.2","data":{"app":"PHP"},"id":"67385db8a6022"}
Paste output here
nothing specific, News is actually working Read http://ggnome.com/wiki/Using_The_Browser_Error_Console if you are unsure what to put here
Paste output here
@FadeFx FadeFx added the bug label Nov 16, 2024
@FadeFx
Copy link
Author

FadeFx commented Nov 16, 2024

Sorry for not filling the form previously...

@SMillerDev
Copy link
Contributor

Can you also make a report to https://github.com/ezyang/htmlpurifier? Because that doesn't seem like an issue in news.

@FadeFx
Copy link
Author

FadeFx commented Nov 17, 2024

Did so: ezyang/htmlpurifier#428

@FadeFx
Copy link
Author

FadeFx commented Nov 19, 2024

@SMillerDev is HTML Purifyer included with news somehow, or is it installed in the os? I am asked to try and reistall it, but I did not install this in the first place. This could however also be due to it being included in nextcloudpi...

@Grotax
Copy link
Member

Grotax commented Nov 19, 2024

It is shipped with news as a dependency hence the path that includes news.

But I have no idea why you see that in your logs.

@SMillerDev
Copy link
Contributor

It's part of the news app. Maybe the v25 release of today will help?

@Grotax
Copy link
Member

Grotax commented Nov 19, 2024

@SMillerDev just checked my logs I also see these warnings and I'm running 25.0.0

@FadeFx
Copy link
Author

FadeFx commented Nov 19, 2024

Updated to 25.0.0 and it keeps logging

@FadeFx
Copy link
Author

FadeFx commented Nov 20, 2024

@FadeFx
Copy link
Author

FadeFx commented Nov 20, 2024

Actually it is very likely something you imported with the Alpha14, as the issue was not there before.

@wofferl
Copy link
Collaborator

wofferl commented Nov 20, 2024

It was updated in alpha13 (791ea20) from v4.17.0 to 4.18.0 which added this as new feature:

add directive for removing blank nodes (ezyang/htmlpurifier#404) (c9d60c9)

@marlemion

This comment has been minimized.

@FadeFx
Copy link
Author

FadeFx commented Nov 20, 2024

It was updated in alpha13 (791ea20) from v4.17.0 to 4.18.0 which added this as new feature:

add directive for removing blank nodes (ezyang/htmlpurifier#404) (c9d60c9)

Probably, I think I jumped alpha 13

@r-vit

This comment has been minimized.

@pbek
Copy link
Member

pbek commented Nov 24, 2024

Should we revert 791ea20 meanwhile?

@FadeFx
Copy link
Author

FadeFx commented Nov 29, 2024

I think this can be closed, as the downgrade removed the issue, however, it should be keept in sight for next upgrade attempt...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants