Re: [OpenSIPS-Devel] [opensips] opensips 1.9 (latest trunk) hangs when rabbitmq (using event_rabbitmq) drops connection (#81)

[Bobby Smith]

We think things seem stable with the timeout at the moment, but a big
problem I think we have is reestablishing the socket after the timeout
happens and drops the connection (the tcp connection to rabbitmq).  I
believe this requires a restart of OpenSIPS to get messages flowing to
rabbitmq again.  This isn't preventing us with moving forward, it's more of
a nuissance at the moment than anything, and the core hanging problem seems
to have been fixed with this pull request.

It's a problem we really haven't had a lot of time to work on right now,
but Frank's changes have at least prevented OpenSIPS from possibly hanging
processing due to rabbitmq falling behind/becoming unavailable.  The other
thing we want to see added (and might add ourselves and contribute when we
have time) is more than the default exchange configuration, being able to
configure more than the default user/password in the connect url, etc --
just minor enhancements.  I think Frank made a pull request around being
able to configure routing to an exchange other than default, but we haven't
really had a lot of time to test that either and I didn't see that one
accepted.

I hope this helps.


On Tue, Mar 11, 2014 at 6:00 AM, Răzvan Crainea wrote:

> Any updates on this? Are you still experiencing any issues?
>
> —
> Reply to this email directly or view it on 
> GitHub
> .
>

---
Reply to this email directly or view it on GitHub:
https://github.com/OpenSIPS/opensips/issues/81#issuecomment-37367416___
Devel mailing list
Devel@lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/devel

Re: [OpenSIPS-Devel] Error on compling cachedb_cassandra module Opensips 1.10

[Richard Revels]

This seems like a question maybe better suited to Opensips Users mailing
list but in any case, you need to make sure you have thrift installed.  If
you are under the impression it is then use locate and make sure the
include file is somewhere it can be found by the compiler.

[root@registrar-1 ~]# locate Thrift.h
/usr/local/include/thrift/Thrift.h
/usr/local/src/thrift-0.8.0/lib/cpp/src/Thrift.h
/usr/local/src/thrift-0.8.0/lib/hs/src/Thrift.hs


As you can see, I'm using version 0.8.0.  I seem to remember having
problems compiling 0.9.1 on CentOS.

There was another change I had to do in the Makefile in the module
directory after getting thrift installed.  This might be what you were
talking about in your email but just in case, I'll copy it here as well.


"Cassandra came to my attention a couple days ago.  I've never used it so
thought I would give it a try.  To get it to compile on my centos servers I
had to modify the CXXFLAGS line in modules/cachedb_cassandra/Makefile to:"

CXXFLAGS=$(CFLAGS:-Wno-deprecated option=) -DHAVE_NETINET_IN_H


Hope this helps.

Richard



On Wed, Mar 5, 2014 at 5:16 AM, srivastava himanshu  wrote:

> Hi,
>
> I am trying to compile cachedb_cassandra module to integrate my opensips
> with cassandra. However, on compiling , I am getting flow of errors :
>
> make[1]: Entering directory
> `/home/himanshu/opensips-1.10.0-tls/modules/cachedb_cassandra'
> Compiling cachedb_cassandra_dbase.cpp
> Compiling cassandra_constants.cpp
> cc1plus: warning: command line option "-Wold-style-definition" is valid for
> C/ObjC but not for C++
> Compiling cassandra_types.cpp
> cc1plus: warning: command line option "-Wold-style-definition" is valid for
> C/ObjC but not for C++
> Compiling Cassandra.cpp
> cc1plus: warning: command line option "-Wold-style-definition" is valid for
> C/ObjC but not for C++
> cc1plus: warning: command line option "-Wold-style-definition" is valid for
> C/ObjC but not for C++
> In file included from cassandra_constants.h:9,
>  from cassandra_constants.cpp:6:
> cassandra_types.h:9:20: error: Thrift.h: No such file or directory
> cassandra_types.h:10:35: error: TApplicationException.h: No such file or
> directory
> cassandra_types.h:11:32: error: protocol/TProtocol.h: No such file or
> directory
> cassandra_types.h:12:34: error: transport/TTransport.h: No such file or
> directory
> .
> .
> .
> .
> .
> .
> .
> There are many more errors after this, but I think that could be due to
> missing .h files error.
> Opensips version is 1.10
> I tried making change in Makefile but got same result.  Is there anything I
> am missing.   Please help.
>
> Thanks
>
>
>
> --
> View this message in context:
> http://opensips-open-sip-server.1449251.n2.nabble.com/Error-on-compling-cachedb-cassandra-module-Opensips-1-10-tp7589903.html
> Sent from the OpenSIPS - Devel mailing list archive at Nabble.com.
>
> ___
> Devel mailing list
> Devel@lists.opensips.org
> http://lists.opensips.org/cgi-bin/mailman/listinfo/devel
>
___
Devel mailing list
Devel@lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/devel

Re: [OpenSIPS-Devel] [opensips] Fix examples, some presence settings have moved (#76)

[Saúl Ibarra Corretgé]

Ah, sorry about that! Thanks Liviu!

---
Reply to this email directly or view it on GitHub:
https://github.com/OpenSIPS/opensips/issues/76#issuecomment-37323061___
Devel mailing list
Devel@lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/devel

[OpenSIPS-Devel] [OpenSIPS/opensips] 513abb: residential config: Also define the "xcap" db_url

[Liviu Chircu]

  Branch: refs/heads/1.9
  Home:   https://github.com/OpenSIPS/opensips
  Commit: 513abb25a6f1a8c68115dfcb47d1fa2802f59c40
  
https://github.com/OpenSIPS/opensips/commit/513abb25a6f1a8c68115dfcb47d1fa2802f59c40
  Author: Liviu Chircu 
  Date:   2014-03-11 (Tue, 11 Mar 2014)

  Changed paths:
M menuconfig/configs/opensips_residential.m4

  Log Message:
  ---
  residential config: Also define the "xcap" db_url
(cherry picked from commit 4b4ac96e13dc4686172eb4eecb9dee68be2e6f2b)


___
Devel mailing list
Devel@lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/devel

[OpenSIPS-Devel] [OpenSIPS/opensips] 1e3209: residential config: Also define the "xcap" db_url

[Liviu Chircu]

  Branch: refs/heads/1.10
  Home:   https://github.com/OpenSIPS/opensips
  Commit: 1e320928115f2b307895a0b89565adf5bcb46ace
  
https://github.com/OpenSIPS/opensips/commit/1e320928115f2b307895a0b89565adf5bcb46ace
  Author: Liviu Chircu 
  Date:   2014-03-11 (Tue, 11 Mar 2014)

  Changed paths:
M menuconfig/configs/opensips_residential.m4

  Log Message:
  ---
  residential config: Also define the "xcap" db_url
(cherry picked from commit 4b4ac96e13dc4686172eb4eecb9dee68be2e6f2b)


___
Devel mailing list
Devel@lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/devel

Re: [OpenSIPS-Devel] [opensips] Fix examples, some presence settings have moved (#76)

[Liviu Chircu]

Found a presence startup issue and fixed in 
4b4ac96e13dc4686172eb4eecb9dee68be2e6f2b.

---
Reply to this email directly or view it on GitHub:
https://github.com/OpenSIPS/opensips/issues/76#issuecomment-37321379___
Devel mailing list
Devel@lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/devel

[OpenSIPS-Devel] [OpenSIPS/opensips] 4b4ac9: residential config: Also define the "xcap" db_url

[Liviu Chircu]

  Branch: refs/heads/master
  Home:   https://github.com/OpenSIPS/opensips
  Commit: 4b4ac96e13dc4686172eb4eecb9dee68be2e6f2b
  
https://github.com/OpenSIPS/opensips/commit/4b4ac96e13dc4686172eb4eecb9dee68be2e6f2b
  Author: Liviu Chircu 
  Date:   2014-03-11 (Tue, 11 Mar 2014)

  Changed paths:
M menuconfig/configs/opensips_residential.m4

  Log Message:
  ---
  residential config: Also define the "xcap" db_url


  Commit: e93234bf3fb46c515d1ee758e40fb5feb6cc0651
  
https://github.com/OpenSIPS/opensips/commit/e93234bf3fb46c515d1ee758e40fb5feb6cc0651
  Author: Liviu Chircu 
  Date:   2014-03-11 (Tue, 11 Mar 2014)

  Changed paths:
M modules/db_http/README
M modules/db_http/db_http.c
M modules/db_http/doc/db_http_admin.xml
M modules/db_http/http_dbase.c
M modules/db_http/http_dbase.h

  Log Message:
  ---
  Merge branch 'master' of github.com:OpenSIPS/opensips


Compare: 
https://github.com/OpenSIPS/opensips/compare/ab80adec1665...e93234bf3fb4___
Devel mailing list
Devel@lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/devel

Re: [OpenSIPS-Devel] [opensips] presence_dialoginfo: Fix memory leak in dlginfo_agg_nbody. (#174)

[Damien Sandrs]

Hi,

The leak was fixed in my local branch, and I apparently forgot to push it. 
Apologies.

About the bound checking error, the code was cut&pasted from another place in 
the same file where the same error is still present:

Line 165:
/* create the new NOTIFY body */
if ( (pres_user->len + pres_domain->len + 1) > MAX_URI_SIZE) {
LM_ERR("entity URI too long, maximum=%d\n", MAX_URI_SIZE);
return NULL;
}

---
Reply to this email directly or view it on GitHub:
https://github.com/OpenSIPS/opensips/pull/174#issuecomment-37311079___
Devel mailing list
Devel@lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/devel

Re: [OpenSIPS-Devel] [opensips] Revert 4584e60... based on #86 (#132)

[Răzvan Crainea]

Closed #132.

---
Reply to this email directly or view it on GitHub:
https://github.com/OpenSIPS/opensips/pull/132___
Devel mailing list
Devel@lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/devel

[OpenSIPS-Devel] [OpenSIPS/opensips] 53f646: db_http: add value_delimiter parameter

[Razvan Crainea]

  Branch: refs/heads/1.10
  Home:   https://github.com/OpenSIPS/opensips
  Commit: 53f64612847ee5e1a8da660068ded4d5476355ea
  
https://github.com/OpenSIPS/opensips/commit/53f64612847ee5e1a8da660068ded4d5476355ea
  Author: Razvan Crainea 
  Date:   2014-03-11 (Tue, 11 Mar 2014)

  Changed paths:
M modules/db_http/README
M modules/db_http/db_http.c
M modules/db_http/doc/db_http_admin.xml
M modules/db_http/http_dbase.c
M modules/db_http/http_dbase.h

  Log Message:
  ---
  db_http: add value_delimiter parameter

Add a new parameter used to specify the fields delimiters for queries
variables

(cherry picked from commit ab80adec1665b6416d92637d10677a696550c1db)


___
Devel mailing list
Devel@lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/devel

Re: [OpenSIPS-Devel] [opensips] Revert 4584e60... based on #86 (#132)

[Răzvan Crainea]

This should be fixed in ab80adec1665b6416d92637d10677a696550c1db. If there are 
any issues, please re-open ticket #86.

---
Reply to this email directly or view it on GitHub:
https://github.com/OpenSIPS/opensips/pull/132#issuecomment-37290868___
Devel mailing list
Devel@lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/devel

Re: [OpenSIPS-Devel] [opensips] db_http module doesn't match its documentation (#86)

[Răzvan Crainea]

Hi again!

Actually, giving a second thought, this was a misunderstanding of the initial 
report, as well as of the initial fix 4584e605f7233f68e303bd8d0a8066ab9f0e4fa2. 
In fact, that commit was meant to allow the field delimiter to be flexible, but 
it was introducing a constraint on a different separator, namely the variables 
one.
In order to fix this, I added a new parameter in commit 
ab80adec1665b6416d92637d10677a696550c1db, that allows you to change the 
variables delimiter, which now defaults to comma.

Best regards,
Răzvan

---
Reply to this email directly or view it on GitHub:
https://github.com/OpenSIPS/opensips/issues/86#issuecomment-37290651___
Devel mailing list
Devel@lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/devel

Re: [OpenSIPS-Devel] [opensips] db_http module doesn't match its documentation (#86)

[Răzvan Crainea]

Closed #86.

---
Reply to this email directly or view it on GitHub:
https://github.com/OpenSIPS/opensips/issues/86___
Devel mailing list
Devel@lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/devel

[OpenSIPS-Devel] [OpenSIPS/opensips] ab80ad: db_http: add value_delimiter parameter

[Razvan Crainea]

  Branch: refs/heads/master
  Home:   https://github.com/OpenSIPS/opensips
  Commit: ab80adec1665b6416d92637d10677a696550c1db
  
https://github.com/OpenSIPS/opensips/commit/ab80adec1665b6416d92637d10677a696550c1db
  Author: Razvan Crainea 
  Date:   2014-03-11 (Tue, 11 Mar 2014)

  Changed paths:
M modules/db_http/README
M modules/db_http/db_http.c
M modules/db_http/doc/db_http_admin.xml
M modules/db_http/http_dbase.c
M modules/db_http/http_dbase.h

  Log Message:
  ---
  db_http: add value_delimiter parameter

Add a new parameter used to specify the fields delimiters for queries
variables


___
Devel mailing list
Devel@lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/devel

Re: [OpenSIPS-Devel] [opensips] Revert 4584e60... based on #86 (#132)

[Răzvan Crainea]

Reopened #132.

---
Reply to this email directly or view it on GitHub:
https://github.com/OpenSIPS/opensips/pull/132___
Devel mailing list
Devel@lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/devel

[OpenSIPS-Devel] [OpenSIPS/opensips] ba22d0: Add s.index and s.rindex transformations

[Răzvan Crainea]

  Branch: refs/heads/master
  Home:   https://github.com/OpenSIPS/opensips
  Commit: ba22d03a1879d34a51b7a4a40bfe62d477650b8f
  
https://github.com/OpenSIPS/opensips/commit/ba22d03a1879d34a51b7a4a40bfe62d477650b8f
  Author: Ryan Bullock 
  Date:   2013-11-12 (Tue, 12 Nov 2013)

  Changed paths:
M transformations.c
M transformations.h

  Log Message:
  ---
  Add s.index and s.rindex transformations


  Commit: 112f5043a5c31f27e8a6fe18cb9dd7a043658b5e
  
https://github.com/OpenSIPS/opensips/commit/112f5043a5c31f27e8a6fe18cb9dd7a043658b5e
  Author: Răzvan Crainea 
  Date:   2014-03-11 (Tue, 11 Mar 2014)

  Changed paths:
M transformations.c
M transformations.h

  Log Message:
  ---
  Merge pull request #125 from rrb3942/index_trans

Add s.index and s.rindex transformations


Compare: 
https://github.com/OpenSIPS/opensips/compare/1e02b323fa38...112f5043a5c3___
Devel mailing list
Devel@lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/devel

Re: [OpenSIPS-Devel] [opensips] Revert 4584e60... based on #86 (#132)

[Răzvan Crainea]

Closed #132.

---
Reply to this email directly or view it on GitHub:
https://github.com/OpenSIPS/opensips/pull/132___
Devel mailing list
Devel@lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/devel

Re: [OpenSIPS-Devel] [opensips] Add s.index and s.rindex transformations (#125)

[Răzvan Crainea]

Merged #125.

---
Reply to this email directly or view it on GitHub:
https://github.com/OpenSIPS/opensips/pull/125___
Devel mailing list
Devel@lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/devel

Re: [OpenSIPS-Devel] [opensips] event_xmlrpc makes an invalid call when using parameters (#87)

[Răzvan Crainea]

Closed #87.

---
Reply to this email directly or view it on GitHub:
https://github.com/OpenSIPS/opensips/issues/87___
Devel mailing list
Devel@lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/devel

Re: [OpenSIPS-Devel] [opensips] db_http module doesn't match its documentation (#86)

[Răzvan Crainea]

Hi, Stéphane!

Actually, before the 4584e605f7233f68e303bd8d0a8066ab9f0e4fa2, the 
documentation was specifying that the delimiter was ';' but the code was using 
',' as delimiter. Therefore, we can consider that commit a fix.
Probably it makes more sense to use comma as the default delimiter, therefore I 
will mark this ticket as a feature request. Meanwhile, you should explicitly 
set the field_delimiter to ',' in your configuration file.

Best regards,
Răzvan

---
Reply to this email directly or view it on GitHub:
https://github.com/OpenSIPS/opensips/issues/86#issuecomment-37286385___
Devel mailing list
Devel@lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/devel

Re: [OpenSIPS-Devel] [opensips] opensips 1.9 (latest trunk) hangs when rabbitmq (using event_rabbitmq) drops connection (#81)

[Răzvan Crainea]

Any updates on this? Are you still experiencing any issues?

---
Reply to this email directly or view it on GitHub:
https://github.com/OpenSIPS/opensips/issues/81#issuecomment-37279231___
Devel mailing list
Devel@lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/devel