Mail archive
alpine-devel

Re: [alpine-devel] kamailio kamctl script fixed for dbtext support

From: Ted Trask <ttrask01_at_yahoo.com>
Date: Thu, 19 Nov 2009 13:46:38 -0800 (PST)

Ok, I've downloaded the code for 1.5.3 and looked even deeper. The
problem is in the database definition, but not quite what I said before.

Here is the 'subscriber' definition for the various DB types:
scripts/db_berkeley/kamailio/subscriber:id(int)
username(str) domain(str) password(str) email_address(str) ha1(str)
ha1b(str) rpid(str)
scripts/postgres/auth_db-create.sql: email_address VARCHAR(64) DEFAULT '' NOT NULL,
scripts/mysql/auth_db-create.sql: email_address VARCHAR(64) DEFAULT '' NOT NULL,
scripts/oracle/auth_db-create.sql: email_address VARCHAR2(64) DEFAULT '',
scripts/dbtext/kamailio/subscriber:id(int,auto)
username(string) domain(string) password(string) email_address(string)
ha1(string) ha1b(string) rpid(string,null)

Ok, all of the
databases have the email_address column, most saying non-null and
default to "". The problem comes because kamctl doesn't take a
parameter for email_address and tries to set it to null. For postgres
it works just fine, because there's a default defined. Not true for
dbtext. It says non-null, but doesn't define a default. Further, it
appears the dbtext does not differentiate between "" and null. So, if
we remove the non-null restriction, it will allow it to be set to null,
which is actually "", which is what the default should be!

So,
to make this as clear as mud, the one line in
scripts/dbtext/kamailio/subscriber
(/usr/share/kamailio/dbtext/kamailio/subscriber on the device) should
be changed from:

id(int,auto) username(string) domain(string) password(string) email_address(string) ha1(string) ha1b(string) rpid(string,null)

to (adding the word 'null' in the email_address column definition):

id(int,auto)
username(string) domain(string) password(string)
email_address(string,null) ha1(string) ha1b(string) rpid(string,null)

And that should fix the problem.

Ted Trask



----- Original Message ----
From: Ted Trask <ttrask01_at_yahoo.com>
To: Francesco <francesco.colista_at_gmail.com>; alpine-devel_at_lists.alpinelinux.org
Sent: Thu, November 19, 2009 2:38:25 PM
Subject: Re: [alpine-devel] kamailio kamctl script fixed for dbtext support

After looking a little deeper at the problem, I think I found a better solution.

You need to remove "email_address(string)" from the file /usr/share/kamailio/dbtext/kamailio/subscriber so the dbtext database subscriber table no longer has the email_address field.

Ted Trask





----- Original Message ----
From: Francesco <francesco.colista_at_gmail.com>
To: alpine-devel_at_lists.alpinelinux.org
Sent: Thu, November 19, 2009 8:01:55 AM
Subject: [alpine-devel] kamailio kamctl script fixed for dbtext support

Added patch and modified APKBUILD. Increased $pkgrel=$pkgrel+1 :)

Look and review, thanks.

:: Francesco Colista ::



      


---
Unsubscribe:  alpine-devel+unsubscribe_at_lists.alpinelinux.org
Help:         alpine-devel+help_at_lists.alpinelinux.org
---
      
---
Unsubscribe:  alpine-devel+unsubscribe_at_lists.alpinelinux.org
Help:         alpine-devel+help_at_lists.alpinelinux.org
---
Received on Thu Nov 19 2009 - 13:46:38 UTC