<p><a href="https://gerrit.osmocom.org/12357">View Change</a></p><p>3 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/12357/3//COMMIT_MSG">Commit Message:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/12357/3//COMMIT_MSG@20">Patch Set #3, Line 20:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;">Rationale: reading pcaps becomes so much easier when each of osmo-bsc and<br>osmo-msc address their MGW with differing domain names. Otherwise, both will<br>have a '0@mgw' endpoint and it gets really confusing.<br></pre></blockquote></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">How cold both use the same endpoint?  The endpoint names (local part in front of @) is allocated by  […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">In a scenario where both osmo-msc and osmo-bsc use the same osmo-mgw, indeed each new endpoint will get a new number. But when you have one osmo-mgw each, then both osmo-mgw instances independently create endpoints, so for the first call occuring, both actually create a '0@mgw' and wireshark shows identically named endpoints on separate osmo-mgw instances. Then there is the need to also look at the source/dest address and/or port numbers to figure out what is going on.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/12357/3/include/osmocom/mgcp_client/mgcp_client.h">File include/osmocom/mgcp_client/mgcp_client.h:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/12357/3/include/osmocom/mgcp_client/mgcp_client.h@28">Patch Set #3, Line 28:</a> <code style="font-family:monospace,monospace">MG</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">why 64? Is there some kind of maximum DNS domain name length we should use?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I'm not sure at all. In fact I was just picking something at random and not bothering with details, just to quickly use this at 35c3. Same with other parts of this patch. I'm glad code review worked nicely here :)</p><p style="white-space: pre-wrap; word-wrap: break-word;">Found MGCP_ENDPOINT_MAXLEN, though that would include the "123@" part, it is a good rough choice I guess.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/12357/3/src/libosmo-mgcp-client/mgcp_client.c">File src/libosmo-mgcp-client/mgcp_client.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/12357/3/src/libosmo-mgcp-client/mgcp_client.c@830">Patch Set #3, Line 830:</a> <code style="font-family:monospace,monospace"></code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Oh, wow, this breaks Gerrit's syntax highlighting :D</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">yeah, I'm also getting C comment warnings when trying to write 'rtpbridge/*@foo' in a comment :P</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/12357">change 12357</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/12357"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-mgw </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Ia662016f29dd8727d9c4626d726729641e21e1f8 </div>
<div style="display:none"> Gerrit-Change-Number: 12357 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </div>
<div style="display:none"> Gerrit-Owner: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Reviewer: Max <msuraev@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: Stefan Sperling <stsp@stsp.name> </div>
<div style="display:none"> Gerrit-CC: Vadim Yanitskiy <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 03 Jan 2019 01:20:31 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>