Re: [PATCH] custom MO Parameters for smsc-http

Ok, makes sense, I'll do the mods and resend.

PS: I usually wait until there's some acceptance for the patch to  
write the userguide part, my next version will definitely come with  
his userguide patch bundled ;)
--
Alejandro Guerrieri
[email protected]



On 01/06/2009, at 22:50, Alexander Malysh wrote:

> Hi,
>
> here some comments...
>
> in generic_receive_sms you first receive all values than converts it  
> and only
> after this done check authorization. Make no sense... first check  
> auth then convert anything.
>
> +    else if (from == NULL || to == NULL || text == NULL) {
> +
> +        error(0, "HTTP[%s]: Insufficient args",
>
> why extra line?
>
> I think retmsg should also be configurable. Because not all gateways  
> will accept 'Sent'  as response.
> I think even HTTP error code for not accepted MOs/DLRs should be  
> configurable.
>
> Thanks,
> Alex
>
> P.S. And userguide part would be nice ;)
>
> Am 28.05.2009 um 23:00 schrieb Alejandro Guerrieri:
>
>> HI,
>>
>> I've finished my patch to configure the parameter names for MO on  
>> the generic http-smsc.
>>
>> To use it you just add a few entries on the smsc definition, only  
>> with the parameters you want to rename.
>>
>> e.g:
>>
>> generic-param-from = "phoneNumber"
>> generic-param-to = "shortCode"
>> generic-param-text = "message"
>>
>> More details and the patch here:
>>
>> http://www.blogalex.com/archives/171
>>
>> Of course I'm writing the userguide part if this goes forward :)
>>
>> Regards,
>> --
>> Alejandro Guerrieri
>> [email protected]
>>
>>
>