<p> Attention is currently required from: Joshua Colp, George Joseph, Kevin Harwell. </p>
<p>Patch set 2:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4; color: #000000;">Code-Review -1</span></p><p><a href="https://gerrit.asterisk.org/c/asterisk/+/18360">View Change</a></p><p>8 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="null">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.asterisk.org/c/asterisk/+/18360/comment/2913f75b_4fd752d5">Patch Set #2, Line 10:</a> <code style="font-family:monospace,monospace">since I'm going to be offline for a bit.</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Is this commit message still accurate? Either way I feel the last sentence should be replaced with something more substantive about the commit itself.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File res/res_geolocation/wiki/AsteriskImplementation.md:</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.asterisk.org/c/asterisk/+/18360/comment/18580c3a_e3d95900">Patch Set #2, Line 65:</a> <code style="font-family:monospace,monospace">See [RFC6442|Geolocation#rfc6442] for more informatrion.|</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">information</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/18360/comment/a225bb71_aaf31b21">Patch Set #2, Line 134:</a> <code style="font-family:monospace,monospace">You'll notice that neither bob nor alice set {{geoloc_outging_call_profile}} because we never want to send location information _to_ them.</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">outgoing</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File res/res_geolocation/wiki/CivicAddress.md:</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.asterisk.org/c/asterisk/+/18360/comment/ab906075_616a2296">Patch Set #2, Line 1:</a> <code style="font-family:monospace,monospace">{toc}</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Missing new line at EOF</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File res/res_geolocation/wiki/GML.md:</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.asterisk.org/c/asterisk/+/18360/comment/d3880cc2_7cb75c96">Patch Set #2:</a> </p><p style="white-space: pre-wrap; word-wrap: break-word;">Is this file supposed to be empty?</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File res/res_geolocation/wiki/URI.md:</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.asterisk.org/c/asterisk/+/18360/comment/f5ca7ccf_45d9019d">Patch Set #2, Line 10:</a> <code style="font-family:monospace,monospace">Passing location descriptions using URIs is fairly simple from an Asterisk perspective.  It does however, require the implementer to establish and maintain infrastructure to handle the serving of those URIs.  Given the critical nature of the information, setting up such infrasstructure is not trivial and is beyond the scope of Asterisk and this documentation.</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">infrastructure</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/18360/comment/67e36a88_de12d4c1">Patch Set #2, Line 39:</a> <code style="font-family:monospace,monospace">Now let's say that Bob has DID {{12125551212}} assigned to him and he makes an outgoping call which is routed to "my-provider".  Asterisk would automatically add the following header to the INVITE:</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">outgoing, not outgoping</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/18360/comment/9c04ed48_8d21b196">Patch Set #2, Line 45:</a> <code style="font-family:monospace,monospace">Of course, this is a _very_ simple example that would add the Geolocation header to _all_ calls made via "my-provider".  If you oonly routed emergency calls to "my-provider" this would work fine but you probably don't want to leak location information on non-emergency calls.</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">only</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/18360">change 18360</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.asterisk.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.asterisk.org/c/asterisk/+/18360"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: development/16/geolocation </div>
<div style="display:none"> Gerrit-Change-Id: I85b239776209bff3fc3388cf9b0a322a1764db29 </div>
<div style="display:none"> Gerrit-Change-Number: 18360 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@sangoma.com> </div>
<div style="display:none"> Gerrit-Reviewer: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: N A <mail@interlinked.x10host.com> </div>
<div style="display:none"> Gerrit-Attention: Joshua Colp <jcolp@sangoma.com> </div>
<div style="display:none"> Gerrit-Attention: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Attention: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 19 Apr 2022 16:42:41 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: Yes </div>
<div style="display:none"> Gerrit-MessageType: comment </div>