catalinii / minisatip

minisatip is an SATIP server for linux using local DVB-S2, DVB-C, DVB-T or ATSC cards
https://minisatip.org
327 stars 80 forks source link

Incorrect mappings in UI links of streams #996

Closed lars18th closed 1 year ago

lars18th commented 1 year ago

Hi,

I discovered today a new small bug in the UI: the links provided in the STREAMS colum has incorrect values for FE and SRC when you're using "mappings" of different sources to different adapters. In fact the errors are:

  1. The "fe=" parameter is inserted, even when this value should not be used by clients. So it needs to be removed from the link.
  2. The "src=" parameter not corresponds to the "input" value send by the client, but to the one used to request it to the "adapter". So it doesn't correspond to the correct value.

I report this to fix it at some point. Regards.

Jalle19 commented 1 year ago

I've noticed the same as well

9000h commented 1 year ago

not new :-) https://github.com/catalinii/minisatip/issues/562

lars18th commented 1 year ago

not new :-) #562

Ok. So I'll try to fix it in the future... 😉

catalinii commented 1 year ago

The fe= parameter is now fixed in master. I do not understand the issue with src= can u provide more info/logs ?

lars18th commented 1 year ago

Hi @catalinii ,

I do not understand the issue with src= can u provide more info/logs ?

When we use the option -A --virtual-diseqc mapping_string to configure multiple SAT>IP servers with different orbital positions, then the SRC= value in the link is incorrect. Example:

It's necessary then to fix this bug setting the "src=" value in the link to the "requested" value from the client, and not with the "tunning" value used by the adapter.

I hope it's more clear now. 😉

lars18th commented 1 year ago

Hi @catalinii ,

To fix the SRC bug, I've a question for you:

The question is because we need to store both value:s the value previous to calculate the absolute_table[] mapping, and the target value. Or use the mapping table to search for the target value and obtain the original value. But in any case, it seems that both values requires to be shared with the external symbols of the UI.

What you think?

Jalle19 commented 1 year ago

I would introduce a new symbol

lars18th commented 1 year ago

I would introduce a new symbol

Great! Then you will share ad_pos and ad_req_pos (or any other name)?

Jalle19 commented 1 year ago

Maybe something like ad_mapped_pos

lars18th commented 1 year ago

Fixed. Closed too.