<p>Neels Hofmeyr has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/12402">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">enrich context for vlr_subscr_name and ran_conn fi name<br><br>Include all available information in vlr_subscr_name(); instead of either IMSI<br>or MSISDN or TMSI, print all of them when present. Instead of a short log,<br>rather have more valuable context.<br><br>A context info would now look like:<br><br>  Process_Access_Request_VLR(GERAN-A-3_IMSI-901700000014706_MSISDN-2023_TMSI-0x08bde4ec_PAGING_RESP)<br><br>It does get quite long, but ensures easy correlation of any BSSAP / IuCS<br>messages with log output, especially if multiple subscribers are busy at the<br>same time.<br><br>When showing the RAN conn id, instead of<br><br>  GERAN_A-00000017<br><br>write<br><br>  GERAN-A-23<br><br>- We usually write the conn_id in decimal.<br>- Leading zeros are clutter.<br>- Don't mix - and _ separators.<br><br>These changes are combined to produce only one burst of humungous amounts of<br>msc_vlr_tests expected output. That is definitely a weak point of these tests:<br>blowing up the git history. OTOH they do show logging changes nicely.<br><br>Change-Id: I66a68ce2eb8957a35855a3743d91a86299900834<br>---<br>M include/osmocom/msc/ran_conn.h<br>M src/libmsc/gsm_04_08.c<br>M src/libmsc/ran_conn.c<br>M src/libvlr/vlr.c<br>M tests/msc_vlr/msc_vlr_test_authen_reuse.err<br>M tests/msc_vlr/msc_vlr_test_call.err<br>M tests/msc_vlr/msc_vlr_test_gsm_authen.err<br>M tests/msc_vlr/msc_vlr_test_gsm_ciph.err<br>M tests/msc_vlr/msc_vlr_test_hlr_reject.err<br>M tests/msc_vlr/msc_vlr_test_hlr_timeout.err<br>M tests/msc_vlr/msc_vlr_test_ms_timeout.err<br>M tests/msc_vlr/msc_vlr_test_no_authen.err<br>M tests/msc_vlr/msc_vlr_test_reject_concurrency.err<br>M tests/msc_vlr/msc_vlr_test_rest.err<br>M tests/msc_vlr/msc_vlr_test_ss.err<br>M tests/msc_vlr/msc_vlr_test_umts_authen.err<br>16 files changed, 11,382 insertions(+), 11,142 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-msc refs/changes/02/12402/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/12402">change 12402</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/12402"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-msc </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>
<div style="display:none"> Gerrit-Change-Id: I66a68ce2eb8957a35855a3743d91a86299900834 </div>
<div style="display:none"> Gerrit-Change-Number: 12402 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>