Hello,

On 9/12/11 12:04 PM, Ozren Lapcevic wrote:
Hi,

now I'm using t_flush_flags() after setting the accounting flag in falure_route and latest updates (I have a new 3.1 clone without depth parameter and I've made sure your changes are in sources), but it doesn't solve the case, accounting behavior is still exactly the same as described in the first mail.

If it is of any help, previously I've tested the same scenario where I was setting the accounting flag in RELAY and LOCATION routes which were called from failure route.

can you send the debug messages for such a call (use debug=4 in your config)? Also, set a tm onreply_route[x] for such invite where to print with xlog the value of the flags ($mF).

Cheers,
Daniel


Cheers
Ozren

On Mon, Sep 12, 2011 at 9:19 AM, Daniel-Constantin Mierla <mico...@gmail.com <mailto:mico...@gmail.com>> wrote:

    Hello,

    over the weekend I backported the implementation of
    t_flush_flags() as well as auto-update of transaction flags after
    branch/failure route. Just pull the latest branch 3.1 and you are
    ready to compile/install.

    Let us know if it solves the case.

    Cheers,
    Daniel


    On 9/9/11 2:40 PM, Daniel-Constantin Mierla wrote:
    Hello,

    when you pulled the GIT branch 3.1, did you use --depth 1, like next?

      git clone --depth 1 git://git.sip-router.org/sip-router
    <http://git.sip-router.org/sip-router> kamailio

    The you just got the snapshot at that time without the history of
    the commit and probably you did the git clone after I did the
    commit, a new git pull taking newer commit. So try cloning again
    the latest branch 3.1, without --depth 1 parameter.

    Cheers,
    Daniel


    On 9/7/11 2:20 PM, Ozren Lapcevic wrote:
    Hi,

    I've previously installed kamailio from git branch 3.1. Now,
    I've manged to git cherry-pick your patch, but got "fatal: Could
    not find 83620cb7cd14ee3b509eef72d99337567f53967f" when tried to
    get t_flush_flags(). I've double-checked commit and found it
    here:
    
http://git.sip-router.org/cgi-bin/gitweb.cgi?p=sip-router;a=commit;h=83620cb7cd14ee3b509eef72d99337567f53967f.
    I don't know why I can't cherry-pick it.

    Your patch alone, without t_flush_flags(), doesn't change
    anything in my scenario, there is still no logging of 2nd branch.

    Cheers
    Ozren


    On Wed, Sep 7, 2011 at 1:05 PM, Daniel-Constantin Mierla
    <mico...@gmail.com <mailto:mico...@gmail.com>> wrote:

        Hello,


        On 9/7/11 11:25 AM, Ozren Lapcevic wrote:
        Hi Daniel,

        thanks for the quick fix and reply.

        What is the easiest way to try this new patch? I'm running
        kamailio 3.1.4 and there is no t_flush_flags() in tmx
        module in that version. I suppose I need to install
        Kamailio Devel from git
        
(http://www.kamailio.org/dokuwiki/doku.php/install:kamailio-devel-from-git)
        to get t_flush flags() and your patch or is there a
        workaround to apply them to my 3.1.4 branch?

        did you install 3.1.4 from tarball/packages or is it from
        git branch 3.1? If later, then you can do:

        git pull origin
        git cherry-pick -x  83620cb7cd14ee3b509eef72d99337567f53967f
        git cherry-pick -x  c589ca35b2aa3097a3c9e2a5a050514337300c05

        then recompile/install. First cherry-pick brings the
        t_flush_flags, the second auto-update of the flags after
        branch/failure route.

        If you installed from packages, then you would need to
        repackage yourself after patching. The patches are available
        at commit url, for example:


        
http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=c589ca35b2aa3097a3c9e2a5a050514337300c05

        There you find at top of the page a link named 'patch' that
        you can use with git tools to apply or extract the
        diff-patch part and apply with patch.

        Cheers,
        Daniel


        Cheers
        Ozren


        On Tue, Sep 6, 2011 at 2:18 PM, Daniel-Constantin Mierla
        <mico...@gmail.com <mailto:mico...@gmail.com>> wrote:

            Hello,

            can you use t_flush_flags() after setting the
            accounting flag in falure_route? Automatic update was
            missing so far, reported by Alex Hermann as well. I
            just did a patch, so if you want to try it, see the commit:

            
http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=c589ca35b2aa3097a3c9e2a5a050514337300c05

            Actually, reporting if all goes fine with this patch,
            will help in backporting it to 3.1 branch.

            Thanks,
            Daniel




_______________________________________________
SIP Express Router (SER) and Kamailio (OpenSER) - sr-users mailing list
sr-users@lists.sip-router.org
http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-users

--
Daniel-Constantin Mierla -- http://www.asipto.com
Kamailio Advanced Training, Oct 10-13, Berlin: http://asipto.com/u/kat
http://linkedin.com/in/miconda -- http://twitter.com/miconda

_______________________________________________
SIP Express Router (SER) and Kamailio (OpenSER) - sr-users mailing list
sr-users@lists.sip-router.org
http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-users

Reply via email to