Project

General

Profile

Bug #743

Can't open FLAC m3u/pls playlists with unescaped URI to http file server

Added by konkor konkor over 6 years ago. Updated over 6 years ago.

Status:
Rejected
Priority:
Trivial
Assignee:
-
Category:
-
Target version:
-
Start date:
September 11, 2017
Due date:
% Done:

0%

Estimated time:
Affects version:

History

#1 Updated by konkor konkor over 6 years ago

So when it opened from 'File->Open URL' it's playing well.

#2 Updated by John Lindgren over 6 years ago

Please upload the output of "audacious -V" when playing the file from Open URL and when trying to play it from the .m3u file.

Also please upload the .m3u file as an attachment.

Also test to see if the problem still occurs with the latest version.

#3 Updated by John Lindgren over 6 years ago

Also, the title says "unescaped URI" but the example you gave is escaped. Which is it?

#4 Updated by konkor konkor over 6 years ago

John Lindgren wrote:

Also, the title says "unescaped URI" but the example you gave is escaped. Which is it?

My bad sorry... It's escaped for sure. Here is some output

ERROR neon.cc:573 [open_handle]: <0x7f552802b970> Could not parse URL 'http://192.168.1.7:8088/10/Lindsey%20Stirling%20-%20Lindsey%20Stirling%20[Target%20Deluxe%20Edition]%20(2013)%20[FLAC]/09%20Shadows.flac'
ERROR neon.cc:730 [fopen]: <0x7f552802b970> Could not open URL
ERROR util.cc:160 [audgui_simple_message]: Error playing http://192.168.1.7:8088/10/Lindsey%20Stirling%20-%20Lindsey%20Stirling%20[Target%20Deluxe%20Edition]%20(2013)%20[FLAC]/09%20Shadows.flac:
Error parsing URL

So URLs without escaped symbols plays fine but playlist with mp3 files plays well.

PS: Is it possible somehow optimize tag reading? Can you read tags from the one request and don't spam all the time server with 10 request per song? And on hover over songs the player sends this request again and again even for current song... (I know I can disable songs popups but some cache for it would be nice too)

#5 Updated by John Lindgren over 6 years ago

"Error parsing URL" is interesting; it means that ne_uri_parse() didn't like the URL we gave it.

Please upload the .m3u file as an attachment.

Also test to see if the problem still occurs with the latest version.

#9 Updated by John Lindgren over 6 years ago

  • Status changed from New to Rejected

Okay, the problem is the square brackets in the URL. You need to encode those (i.e. '%5b' for '[' and '%5b' for ']'). See here:
https://stackoverflow.com/questions/40568/are-square-brackets-permitted-in-urls/1016737#1016737

If you got that broken URL from some other piece of software (for example, a file server) you should report a bug to its developers.

In this case, a working URL would be:
http://192.168.1.7:8088/10/Lindsey%20Stirling%20-%20Lindsey%20Stirling%20%5bTarget%20Deluxe%20Edition%5d%20(2013)%20%5bFLAC%5d/01%20Electric%20Daisy%20Violin.mp3

#10 Updated by konkor konkor over 6 years ago

I'm sorry for my bug. Thank you very much for your time and attention.
I'll recommend your player to all ;)

Also available in: Atom PDF