-
-
Notifications
You must be signed in to change notification settings - Fork 32
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Client port in UI #122
Comments
Hi @Barracuda09 , I hope you can implement this. It seems easy; and powerful for testing. |
Hi @Barracuda09 , To be more precise: In the "Frontend Info", in the section "Stream Info", when you write the "Owner", the idea is to print not only the IP address of the client, but the pair of RTP UDP ports when streaming as well. Example: Thank you! |
Hi @Barracuda09 , I see the changes in the commit cf539a4. However, after pull the last version and recompile, I don't see this in the web page. Only a new row "Stream URL" appears (even the link is wrong as it every time points to "http://satpi/frontend.html"). However, when I check for the "page source" the new lines for them are here:
You know what happens? |
Hi @lars18th
|
Yes, that's what I expected. However, in my case I only see the rows (but in the source of the page the code it's here):
Please, remember that I've enabled ONLY virtual tuners:
Could this have any influence? |
My Chrome prints this in the console:
Any idea? In the directory where the SATPI lives in
|
The example picture I gave was with Child PIPE. |
I am not sure if this is the problem, as I have it as well |
Here you can get them and try it |
Hi @Barracuda09 , Problem fixed. It's not related to the bootstrap. However, I've updated it. Before close this issue (now fixed/implemented) I request to enlarge a lot the default vertical size of the Frontend Info tab. Then we don't need to use the scroll as now just for a very small points the scroll is activated. |
Hi @Barracuda09 ,
I don't want to request for stupid and non-useful changes. However, some small changes in the UI can help much when doing testing or debug (to the SATPI or to a SAT>IP client).
So, another missing information in the UI is the RTP port used by the client. In the Frontend Overview (and in the detailed view of the FE), the "Owner" is the address of the client. Great! But then can you please add the RTP port too? In fact the target port, as when using the RTSP protocol it will be the RTP (UDP base port), and when using the HTTP protocol is the TCP client port.
With this information will be more easy to know the port for example to launch a simple tcpdump.
You want to implement it, please?
Thank you!
The text was updated successfully, but these errors were encountered: