Re: [PATCH] get the "foreign" (smsc-provided) message_id
Alejandro Guerrieri <[email protected]>
| Newsgroups | gmane.comp.mobile.kannel.devel |
|---|---|
| Message-ID | <[email protected]> |
+1 It should be noted on ChangeLog, of course (it may break things for people parsing the logs). Regards, -- Alejandro Guerrieri [email protected] El 01/12/2008, a las 08:32 a.m., Alexander Malysh escribió: > Hi, > > should we add it to "regular" access log as [FID:%s] ? > IMO it should be added because user want to see this id without any > additional configuration. > > Thanks, > Alex > > Alejandro Guerrieri schrieb: >> Hi, >> Attached is the patch to provide the SMSC's message_id as %F. >> As Alex suggested, the parameter is now added into dlr_add(), thus >> this should work for all SMSC's supporting the parameter. >> I had to modify dlr_add()'s prototype and remove msg's "const" >> attribute to be able to modify it inside the function of course. >> Regards, >> -- >> Alejandro Guerrieri >> [email protected] > >