-
Notifications
You must be signed in to change notification settings - Fork 112
Fix bug preventing full directory from being fetched while reviewing search results #1324
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
Conversation
|
Version: 0.22.4 @jpdillingham this doesn't appear to fix the issue for me of not being able to fetch directory contents. Before this merge there was an error visible in the log: Now this error doesn't appear anymore with the new release, but the webinterface shows this error dialog instead: The directory contents are then not retrieved. |
|
@litemotiv can you try doing a hard refresh on your browser? shift+f5. You might have old application code cached. If that doesn't fix it can you please open the console (f12) and screenshot/paste any errors you see? |
|
@jpdillingham the error appears to persist after hard refreshing and restarting the browser (Firefox 137). This is the output in the console: |
|
I added some additional error handling that should solve the exact problem you're having, but I'm somewhat concerned that we're getting no results back when requesting a directory. I don't know of any clients except perhaps early builds of slskd that would do that. #1328 is going to go out to the canary build in 20-30 minutes, I'd appreciate if you could pull that and see if that changes anything. Are you getting this for a specific user, or all users? |
|
Here's the build: https://github.com/slskd/slskd/actions/runs/14433060140. The canary image will be pushed to docker hub; if you're running from the binaries you can get them out of the artifacts from this build (unless that's restricted to maintainers only) |
The only client I'm aware of is Soulseek NS, which doesn't understand paths encoded to UTF-8 sometimes. It will send an empty response until you encode the folder path to latin-1 (ISO-8859-1): https://github.com/nicotine-plus/nicotine-plus/blob/729710cc9a1e5169c868435a3f9073c16f13ef2c/pynicotine/downloads.py#L1001-L1005 Edit: Another case is SoulseekQt, which doesn't send any response at all for folder content requests on private folders. Might not be related to the issue, but it's useful for future reference. |
|
Update: directory fetching works well for me using the canary build |
|
Thanks for the confirmation @litemotiv; 0.22.5 is on its way now |
|
I still get this error with 0.22.5 when pressing "Get full directory contents". I've done a hard reset of Firefox. |
|
@MitraMai you're getting the If so could you please enable debugging and try fetching the same directory from the same user, and see if any errors are shown? |
Soulseek.NET v7 changed the return type of the underlying method to return multiple directories, and in doing so created a bug. That bug was fixed with the upgrade to 7.0.3.
Additionally, I never adjusted the related logic in slskd to handle multiple directories. This PR does that, taking the first directory only.
I might update things later to include subdirectories, but because this only works with Soulseek NS (Qt and Nicotine+ don't send more than 1 directory, nor does slskd) it's not a high priority.