On Thursday 06 December 2007, Juha Heinanen wrote:
> i noticed that mwi publish caused the following error message to
> syslog:
>
> Dec 6 20:34:29 tutpro /usr/sbin/sems_openser[25383]: Handling PUBLISH
> to <sip:[EMAIL PROTECTED]> Dec 6 20:34:29 tutpro
> /usr/sbin/sems_openser[25377]: ERROR:pua_mi:mi_publ_rpl_cback: lexpire=
> 7776001
>
> looks like it is unavoidable:
>
> if(statuscode== 200)
> {
> /* extract ETag and expires */
> lexpire = ((exp_body_t*)reply->expires->parsed)->val;
> LM_ERR("lexpire= %d\n", lexpire);
> ...
>
> should LM_ERR be changed to something else or is it a real error?
It looks like someone wanted to see the value during development/testing
and forgot to remove the print. I think it would be a good idea to use
LM_INFO or LM_NOTICE for such temporary development prints, because that
way they won't cause confusion if the message is a real error message or
not.
--
Dan
_______________________________________________
Devel mailing list
[email protected]
http://lists.openser.org/cgi-bin/mailman/listinfo/devel
