5 Jan
2011
5 Jan
'11
1:15 p.m.
On 1/5/11 11:58 AM, Klaus Darilion wrote:
Am 05.01.2011 11:26, schrieb Daniel-Constantin Mierla:
Thanks, I can merge them.
Just some notes:
- $hu should be better documented in pseudo-variables cookbook --
somehow I missed adding it, but wiki is the place where all PV are
documented.
- I added outbound proxy, so will fix the part with the contact, but I
wanted to double check a thing -- the change in reply_200() function is
just parameter name, I see no reason to change that since it refers to
the contact address, the problem was the value given, not inside this
function
Cheers,
Daniel
[...]
Btw, you kept me back from some commits because you said you have some
to do. Is it still valid?
Yes, you are right. Sorry - I'm currently just to busy to apply them
in a clean way. (it would take me at least 1 day to get used to git
again :-)
Attached is a patch with all the changes I made in my local test setup
- maybe you find some time to merge them with your fixes.
The attached patch includes:
- remove purple compilation as it didn't worked for me (probably you
can skip that)
- extend documentation of xhttp
- extend documentation of rls with server_adress (local contact) and
presence_server (outboundproxy)
- do not start if RLs "server_address" is not set
- add presence_server (outbound proxy) support, obsolete by your patch
- fix: CSeq was reported as wrong although it correct due to wrong
checking
- fix: respond with proper contact header
- fix: remove duplicate Max-Forwards header
- fix: use "Supported" instead of "Support"
- fix: xcap exported pseudo variables: "uri" is only 3 characters
long, not 4
- increased table version in xcap module to 4 to write directly into
opensips xcap tables (probably you can skip that)
- debian fix: remove rls from EXTRA_EXCLUDED_MODULES
Thanks
Klaus
--
Daniel-Constantin Mierla
Kamailio (OpenSER) Advanced Training
Jan 24-26, 2011, Irvine, CA, USA
http://www.asipto.com