<p></p>
Daniel-Constantin Mierla writes:<br>
<br>
> The commit message first line has to start with the name of the module<br>
> ("rr: ...") like per contributing guide: <br>
> <br>
>   * https://github.com/kamailio/kamailio/blob/master/.github/CONTRIBUTING.md#commit-message-format<br>
> <br>
> It is important for generating changelogs and releases. I made a<br>
> similar remark to your latest commit as well.<br>
<br>
OK, will add if this leads to anywhere.<br>
<br>
> Then, related to the commit itself, `myself` matching is related to<br>
> routing address (ip/domain) matching of the server, this should be the<br>
> default behaviour. Otherwise, on this case, it can happen that is<br>
> matching headers added by another node using same socket names, which<br>
> can have unexpected side effects in processing .<br>
<br>
If such conflicts can happen or not, depends on the proxy deployment<br>
environment.<br>
<br>
> Personally I do not really see the need for what this patch brings,<br>
> because you can specify the domains associated with the sever on<br>
> several ways: config alias parameters, domain module (with register<br>
> myself) for dynamic or large number of domains and corex module for<br>
> subdomains.<br>
<br>
In multi-tenant setup, MS Teams does not know which tenant INVITE from<br>
Kamailio tries to reach if first R-R URI does not contain FQDN of the<br>
tenant and, like everyone knows, MS does not care sh*t about standards<br>
compatibility.  So the FQDN needs to be there and also loose_route()<br>
needs to work.  I don't think that the other options you mention in the<br>
above can help here.<br>
<br>
> However, I am fine to add the behaviour of the patch, but controlled<br>
> by a new rr module parameter (e.g, myself_mode) , default being<br>
> disabled (e.g., myself_mode=0) so the match is done like so far, on<br>
> address elements only.<br>
<br>
OK, I'll try to add if the patch otherwise is OK.<br>
<br>
-- Juha<br>


<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/kamailio/kamailio/pull/2643#issuecomment-782832420">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABO7UZNUEYPIBWSGWLIS43DTADMCPANCNFSM4X6XS4GA">unsubscribe</a>.<img src="https://github.com/notifications/beacon/ABO7UZNVCJLB5DICFTZTXNLTADMCPA5CNFSM4X6XS4GKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOF2URGJA.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/kamailio/kamailio/pull/2643#issuecomment-782832420",
"url": "https://github.com/kamailio/kamailio/pull/2643#issuecomment-782832420",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>