El Miércoles, 24 de Octubre de 2007, MAS escribió:
Hola,
Mi ñapa, con mis conocimientos, es la siguiente (se aceptan mejoras,sugerencias, problemas, errores, etc):
Primeramente, a modo de opinión personal, creo que es más sencillo usar alias con el módulo alias_db ya que usa una tabla "dbaliases" mucho más sencilla que la de "aliases" y tiene sus propias funciones para buscar y tal: http://www.openser.org/docs/modules/devel/alias_db.html
if ($fu=~"^sip:[1-9][0-9][0-9][0-9][0-9]@.*") { avp_db_query("select contact from aliases where username='$fU'","$avp(s:can_uri)");
¡¡ Ojito ahí, que te pueden meter un SQL injection si ponen una comilla simple en el nombre del "From" !! Yo tb lo tengo pendiente de corregir (en un caso similar), supongo que haré una substitución de caracteres prohibidos o incluso una detección y si existen a la porra el mensaje.
avp_printf("$avp(s:can_urid)","sip:$au@$fd");
Desde OpenSer 1.2.X todo eso es más "humano" en plan variables de toda la vida:
$avp(s:can_urid) = "sip:" + $au + "@" + $fd; :) y creo que tb valdría: $avp(s:can_urid) = "sip:$au@$fd"; (pero igual no por la falta de espacios).
Lo único ojo con ese "sip:" a machete, ¿y si alguien usa SIPS?
if (!avp_check("$avp(s:can_uri)","eq/$avp(s:can_urid)/g")) {
Idem:
if ( $avp(s:can_uri) !~ $avp(s:can_urid) )
sl_send_reply("403","Use From!=Authenticate ID"); return(-1); } } else if (!(check_from())) { sl_send_reply("403","Use From!=Authenticate ID"); return(-1); }
Por lo demás me parece bien, ¿has comprobado que funciona?
Saludos.