<p>For the record. This should probably be improved to handle in-memory contacts as well, since this can now be binary-replicated (right?)</p>
<p>16:28 < lirakis> bogdan_vs, if there was a "smart" nat ping ... then i think you could do distributed usrloc out of the box with replication, path headers, and a smart nat ping<br>
...<br>
16:31 < wdoekes> lirakis: smart nat ping? do you mean: <a href="https://github.com/OpenSIPS/opensips/pull/127" class="issue-link" title="Add listen interface (socket) checking to db based natping.">#127</a> ?<br>
16:32 < lirakis> wdoekes, yes<br>
16:33 < lirakis> was ... this merged?<br>
16:33 < wdoekes> no<br>
16:33 < lirakis> but yeah<br>
16:34 < lirakis> i mean ... we do this already with our own custom usrloc stuff<br>
16:34 < lirakis> im just saying for "out of the box" functionality<br>
16:34 < lirakis> i think that would be the only thing missing that i can think of<br>
16:34 < wdoekes> one could poke razvan to get it merged<br>
16:35 < lirakis> razvan ... poke<br>
16:35 < lirakis> he he<br>
16:35 < wdoekes> =)<br>
16:35 < lirakis> so .. you are actually just changing usrloc to return only local contacts from get_all_u_contacts ?<br>
16:36 < lirakis> but ... are you actually stlil getting contacts on other servers when you do a lookup ?<br>
16:36 < lirakis> b/c i would still wan that behavior<br>
16:36 < lirakis> so you can use path headers to route from an adjecent proxy to the "home" proxy<br>
16:36 < lirakis> wdoekes, not sure if im being clear<br>
16:38 < wdoekes> it's only the "get_all" that is affected<br>
16:38 < wdoekes> and then only the db-version<br>
16:38 < wdoekes> so for your binary replication, we may need to improve the patch<br>
16:39 < lirakis> ah<br>
16:39 < lirakis> ok<br>
16:39 < wdoekes> the get_all is only called from nathelper (afaics)<br>
16:39 < lirakis> right<br>
16:39 < lirakis> vs "lookup"<br>
16:39 < wdoekes> correct<br>
16:39 < lirakis> cool<br>
16:40 < wdoekes> I don't store users in mem, so I didn't need that. but I see how that could be an improvement<br>
16:41 < wdoekes> (and probably, the get_all_ should be renamed to get_all_local or something)<br>
...<br>
16:41 < lirakis> but again - im just thinking about generic out of the box functionality</p>
<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br>Reply to this email directly or <a href='https://github.com/OpenSIPS/opensips/pull/127#issuecomment-43891801'>view it on GitHub</a>.<img src='https://github.com/notifications/beacon/5479561__eyJzY29wZSI6Ik5ld3NpZXM6QmVhY29uIiwiZXhwaXJlcyI6MTcxNjM4NjUzMCwiZGF0YSI6eyJpZCI6MjAyNjE1NTR9fQ==--a79f20c3b14a6698a23165848b1cc556f53b8b87.gif' height='1' width='1'></p>