<p><a href="https://gerrit.osmocom.org/12302">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/12302/1/src/libmsc/msc_vty.c">File src/libmsc/msc_vty.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/12302/1/src/libmsc/msc_vty.c@437">Patch Set #1, Line 437:</a> <code style="font-family:monospace,monospace">      "Send each IMEI to the EIR to ask if it is permitted or not. The EIR is implemented as part of osmo-hlr, "</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">slightly out-of-scope, we don't know whether the user is using OsmoHLR or something else.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/12302/1/src/libmsc/msc_vty.c@440">Patch Set #1, Line 440:</a> <code style="font-family:monospace,monospace">      "1 = send each IMEI to the EIR\n")</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">(we normally don't name the values again, just write the string for that value. online vty doc will print the value and add your string to it)</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/12302/1/src/libmsc/msc_vty.c@448">Patch Set #1, Line 448:</a> <code style="font-family:monospace,monospace">""</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">It might be helpful to understand what "early" means in here. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">IIUC this patch doesn't require adding IMEISV config, so maybe we can push this off into the future instead?</p><p style="white-space: pre-wrap; word-wrap: break-word;">Some explanation:</p><p style="white-space: pre-wrap; word-wrap: break-word;">IMEISV can be</p><pre style="font-family: monospace,monospace; white-space: pre-wrap;"> * not requested;<br> * requested as an explicit Identity Request roundtrip;<br> * requested as piggy-back on the Ciphering Mode Complete, partly to save roundtrips, but actually more importantly to not leak the identity in an unencrypted message, because above ID Request typically happens before ciphering.</pre><p style="white-space: pre-wrap; word-wrap: break-word;">So the IMEISV-Early says "We require the IMEISV, and ask for it in the beginning via ID Request".<br>And the IMEISV-ciphered says "We require IMEISV, but wait until after ciphering".</p><p style="white-space: pre-wrap; word-wrap: break-word;">I think to the user, it could make sense to offer:</p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">  retrieve-imeisv (0|1|early)<br>  0  Do not request an IMEISV<br>  1  Request an IMEISV. With encryption enabled, this will be requested during Ciphering Mode negotiation.<br>     Without encryption, this will cause a separate Identity Request.<br>  early  Request an IMEISV, always use a separate, unencrypted Identity Request even if encryption is enabled.</pre><p style="white-space: pre-wrap; word-wrap: break-word;">Not sure about:</p><ul><li>do we even need the 'early' option?</li><li>I can imagine requesting IMEISV could be an implicit internal process instead of a user config, and we only have these flags in vlr because we haven't implemented the code that would require an IMEISV yet? (the SV, software version, does indicate some traits but I forget what it was)</li></ul></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/12302">change 12302</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/12302"/><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: comment </div>
<div style="display:none"> Gerrit-Change-Id: Iee516b9cd7877b21207ce9a6d954109f19558163 </div>
<div style="display:none"> Gerrit-Change-Number: 12302 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Reviewer: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: Max <msuraev@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: Stefan Sperling <stsp@stsp.name> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 17 Dec 2018 15:27:11 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>