<p style="white-space: pre-wrap; word-wrap: break-word;">Note: *not* CallID, just *Connection ID*.<br>CallID is the one that the MGCP clients send to the MGW to indicate which call reference is served, like an MNCC call reference.</p><p style="white-space: pre-wrap; word-wrap: break-word;">The Connection ID in contrast only is the name of one connection on an endpoint, sort of like a fancy array index. Osmo-MGW invents these freely, and the clients only ever echo them back later to reference the particular "half call leg" on an endpoint.</p><p style="white-space: pre-wrap; word-wrap: break-word;">For example, osmo-bsc does a CRCX on 1@mgw.<br>osmo-mgw replies with an OK, including "I: 1234abcd".<br>osmo-bsc remembers ths 1234abcd as the MGCP referene to pass for MDCX or DLCX later.</p><p style="white-space: pre-wrap; word-wrap: break-word;">The proper way would be to refactor osmo-mgw to have a maximum connection id length and make it configurable from vty what length is actually dealt out. But in fact, the length of the Conn ID is entirely and solely up to osmo-mgw to determine. If we set one specific conn ID to be generated, they will *always* be that length, because no other entity besides osmo-mgw is allowed to create these IDs.</p><p style="white-space: pre-wrap; word-wrap: break-word;">So we cannot modify libosmo-mgcp-client to create shorter IDs, because osmo-mgw creates these IDs. libosmo-mgcp-client should still be able to receive and handle any length of Connection ID it receives from osmo-mgw, we're just making osmo-mgw hand out shorter ones.</p><p style="white-space: pre-wrap; word-wrap: break-word;">But true, if a client sends a Connection ID that is too long, we should make sure to mismatch and not truncate that when parsing the incoming message. I think that's the case, but let me verify.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Does this patch make more sense now?</p><p><a href="https://gerrit.osmocom.org/10676">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/10676">change 10676</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/10676"/><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: Ia290c22a91fca0e5aa44515fca6df00064aff100 </div>
<div style="display:none"> Gerrit-Change-Number: 10676 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </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: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 30 Aug 2018 14:19:22 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-HasLabels: No </div>