[Asterisk-code-review] res_geolocation: Add initial markdown for Geolocation wiki pages (asterisk[development/16/geolocation])
George Joseph
asteriskteam at digium.com
Wed May 25 12:27:48 CDT 2022
Attention is currently required from: N A, Joshua Colp, Kevin Harwell.
George Joseph has posted comments on this change. ( https://gerrit.asterisk.org/c/asterisk/+/18360 )
Change subject: res_geolocation: Add initial markdown for Geolocation wiki pages
......................................................................
Patch Set 2:
(9 comments)
Commit Message:
https://gerrit.asterisk.org/c/asterisk/+/18360/comment/f9533b47_b281131a
PS2, Line 10: since I'm going to be offline for a bit.
> Is this commit message still accurate? Either way I feel the last sentence should be replaced with something more substantive about the commit itself.
It's just a temporary commit.
Patchset:
PS2:
Updates and rebase coming shortly.
File res/res_geolocation/wiki/AsteriskImplementation.md:
https://gerrit.asterisk.org/c/asterisk/+/18360/comment/8da98f62_084d2bf9
PS2, Line 65: See [RFC6442|Geolocation#rfc6442] for more informatrion.|
> information
Done
https://gerrit.asterisk.org/c/asterisk/+/18360/comment/eab4cec2_e76be1a3
PS2, Line 134: You'll notice that neither bob nor alice set {{geoloc_outging_call_profile}} because we never want to send location information _to_ them.
> outgoing
Done
File res/res_geolocation/wiki/CivicAddress.md:
https://gerrit.asterisk.org/c/asterisk/+/18360/comment/b3de61ec_67e89fd9
PS2, Line 1: {toc}
> Missing new line at EOF
Done
File res/res_geolocation/wiki/GML.md:
PS2:
> Is this file supposed to be empty?
In the temporary commit, yes. It was just a placeholder there so the links to it resolved.
File res/res_geolocation/wiki/URI.md:
https://gerrit.asterisk.org/c/asterisk/+/18360/comment/c02aacc5_bb87427c
PS2, Line 10: 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.
> infrastructure
Done
https://gerrit.asterisk.org/c/asterisk/+/18360/comment/3c133ab4_8279573f
PS2, Line 39: 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:
> outgoing, not outgoping
Done
https://gerrit.asterisk.org/c/asterisk/+/18360/comment/73379055_3ea52a54
PS2, Line 45: 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.
> only
Done
--
To view, visit https://gerrit.asterisk.org/c/asterisk/+/18360
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings
Gerrit-Project: asterisk
Gerrit-Branch: development/16/geolocation
Gerrit-Change-Id: I85b239776209bff3fc3388cf9b0a322a1764db29
Gerrit-Change-Number: 18360
Gerrit-PatchSet: 2
Gerrit-Owner: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: Joshua Colp <jcolp at sangoma.com>
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>
Gerrit-Reviewer: N A <mail at interlinked.x10host.com>
Gerrit-Attention: N A <mail at interlinked.x10host.com>
Gerrit-Attention: Joshua Colp <jcolp at sangoma.com>
Gerrit-Attention: Kevin Harwell <kharwell at digium.com>
Gerrit-Comment-Date: Wed, 25 May 2022 17:27:48 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: N A <mail at interlinked.x10host.com>
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20220525/73415e3c/attachment.html>
More information about the asterisk-code-review
mailing list