Am 03.06.2014 14:56, schrieb Alexander Malysh:
Hi Stipe,

this chunk -1:

+    /* PDU allowed payload size is 64K, so change max SMS octet value */
+    if (cfg_get_bool(&smpp->message_payload, grp, 
octstr_imm("message-payload")) == 1)
+        conn->max_sms_octets = 65536;

I would prefer that user set max-sms-octets in the config because this will 
break many installations and this is even wrong
because if user set max-sms-octets in the config you just overwrite it here and 
user has no chance to change it.

ok, the last reason seems reasonable for me, I agree.

But if we don't set the max_sms_octets, then the abstraction layer will pre-chunk any SMS into GSM PDU length sized chunks and hence in the SMPP layer we can't send anything bigger then 140 octets that uses .message_payload. This doesn't make sense either.

So, we would need to check IF conn->max_sms_octets != default, then we know user has individually set an alternative value and stick to that value, otherwise we DO set to 64K for the sake of getting the full length supported of the PDU field.

Stipe

--
-------------------------------------------------------------------
Kölner Landstrasse 419
40589 Düsseldorf, NRW, Germany

tolj.org system architecture      Kannel Software Foundation (KSF)
http://www.tolj.org/              http://www.kannel.org/

mailto:st_{at}_tolj.org           mailto:stolj_{at}_kannel.org
-------------------------------------------------------------------

Reply via email to