Skip to content
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

Open
lars18th opened this issue Mar 7, 2021 · 11 comments
Open

Client port in UI #122

lars18th opened this issue Mar 7, 2021 · 11 comments
Labels
Fixed Testers Wanted If like to help out, please test this

Comments

@lars18th
Copy link
Contributor

lars18th commented Mar 7, 2021

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!

@Barracuda09 Barracuda09 added enhancement good first issue Please feel free to ask me for pointers labels Mar 9, 2021
@lars18th
Copy link
Contributor Author

lars18th commented Apr 8, 2021

Hi @Barracuda09 ,

I hope you can implement this. It seems easy; and powerful for testing.
So, please can you add the TARGET PORT of the client in the UI?
Thank you!

@lars18th
Copy link
Contributor Author

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: Owner: 192.168.1.51:43002-43003.

Thank you!

@Barracuda09 Barracuda09 added Fixed Testers Wanted If like to help out, please test this and removed enhancement good first issue Please feel free to ask me for pointers labels May 10, 2021
@lars18th
Copy link
Contributor Author

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:

page += addTableLineEntry("RTP Port", xmlDoc, streamID + "rtpPort");
page += addTableLineEntry("RTCP Port", xmlDoc, streamID + "rtcpPort");
page += addTableLineEntry("HTTP Port", xmlDoc, streamID + "httpPort");

You know what happens?

@Barracuda09
Copy link
Owner

Hi @lars18th

  • The "Stream URL" only is implemented for now, when using DVB-C/S2. So not for Child-PIPE/Streamer inputs.
  • I am not sure why you don't see the lines, here is how I see it:

image

@lars18th
Copy link
Contributor Author

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):

  • Stream URL
  • Owner
  • Session ID
  • User-Agent
  • RTP packet count
  • RTP streamed (MB)

Please, remember that I've enabled ONLY virtual tuners:

  • Frontend 0: TS Reader
  • Frontend 1: Streamer
  • Frontend 2: Child PIPE - TS Reader
  • Frontend 3: Child PIPE - TS Reader

Could this have any influence?

@lars18th
Copy link
Contributor Author

lars18th commented May 15, 2021

My Chrome prints this in the console:

DevTools failed to load SourceMap: Could not load content for http://192.168.1.35:8080/assets/js/bootstrap.min.js.map: HTTP error: status code 404, net::ERR_HTTP_RESPONSE_CODE_FAILURE
DevTools failed to load SourceMap: Could not load content for http://192.168.1.35:8080/assets/css/bootstrap.min.css.map: HTTP error: status code 404, net::ERR_HTTP_RESPONSE_CODE_FAILURE

Any idea?

In the directory where the SATPI lives in web/assets/js the ls -l:

-rw-r--r-- 1 root root  6081 May 15 18:09 addtableentry.js
-rw-r--r-- 1 root root 63240 Feb 22 13:42 bootstrap.min.js
-rw-r--r-- 1 root root   801 Feb 22 13:42 cookies.js
-rw-r--r-- 1 root root 89476 Feb 22 13:42 jquery.min.js
-rwxr-xr-x 1 root root  1051 Feb 22 13:42 loadxmldoc.js
-rw-r--r-- 1 root root  2199 Feb 22 13:42 menu.js
-rw-r--r-- 1 root root   740 Feb 22 13:42 postxmldoc.js
-rw-r--r-- 1 root root  1111 May 15 18:09 utils.js

@Barracuda09
Copy link
Owner

The example picture I gave was with Child PIPE.

@Barracuda09
Copy link
Owner

My Chrome prints this in the console:

I am not sure if this is the problem, as I have it as well

@Barracuda09
Copy link
Owner

@lars18th
Copy link
Contributor Author

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.

@Barracuda09
Copy link
Owner

@lars18th

Yes, or maybe just have Bootstrap resize it (But I am not an expert in Web Development/Design I have managed to get this far, and with help from @issue )

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fixed Testers Wanted If like to help out, please test this
Projects
None yet
Development

No branches or pull requests

2 participants