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

Best settings for DSub? #13

Closed
sardaukar opened this issue Nov 25, 2019 · 16 comments
Closed

Best settings for DSub? #13

sardaukar opened this issue Nov 25, 2019 · 16 comments
Assignees
Labels

Comments

@sardaukar
Copy link

Anyone seen this issue before?

photo5972136799885176800

I get it navigating to some folders in my Library view. On the gonic logs, I just see a successful request

/rest/getMusicDirectory.view?u=admin&p=enc:<OMITTED>&v=1.2.0&c=DSub&id=8330

@sentriz
Copy link
Owner

sentriz commented Nov 25, 2019

Hmm I have no idea what that is. Could you show the output when curling the above link?

@sardaukar
Copy link
Author

output.xml.zip

I had to add -L to curl to follow the initial 301

@sentriz
Copy link
Owner

sentriz commented Nov 25, 2019

still having trouble reproducing. what version of dsub are you running?

@sardaukar
Copy link
Author

The latest on the Play Store (at least for me) is 5.4.4

@sentriz
Copy link
Owner

sentriz commented Nov 25, 2019

hmmm. did that folder work for your old subsonic server? and if so would it be possible to show the curl of that so I could compare the two responses? thank you

@sardaukar
Copy link
Author

Let me just try to replicate the curl.

@sardaukar
Copy link
Author

Here's with Airsonic (my Subsonic fork, that works 100% with DSub)

output_airsonic.zip

@sardaukar
Copy link
Author

Looks like gonics timestamps are always 0001-01-01T00:00:00Z which might trip DSub up

@sentriz
Copy link
Owner

sentriz commented Nov 26, 2019

thank you for that output. will be very helpful for debugging. I just pushed an update (it's building on dockerhub right now)
it always returns isDir/isVideo correctly for getMusicDirectory (since dsub is giving you errors about bools) and also returns timestamps correctly as you said.

would you mind trying this one out?

@sardaukar
Copy link
Author

Will try!

@sardaukar
Copy link
Author

Ok so still have the same DSub error :( here's another dump of 2 directories, one working the other not (the same as the original XML error dump)
outputs.zip

@sentriz
Copy link
Owner

sentriz commented Nov 26, 2019

strange. can you try one more thing?

install the latest dsub from github (I remember a difference in the play store and github versions)

https://github.com/daneren2005/Subsonic/releases/download/5.4.4/DSub.5.4.4.apk

@sardaukar
Copy link
Author

Same thing on this version from Github. Something about this folder's output makes it crash out :( I can give you an account on my server to test it out if you'd like.

@sentriz
Copy link
Owner

sentriz commented Nov 27, 2019

ok thanks for trying and the outputs. going to try diffxml now to see what might differ between the servers

@sentriz sentriz mentioned this issue Dec 5, 2019
@sentriz sentriz self-assigned this Jan 21, 2020
@sentriz sentriz added the bug label Jan 21, 2020
@sentriz
Copy link
Owner

sentriz commented Jan 21, 2020

hey would you minding sending me an email if you're still interesting in this? i have a different idea. thanks!

(link on profile)

@sardaukar
Copy link
Author

Thanks for sticking with this - seems to work fine now! :D

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