On Wednesday 02 June 2010, Iñaki Baz Castillo wrote:
The problem is that more SIP username related stuff is also case insensitive in Kamailio, i.e. the database columns. For example a user "TEST" with a location entry with username "test" woudl be retrieved because the location.username column is case insensitive.
IIRC that depends on the DB backend: MySQL is by default case insensitive, Postgresql is by default case sensitive.
Yes, this is correct. Then:
- check_from / check_to should b fixed to use case sensitive comparison.
Hi Iñaki,
this should be properly documented, as it has the potential of breaking some configurations.
- MySQL database schema should use case sensitive columns for
'username' in subscriber, location and any other tables containing an 'username' column.
I think we should think a bit more about this step, as it would be a major change in the way most people expect the database to work, which happens to be mostly mysql in our case. Add devel list as CC.
Henning
2010/6/2 Henning Westerholt henning.westerholt@1und1.de:
Yes, this is correct. Then:
- check_from / check_to should b fixed to use case sensitive comparison.
Hi Iñaki,
this should be properly documented, as it has the potential of breaking some configurations.
- MySQL database schema should use case sensitive columns for
'username' in subscriber, location and any other tables containing an 'username' column.
I think we should think a bit more about this step, as it would be a major change in the way most people expect the database to work, which happens to be mostly mysql in our case. Add devel list as CC.
Sure, it would involve an important change so it should be part of a new version and be properly documented. I will check out all the stuf that this would require (table columns, module functions and so) and will document it.
Regards.
2010/6/2 Iñaki Baz Castillo ibc@aliax.net:
- check_from / check_to should b fixed to use case sensitive comparison.
A workaround for this would be using:
if ! avp_check("$tU","eq/$au")
instead of:
if ! check_to()
On 6/3/10 12:52 PM, Iñaki Baz Castillo wrote:
2010/6/2 Iñaki Baz Castilloibc@aliax.net:
- check_from / check_to should b fixed to use case sensitive comparison.
A workaround for this would be using:
if ! avp_check("$tU","eq/$au")
instead of:
if ! check_to()
note that cmp_str/cmp_istr are availave in 1.5.x as well:
http://kamailio.org/docs/modules/1.5.x/textops.html#id2507121
Cheers, Daniel
2010/6/3 Daniel-Constantin Mierla miconda@gmail.com:
note that cmp_str/cmp_istr are availave in 1.5.x as well:
http://kamailio.org/docs/modules/1.5.x/textops.html#id2507121
Ops, I looked for them yesterday in 1.5 documentation and didn't find them :) (my fault).
Thanks a lot.
-
Daniel-Constantin Mierla -
Henning Westerholt -
Iñaki Baz Castillo