Menu

#173 DLNA search query not as minidlna expects

Unstable (example)
closed-fixed
None
5
2018-01-19
2017-12-27
Joris_M
No

Hello,

I had a issue with a dlna client not being able to use the search function. I investigated and it turned out that the query was not as minidlna expected it to be. I made a patch and now the search is working.

1 Attachments

Discussion

  • Shrimpkin

    Shrimpkin - 2018-01-01

    Post your client info, profile, user-agent, and debugging logs for the failed search query.

     
  • Joris_M

    Joris_M - 2018-01-01

    I made the change to the code maybe 1 year ago, so I don't have the debug logs any more. The client I was using was the C4 app for ios. Recently I tested the most resent version of minidlna and found the search was still not working, so I applied the patch and after it was working again. I think the problem was with missing quotes. I will try to make a new debug log :)

     
  • Joris_M

    Joris_M - 2018-01-01

    I have created the logfile for the failed query.

     
    • Shrimpkin

      Shrimpkin - 2018-01-04

      Need debug logs. Run in debug mode with -d or set log_level=debug in minidlna.conf.

       
  • Joris_M

    Joris_M - 2018-01-18

    Ok, I have created a new log :)

     
  • Joris_M

    Joris_M - 2018-01-18

    I discovered I made the patch file incorrectly, I have included a new patch file :)

     
  • Justin Maggard

    Justin Maggard - 2018-01-19

    Thanks for posting those logs. Your patch file would have broken SearchCriteria with proper syntax, in order to fix this C4 bug. So I've pushed an alternative change to git master to work around this for your client.

     
  • Justin Maggard

    Justin Maggard - 2018-01-19
    • status: open --> closed-fixed
    • assigned_to: Justin Maggard
     

Log in to post a comment.