Opened 8 years ago

Last modified 3 years ago

#793 new defect

Search: Downloading containing foler doesn't work from my shares

Reported by: artemv <abublic@…> Owned by: quinox
Priority: normal Milestone: Release 1.3.0
Component: nicotine Version: 1.2.16
Keywords: Cc:

Description

Remote users complain that they cannot download folders from me if they do it via search results. Downloading single files works.

Tried it myself, confirmed: downloading folder from search failed, and it worked from browsing my files. When I try to download folder from search I get an exception in console, will put it into comment. The folder files don't appear in Downloads tab. Ubuntu 11.04, Nicotine+ 1.2.16

Change History (5)

comment:1 Changed 8 years ago by artemv <abublic@…>

When I try to download folder from search I get an exception in console:

13:32:06 Exception during parsing first <pynicotine.slskmessages.FolderContentsResponse instance at 0xb410a0c>: unpack requires a string argument of length 4
Exception during parsing second <pynicotine.slskmessages.FolderContentsResponse instance at 0xb410a0c>: unpack requires a string argument of length 4
Offending package: "\x01\x00\x00\x00V\x00\x00\x00\\media\\TRANSCEND\\music\\foreign\\sasha suchkov\\Lifescapes\\Native American Flute & Guitar\x01\x00\x00\x00V\x00\x00\x00\\media\\TRANSCEND\\music\\foreign\\sasha suchkov\\Lifescapes\\Native American Flute & Guitar\x0f\x00\x00\x00\x01!\x00\x00\x0001 Streams Of Silver And Gold.mp3\xb4\xba\x9d\x00\x00\x00\x00\x00\x00\x00\x00\x00\x03\x00\x00\x00mp3\x03\x00\x00\x00\x00\x00\x00\x00@\x01\x00\x00\x01\x00\x00\x00\x02\x01\x00\x00\x02\x00\x00\x00\x00\x00\x00\x00\x01\x19\x00\x00\x0002 Sheepherder's Pass.mp3\xd02\xa0\x00\x00\x00\x00\x00\x00\x00\x00\x00\x03\x00\x00\x00mp3\x03\x00\x00\x00\x00\x00\x00\x00@\x01\x00\x00\x01\x00\x00\x00\x06\x01\x00\x00\x02\x00\x00\x00\x00\x00\x00\x00\x01\x14\x00\x00\x0003 Absaroka Wind.mp3\xa4}\x87\x00\x00\x00\x00\x00\x00\x00\x00\x00\x03\x00\x00\x00mp3\x03\x00\x00\x00\x00\x00\x00\x00@\x01\x00\x00\x01\x00\x00\x00\xde\x00\x00\x00\x02\x00\x00\x00\x00\x00\x00\x00\x01\x1c\x00\x00\x0004 Bitterroot Rendezvous.mp3\xbc\x1a\x9c\x00\x00\x00\x00\x00\x00\x00\x00\x00\x03\x00\x00\x00mp3\x03\x00\x00\x00\x00\x00\x00\x00@\x01\x00\x00\x01\x00\x00\x00\xff\x00\x00\x00\x02\x00\x00\x00\x00\x00\x00\x00\x01\x17\x00\x00\x0005 Dance In The Sky.mp3\xdc\xcc\x9f\x00\x00\x00\x00\x00\x00\x00\x00\x00\x03\x00\x00\x00mp3\x03\x00\x00\x00\x00\x00\x00\x00@\x01\x00\x00\x01\x00\x00\x00\x05\x01\x00\x00\x02\x00\x00\x00\x00\x00\x00\x00\x01\x1b\x00\x00\x0006 Headwater's Crossing.mp3\x9c|\x9c\x00\x00\x00\x00\x00\x00\x00\x00\x00\x03\x00\x00\x00mp3\x03\x00\x00\x00\x00\x00\x00\x00@\x01\x00\x00\x01\x00\x00\x00\x00\x01\x00\x00\x02\x00\x00\x00\x00\x00\x00\x00\x01\x19\x00\x00\x0007 Beatiful Josephine.mp3\xc8@\xb8\x00\x00\x00\x00\x00\x00\x00\x00\x00\x03\x00\x00\x00mp3\x03\x00\x00\x00\x00\x00\x00\x00@\x01\x00\x00\x01\x00\x00\x00.\x01\x00\x00\x02\x00\x00\x00\x00\x00\x00\x00\x01 \x00\x00\x0008 Before I Leave This Place.mp3` \x8c\x00\x00\x00\x00\x00\x00\x00\x00\x00\x03\x00\x00\x00mp3\x03\x00\x00\x00\x00\x00\x00\x00@\x01\x00\x00\x01\x00\x00\x00\xe5\x00\x00\x00\x02\x00\x00\x00\x00\x00\x00\x00\x01\x13\x00\x00\x0009 Open Prairie.mp3\xdc\xd6\xa1\x00\x00\x00\x00\x00\x00\x00\x00\x00\x03\x00\x00\x00mp3\x03\x00\x00\x00\x00\x00\x00\x00@\x01\x00\x00\x01\x00\x00\x00\t\x01\x00\x00\x02\x00\x00\x00\x00\x00\x00\x00\x01\x1a\x00\x00\x0010 When The Ice Breaks.mp3\x00\x1d\xb9\x00\x00\x00\x00\x00\x00\x00\x00\x00\x03\x00\x00\x00mp3\x03\x00\x00\x00\x00\x00\x00\x00@\x01\x00\x00\x01\x00\x00\x00/\x01\x00\x00\x02\x00\x00\x00\x00\x00\x00\x00\x01\x14\x00\x00\x0011 Sweet Mercies.mp3h\x8e\xc1\x00\x00"

comment:2 Changed 8 years ago by artemv <abublic@…>

Note that this is *not* about downloading folders from other users - this works fine for me.

comment:3 Changed 8 years ago by artemv <abublic@…>

Remote user confirmed that he can download folders via browsing my files (but not via search)

comment:4 Changed 8 years ago by anonymous

Thank you for your report.

It recently came to my attention that there is a serious bug in the official SLSK client: when it received a reply on a search query it destroys the filenames by casting everything to lower casing, EG. "/Music/Bladibla?.mp3" becomes "/music/bladibla.mp3". When it then tries to download these files from N+ it gets a "File not found" reply since those file don't exist. When the official client browses someone this behaviour does not appear, resulting in the situation you describe.

I've implemented a workaround with r1488 to counter this bug - it tries to recreate the letter casing. So if you run SVN you won't have this problem any longer (I don't know when we release the next version of N+).

comment:5 Changed 3 years ago by gfarmerfr

Milestone: Release 1.2.17Release 1.3.0
Note: See TracTickets for help on using tickets.