-
Notifications
You must be signed in to change notification settings - Fork 32
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
NC29:Fulltextsearch live crash #375
Comments
Hello, |
I'm seeing this error on two instances immediately after running occ fulltextsearch:reset and freshly reindexing everything. Here is the console output:
|
yes, same here, but fulltextsearch:index seams to work |
Still happening. After a restart of the server I had to reset and recreate the indexes for two of my instances. This seems to affect every NC instance I manage, across multiple servers. |
Complete reset and rebuild the index solves the issue at me, including PDF indexing |
Until after a while it fails again. That's what I am experiencing. |
I'm also seeing that issue occasionally on one of my instances (NC29.0.7). On this specific instance, there's a lot of work done in groupfolders. Some of them using ACL. My assumption is that this is part of the issue (maybe the the live process fails on files which are restricted for some users/groups!?). |
Hello maintainers, what is the root cause of this issue? Is there a stable fix? I applied the code change proposed here nextcloud/fulltextsearch#849 (comment), but it would be great to have a proper solution. Thanks! |
Many months since report and still doesn't fix. |
I'm now able to reproduce the issue on NC30.0.2rc2. It would be great if some of you could double check whether you can reproduce it. Steps:
Expected behavior: Actual behavior: Futher details: Example how
|
I think the real problem is a debug line carelessly trying to access an uninitialized document variable.
I commented out that line and the error is gone... |
Hello, which number of line you are commented - 315?
I think, that is necessery make change in git, becouse after upgrade everything changes will be gone. Line number 315 look as not important for function of plugin. Can you test wraped this line to "if"?
FYI: I'm not programer |
This is as good as the formerly mentioned "workaround". nextcloud/fulltextsearch#849 (comment) This may fix the crash itself but the question remains whether this uninitialized $access variable is really expected or still needed at some other place? |
I also have this issue. |
Issues that I think might arise. Maybe it's fine during indexing but now is missing said attribute when trying to access a file from search. Will test said behavior. |
There is still no confirmation how to reproduce. At least I have provided steps how I can reproduce (#375 (comment)) but so far nobody confirmed or provided other steps how to reproduce. |
Ah I see, I don't have any groups or group folders, it seems to happen under different circumstances as well. In that case further logging of the path that has been attempted to to be indexed might help. I'm not an expert in PHP development though, maybe it's possible by wrapping the return statement in line 754 into a try-catch statement? PS: I'm sorry if I'm not the most helpful commentator. |
Created a test-file with a weird long string inside of it, was able to find and access it through the search while line 757 was commented out. |
Hello, 2 days ago my service for fultextsearch live crash and since this day does not start. When i run occ fulltextsearch:live by hand
./occ fulltextsearch:live
is returning error to stdout:
NC version 29.0.2
The text was updated successfully, but these errors were encountered: