Attention is currently required from: dexter, keith, osmith.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-mgw/+/34150?usp=email )
Change subject: vty: include local port when dumping RTP conns ......................................................................
Patch Set 2:
(1 comment)
File src/libosmo-mgcp/mgcp_conn.c:
https://gerrit.osmocom.org/c/osmo-mgw/+/34150/comment/83bcd00e_e5ce91aa PS1, Line 373: "rtp:%u<->%u rtcp:%u)",
Seeing as how we are not specifying RTCP port in the SDP, that is we adhere to RFC1889 [the correspo […]
I'm totally fine with dropping the rtcp part. In any case, please add the "l:<->r:" prefixes as I suggested, otherwise it becomes confusing trying to figure out which IP is the remote and which the local one. It may seem obvious if you are the operator, but usually the case is that someone else is looking at the logs without direct access or knowledge of the network.