-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Argument 3 passed to OCA\Files\Service\TagService::__construct()
must be an instance of OCP\ITags
#3376
Comments
There's a condition where no server/lib/private/TagManager.php Lines 83 to 86 in fa96b9e
|
#3375 is on the same instance, maybe this is related somehow? But how could this happen? 🤔 |
Well that other error happens on a POST against /login, so there's probably no authenticated user too. |
So we ignore this, until the other issue is resolved? |
I see the same issue in the logs on my NC 11.0.3 |
The main question is still: why is this called at all. Guests can not access any of the routes that share this code path. But after fixing the error with the ITags, it continues with user folder being null, etc. So not sure there is a lot we can do, otherwise we need to fix all DI and make it optional?! |
Shouldn't the milestone be changed? This happens in 12.0.0 as well. |
I saw this in my log today, in nextcloud 12
|
Nope... anyway let me step trough why this happens... |
Ah ok. So it is not that hard to see why actually. We first try to get the Controller at https://github.com/nextcloud/server/blob/master/lib/private/AppFramework/App.php#L91 I feel the 'best' course of action would probably be to run the middleware before we query for the actual controller. However, 'hacking' that in is not trivial. |
JFTR: This exact error just happened this morning around 6:08 CEST on my Nextcloud 13.0.1 instance six times within two minutes. |
Hi,
|
I just had the error pop up as well.
|
Issue is till present on NC 21.0.11
|
OCA\Files\Service\TagService::__construct()
must be an instance of OCP\ITags
present on 22.2.6.2 as well
|
This happens when a request is being sent but the session expired in the meantime or when a guest sends a request to a controller that has no method accessible for guests. Also becomes clearly visible in your log entry. Therefor while annoying as it creates an error in the log, it actually has no negative impact for any real user and can therefor be ignored. |
Hi, please update to at least 23.0.12 and report back if it fixes the issue. Thank you! |
As per above this is still happening. Seeing it at least once a week on our 25 instance |
Sadly, the issue persists on my Nextcloud Hub 3 25.0.1 as well. |
As sad as you are about it, it does not mean anything is broken or not working. It's a guest loading a controller that does not allow guests to do anything. So while the log entry is annoying, nothing impacts the user in the real world |
just updated to NC25.0.2 and I got this problem! Any workaround? |
See comment above your's |
The text was updated successfully, but these errors were encountered: