| 00:03.14 | *** join/#asterisk-dev snuff-work (~snuffy@210.8.167.13) |
| 00:03.14 | *** mode/#asterisk-dev [+o snuff-work] by ChanServ |
| 01:18.50 | *** join/#asterisk-dev opticron (~opticron@pianoben.ch) |
| 01:18.50 | *** mode/#asterisk-dev [+o opticron] by ChanServ |
| 01:40.34 | *** join/#asterisk-dev ivan` (~ivan@unaffiliated/ivan/x-000001) |
| 05:23.38 | *** join/#asterisk-dev oej (~Adium@h87-96-134-129.dynamic.se.alltele.net) |
| 05:23.39 | *** mode/#asterisk-dev [+o oej] by ChanServ |
| 06:15.21 | *** join/#asterisk-dev snuff-work (~snuffy@210.8.167.13) |
| 06:15.21 | *** mode/#asterisk-dev [+o snuff-work] by ChanServ |
| 06:51.58 | *** join/#asterisk-dev oej (~Adium@h87-96-134-129.dynamic.se.alltele.net) |
| 06:51.59 | *** mode/#asterisk-dev [+o oej] by ChanServ |
| 07:10.53 | oej | Morning folks! |
| 07:27.44 | *** join/#asterisk-dev hehol (~hehol@2001:1438:1009:200:e953:b73b:c133:d78) |
| 07:44.27 | snuff-work | ello oej |
| 08:02.43 | *** join/#asterisk-dev tzafrir (~tzafrir@local.xorcom.com) |
| 08:02.44 | *** mode/#asterisk-dev [+o tzafrir] by ChanServ |
| 08:27.22 | *** join/#asterisk-dev hehol (~hehol@2001:1438:1009:200:b436:70cb:cc90:ab95) |
| 09:05.44 | *** join/#asterisk-dev danjenkins (~dan@cpc65687-folk2-2-0-cust207.1-2.cable.virginm.net) |
| 10:43.10 | *** join/#asterisk-dev coreyf1513 (~cfarrell@2001:4830:117f:2:fc1d:f303:8e40:1391) |
| 11:38.36 | *** join/#asterisk-dev Marquis42 (~mrbmw@d149-67-95-208.try.wideopenwest.com) |
| 11:52.17 | *** join/#asterisk-dev puzzled (~patrick@5ED26DE9.cm-7-3b.dynamic.ziggo.nl) |
| 11:52.25 | puzzled | hi |
| 11:54.53 | puzzled | I created a patch with diff -uNr asterisk-11.org asterisk-11 > my.patch. Now want to put it in reviewboard. Can anyone please tell me what I need to put in "Base Directory" in the New Review Request? |
| 12:01.00 | opticron | branches/11 should do it |
| 12:01.48 | opticron | puzzled, ^^ |
| 12:02.01 | puzzled | opticron: thanks |
| 12:03.04 | puzzled | ugh Unable to parse diff revision header '2014-03-10 19:13:16.000000000 +0100' |
| 12:03.44 | opticron | can't just do a svn diff to generate the patch? |
| 12:04.39 | *** join/#asterisk-dev [TK]D-Fender (~chatzilla@216-191-106-163.dedicated.allstream.net) |
| 12:13.12 | *** join/#asterisk-dev protocoldoug (~quassel@unaffiliated/protocoldoug) |
| 12:17.58 | *** join/#asterisk-dev puzzled (~patrick@5ED26DE9.cm-7-3b.dynamic.ziggo.nl) |
| 12:19.03 | puzzled | opticron: my laptop battery died. Do you have any suggestion how to get reviewboard to accept a simple 3 line patch? |
| 12:21.06 | russellb | post-review seems to always be the best bet |
| 12:23.00 | opticron | I have no comments on post-review |
| 12:23.12 | opticron | puzzled, have you tried generating a patch with "svn diff" |
| 12:23.14 | opticron | ? |
| 12:23.31 | opticron | russellb, it used to be extremely flaky, so I removed it from my workflow |
| 12:23.39 | opticron | I hear it's better now |
| 12:23.50 | puzzled | opticron: I don't even have subversion installed. Haven't used it in many years |
| 12:23.55 | opticron | ah |
| 12:23.56 | russellb | post-review? *nods* i think it was running an old reviewboard or something |
| 12:24.30 | russellb | not post-review running an old version, i mean that being the reason it wasn't working well |
| 12:24.33 | russellb | anyway |
| 12:30.29 | opticron | puzzled, you appear to be having same problem as the person here: http://lists.digium.com/pipermail/asterisk-dev/2012-June/055463.html |
| 12:31.15 | opticron | the revision header should be more like --- file/path (revision revnum) |
| 12:31.21 | puzzled | opticron: I used diff -uNr on the regular 11.8.1 tarball |
| 12:31.23 | opticron | +++ file/path (working copy) |
| 12:31.33 | puzzled | seems reviewboard is as stupid as the other atlassian tools |
| 12:31.45 | opticron | reviewboard isn't an atlassian tool :) |
| 12:32.05 | puzzled | opticron russellb thank you for your suggestions but I give up. Too much effort for a 3 line patch |
| 12:34.26 | *** part/#asterisk-dev puzzled (~patrick@5ED26DE9.cm-7-3b.dynamic.ziggo.nl) |
| 12:34.57 | russellb | heh |
| 12:35.33 | russellb | ¯\_(ツ)_/¯ |
| 12:37.42 | russellb | <3 git-review :-p |
| 12:37.55 | Marquis42 | heh |
| 12:38.14 | *** join/#asterisk-dev roderickm (~roderickm@67.63.143.254) |
| 12:56.12 | opticron | our workflow is pretty svn-centric atm |
| 12:57.05 | *** join/#asterisk-dev roderickm (~roderickm@67.63.143.254) |
| 13:43.46 | *** join/#asterisk-dev newtonr (~newtonr@nat/digium/x-wiaruqhezylwuotb) |
| 13:43.46 | *** mode/#asterisk-dev [+o newtonr] by ChanServ |
| 13:49.52 | *** join/#asterisk-dev putnopvut (~putnopvut@asterisk/master-of-queues/mmichelson) |
| 13:49.52 | *** mode/#asterisk-dev [+o putnopvut] by ChanServ |
| 13:50.43 | *** join/#asterisk-dev anonymouz666 (~anonymouz@187.76.181.102) |
| 13:57.50 | *** join/#asterisk-dev Tim_Toady (~fuzzy@83.212.108.130) |
| 14:07.59 | *** join/#asterisk-dev mjordan_ (~mjordan@nat/digium/x-safucndhmvauvunf) |
| 14:08.01 | *** mode/#asterisk-dev [+o mjordan_] by ChanServ |
| 14:10.07 | mjordan_ | file: are you ready for Tuesday? |
| 14:10.13 | file | no |
| 14:10.14 | file | NEVER |
| 14:10.25 | mjordan_ | I never could get the hang of Tuesdays |
| 14:10.33 | file | mjordan_, you've grown a tail |
| 14:10.36 | mjordan_ | I have? |
| 14:10.42 | oej | This Tuesday has a strange taste of Monday. |
| 14:10.42 | mjordan_ | that's why sitting has been so painful |
| 14:10.46 | mjordan_ | oej: I agree |
| 14:11.00 | mjordan_ | oej: Monday was something of a non-starter for me |
| 14:11.12 | oej | We had a public holiday yesterday… |
| 14:11.18 | mjordan_ | oej: which one? |
| 14:11.25 | oej | Thanks for all the feedback on the reviews! |
| 14:11.38 | oej | Will try to get time to work on them soon. |
| 14:11.41 | mjordan_ | oej: np, I still need to look at your SRTP proposal. It's on my list today |
| 14:11.57 | oej | There's a problem with MKI. |
| 14:12.10 | oej | If we claim to support it, we need a libsrtp with support for it. And we don't have it. |
| 14:12.23 | oej | There's an old patch around. |
| 14:12.41 | oej | Microsoft sends ANSWERs with both a crypto line with MKI and one without. |
| 14:12.47 | oej | That works with my code. |
| 14:12.49 | mjordan_ | blech |
| 14:12.56 | mjordan_ | libsrtp hasn't patched? |
| 14:13.06 | oej | But when we send SDP without MKI, they answr with MKI. Booom. |
| 14:13.28 | oej | no, libsrtp has not. The patch has been around for a few years. Tried mailing their list. |
| 14:13.45 | mjordan_ | that's disappointing. Probably why most other projects embed libsrtp |
| 14:13.59 | mjordan_ | I'm not advocating that mind you, merely an observation |
| 14:14.12 | oej | Yes. Well, if the old patch still can be applied, I can resubmit it. Will just have to buy a beer to the developer, who's Swedish. |
| 14:14.35 | mjordan_ | oej: I'm willing to contribute to that cause :-) |
| 14:14.48 | oej | Noticed that the guys behind the new kamailio RTP engine wrote their own SRTP stack. GPL 3 sadly. |
| 14:14.55 | oej | Maybe we can discuss that with Andreas. |
| 14:15.08 | mjordan_ | oej: interesting. I noticed a few other projects starting to use GPL 3 as well. |
| 14:15.38 | oej | Service providers are not very happy about it. |
| 14:15.44 | mjordan_ | I imagine not |
| 14:16.18 | oej | DNS guru coming my way on a motorbike. Doesn't happen every day. Time for NAPTR; SRV and IPv6 discussions! |
| 14:16.43 | mjordan_ | fun! |
| 14:16.43 | oej | Will continue my DNS SRV patch work soon. We will have to discuss the core parts at some point. |
| 14:16.49 | mjordan_ | sure |
| 14:17.24 | oej | Having fun with Ghost Peers in the SIP channel. WIll be a tough thing to educate asterisk users about. |
| 14:17.38 | oej | I haven't checked how you handle that in the PJSIP channel. |
| 14:17.54 | mjordan_ | It doesn't tie the endpoint to the IP address |
| 14:18.31 | file | it also does not currently store multiple address for matching given a resolveable host or refresh it, albeit the infrastructure is there to do such a thing |
| 14:18.34 | oej | So if I register to edvina.net that has three servers with dual stack - how do you match incoming calls from all these servers? |
| 14:19.02 | oej | Ok, so you need pjsip ghosts as well… |
| 14:19.04 | mjordan_ | oej: it depends. There are modules that perform identification for inbound requests |
| 14:19.10 | file | we don't need ghosts |
| 14:19.21 | oej | Identification - how? |
| 14:19.43 | oej | this needs to be dynamic and relookup or expire on ttl. |
| 14:19.45 | anonymouz666 | oej: I saw a comment you wrote in chan_sip.c, refer with replaces header works on 11? |
| 14:19.55 | file | there are two modules, one which you can explicitly list hosts/IP addresses (multiple) and associate them with an endpoint, and also one which uses the user portion of the From header |
| 14:20.03 | oej | It worked on 1.4 - anonymouz666 |
| 14:20.14 | file | but an API exists and additional logic can be added or the existing stuff extended |
| 14:20.37 | oej | That is the API that needs to be used for ghosts :-) |
| 14:20.40 | mjordan_ | oej: that goes back to the part of the stack that I was talking about at Kamailio world that can be extended |
| 14:21.30 | mjordan_ | oej: I agree with file, it doesn't necessarily have to have ghost endpoints. The location of an endpoint is not tied explicitly to the endpoint - you can have multiple locations associated with an endpoint. How the stack determines that a request maps to an endpoint is 'plugable' however. |
| 14:21.41 | mjordan_ | It wouldn't surprise me if there are some more complex scenarios that aren't handled by the default modules |
| 14:22.06 | mjordan_ | but because the step that says "please associate this request with an endpoint" is a pluggable piece of logic, that can be extended with a new module (or the existing ones improved) |
| 14:22.46 | file | heck I had a patch up long ago which added a unique identifier to outbound REGISTER requests and registered custom logic for the matching, which then looked at the incoming INVITE traffic to match it back to a configured endpoint |
| 14:23.30 | file | did plan to resurrect that eventually |
| 14:24.17 | mjordan_ | anonymouz666: if refer w/replaces isn't working, please file a bug report. It should be working. |
| 14:26.50 | oej | file: We have that in production in chan_sip at Inteno… |
| 14:27.00 | mjordan_ | interesting |
| 14:28.17 | *** join/#asterisk-dev kharwell (~kharwell@nat/digium/x-gyojosyiriilazuy) |
| 14:39.54 | *** join/#asterisk-dev rmudgett (~rmudgett@nat/digium/x-xxsbluplwliqnxsb) |
| 14:43.44 | *** join/#asterisk-dev leifmadsen (~free-mads@asterisk/documenteur-extraordinaire/blitzrage) |
| 14:43.46 | *** mode/#asterisk-dev [+o leifmadsen] by ChanServ |
| 14:57.42 | *** join/#asterisk-dev gtjoseph (~gtj0@unaffiliated/gtj) |
| 14:59.27 | *** join/#asterisk-dev rmudgett (~rmudgett@nat/digium/x-jfjszbnwiowqknfn) |
| 15:16.31 | *** join/#asterisk-dev anonymouz666 (~anonymouz@189-25-100-226.user.veloxzone.com.br) |
| 15:26.11 | *** join/#asterisk-dev anonymouz666 (~anonymouz@187.76.181.102) |
| 16:19.49 | *** join/#asterisk-dev vlad_starkov (~vlad_star@212.34.52.66) |
| 16:23.32 | *** join/#asterisk-dev oej (~Adium@h87-96-134-129.dynamic.se.alltele.net) |
| 16:23.32 | *** mode/#asterisk-dev [+o oej] by ChanServ |
| 16:44.56 | file | jrose_atDigium, your pjsip hold test makes me happy |
| 16:51.46 | skrusty | just spotted 22677 is fixed, great news! :) |
| 17:26.08 | *** join/#asterisk-dev anthm (~anthm@freeswitch/developer/anthm) |
| 18:03.35 | jrose_atDigium | Oh, did it catch something? |
| 18:06.22 | file | a cold! |
| 18:06.35 | file | no, I just needed to add more scenarios and it was easy and then confirmed the problem |
| 18:06.46 | jrose_atDigium | neat |
| 18:06.47 | file | and also that there were no regressions from my fix |
| 18:06.57 | file | testing works? say what/ |
| 18:37.42 | gtjoseph | hey file: if you get a chance today can you peruse https://reviewboard.asterisk.org/r/3405/? rmudgett gave it a "ship it" but I just wanted one more. |
| 18:38.45 | file | I can get to it tomorrow |
| 18:41.37 | gtjoseph | file: cool, thanks |
| 19:20.55 | mjordan_ | skrusty: yeah, that should go out in 12.3.0 |
| 19:22.19 | *** join/#asterisk-dev danjenkins (~dan@cpc65687-folk2-2-0-cust207.1-2.cable.virginm.net) |
| 19:59.03 | *** join/#asterisk-dev otherwiseguy (~otherwise@23-255-171-52.mci.googlefiber.net) |
| 20:23.41 | *** join/#asterisk-dev danjenkins (~dan@cpc65687-folk2-2-0-cust207.1-2.cable.virginm.net) |
| 22:02.59 | *** join/#asterisk-dev roderickm (~roderickm@99-124-89-102.lightspeed.dctral.sbcglobal.net) |
| 22:27.51 | *** join/#asterisk-dev skirmisha (~vk@46.47.82.189) |
| 22:28.00 | skirmisha | finally |
| 22:28.05 | skirmisha | hi guys |
| 22:29.56 | skirmisha | i have strange issue which i can't figure out |
| 22:31.40 | skirmisha | when extension unregister , most probably just connect to another server and not sending unregister whatsoever, the old server keeps sending options msg to that extension thinking its still registered. I see that this stays in the memory of the asterisk server and sip reload fix it, but is there a way or something to change in the code so it does after a while? |
| 22:32.03 | skirmisha | talking about sip |
| 22:35.25 | *** join/#asterisk-dev [TK]D-Fender (~chatzilla@64.235.216.2) |
| 22:35.53 | skirmisha | any ideas why this happens |
| 22:59.32 | *** join/#asterisk-dev Juggie (~Juggie@unaffiliated/juggie) |
| 23:02.06 | [TK]D-Fender | Why he quits? |
| 23:20.58 | newtonr | you only had three minutes to answer that one and you missed it! |
| 23:21.12 | newtonr | ten demerits! |
| 23:30.35 | *** part/#asterisk-dev kharwell (~kharwell@nat/digium/x-gyojosyiriilazuy) |
| 23:55.20 | *** join/#asterisk-dev roderickm (~roderickm@99-124-89-102.lightspeed.dctral.sbcglobal.net) |