hang while trying to import .PLS #466

Open
opened 2023-05-11 23:42:05 +00:00 by yuv · 1 comment

Describe the bug
Description of what the bug is.

To Reproduce
Steps to reproduce the behavior:

  1. Go to 'initial screen'
  2. Tap on '+ Add new station'
  3. enter 'https://playerservices.streamtheworld.com/pls/XHMFM.pls'
  4. A moving bar appears and the application seems to hang in a loop

Expected behavior
XHMFM added as a single radio station. see also comment on bug #453

Device:

  • Irrelevant, tested on multiple devices and networks

Additional context

The PLS file is a single radio station. Streamtheworld.com is a very widely used service. Their PLS lists a number of dynamically allocated servers, each of which plays the same radio stream. I believe they scale up/down those servers based on number of listeners / traffic, so Transistor should pull the playlist every time the user tries to tune-in and select one of the servers for streaming, but not save that one server (e.g. https://24923.live.streamtheworld.com:443/XHMFM_SC) because it can and does change (see also comment on bug #453). It would be nice if there was some automated mechanism to determine which of the listed servers is the least busy (ping?) and to switch to another server when the current broadcast goes wrong.

**Describe the bug** Description of what the bug is. **To Reproduce** Steps to reproduce the behavior: 1. Go to 'initial screen' 2. Tap on '+ Add new station' 3. enter 'https://playerservices.streamtheworld.com/pls/XHMFM.pls' 4. A moving bar appears and the application seems to hang in a loop **Expected behavior** XHMFM added as a single radio station. see also comment on bug #453 **Device:** - Irrelevant, tested on multiple devices and networks **Additional context** The PLS file is a single radio station. Streamtheworld.com is a very widely used service. Their PLS lists a number of *dynamically* allocated servers, each of which plays the same radio stream. I believe they scale up/down those servers based on number of listeners / traffic, so Transistor should pull the playlist every time the user tries to tune-in and select one of the servers for streaming, but not save that one server (e.g. https://24923.live.streamtheworld.com:443/XHMFM_SC) because it can and does change (see also comment on bug #453). It would be nice if there was some automated mechanism to determine which of the listed servers is the least busy (ping?) and to switch to another server when the current broadcast goes wrong.
yuv added the
bug
label 2023-05-11 23:42:05 +00:00

Hi @yuv

I am currently reworking the "Add Station" workflow (see a647327d73). Since I do not have a lot of time at the moment this rework will probably take some time.

But I will definitively test the new mechanism on how it handles stations that are manually added via PLS. And I am glad to have a PLS with multiple embedded streams to test. 👍

Transistor should pull the playlist every time the user tries to tune-in and select one of the servers for streaming, but not save that one server

I understand what you are suggesting, but that would be actually a pretty big change to the way Transistor stores radio stations at rest and how it handles stations internally. (Transistor just stores the actual streaming links.) Since I am notoriously short on time I have no plans to implement such a change. Sorry. I hope you are not too disappointed.

Hi @yuv I am currently reworking the "Add Station" workflow (see a647327d73). Since I do not have a lot of time at the moment this rework will probably take some time. But I will definitively test the new mechanism on how it handles stations that are manually added via PLS. And I am glad to have a PLS with multiple embedded streams to test. 👍 > Transistor should pull the playlist every time the user tries to tune-in and select one of the servers for streaming, but not save that one server I understand what you are suggesting, but that would be actually a pretty big change to the way Transistor stores radio stations at rest and how it handles stations internally. (Transistor just stores the actual streaming links.) Since I am notoriously short on time I have no plans to implement such a change. Sorry. I hope you are not too disappointed.
y20k self-assigned this 2023-06-01 12:26:37 +00:00
Sign in to join this conversation.
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: y20k/transistor#466
There is no content yet.