<p><a href="https://gerrit.osmocom.org/c/osmo-hlr/+/16204">View Change</a></p><p>4 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-hlr/+/16204/18/contrib/dgsm/esme_dgsm.py">File contrib/dgsm/esme_dgsm.py:</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/osmo-hlr/+/16204/18/contrib/dgsm/esme_dgsm.py@4">Patch Set #18, Line 4:</a> <code style="font-family:monospace,monospace">Copyright 2019 sysmocom s.f.m.c GmbH <info@sysmocom.de></code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">This file is based on code from Rhizomatica, released under AGPL (see comment below in the file).</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">If it's possible legally, as the author of the file this was based on, i'd give explicit permission to apply any less restrictive license.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-hlr/+/16204/18/contrib/dgsm/esme_dgsm.py@42">Patch Set #18, Line 42:</a> <code style="font-family:monospace,monospace">        logging.info('PDU is not a DeliverSM. Is OsmoMSC configured properly?')</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">You may log a lot of Alert Notifications (1 for each LUR) at log level info. These do not necessarily reflect improper configuration, although it is advisable to not bombard this blocking script with them</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-hlr/+/16204/18/contrib/dgsm/esme_dgsm.py@48">Patch Set #18, Line 48:</a> <code style="font-family:monospace,monospace">                     " example, dropping message.")</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">OOoops. There is certainly something wrong in RCCN! There is some unfinished code in there that should not exec, and a python indentation error, but the intention was to only reject certain messages with IEI != SMPP_UDHIEIE_CONCATENATED, these were mostly provider tracking messages or other non-user initiated SMS</p><p style="white-space: pre-wrap; word-wrap: break-word;">Multipart SMS should be handled. Just pass the pdu through. there's no reason not to as the receiving ME will reassemble.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I would delete lines 45-49</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-hlr/+/16204/18/contrib/dgsm/esme_dgsm.py@87">Patch Set #18, Line 87:</a> <code style="font-family:monospace,monospace">        registered_delivery=False,</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">OK, this is a P.O.C. for testing, but why not respect the delivery report request anyway? The receiving osmo-msc should generate the delivery report which should come back through the same routing as this message.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-hlr/+/16204">change 16204</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/c/osmo-hlr/+/16204"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-hlr </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I26e8dd8d9a08187fccb3e74ee91366bc24f6c608 </div>
<div style="display:none"> Gerrit-Change-Number: 16204 </div>
<div style="display:none"> Gerrit-PatchSet: 18 </div>
<div style="display:none"> Gerrit-Owner: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: keith <keith@rhizomatica.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 08 Jan 2020 16:04:45 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Comment-In-Reply-To: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>