Re: SV: Ericsson Support for SMS receive

[Torgny Johansson <torgny johansson ericsson com>]

Hi Marcel,

måndagen den 21 mars 2011 17.57.30 skrev  Marcel Holtmann:
> Hi Torgny,
> 
> > CCing NetworkManager mailing list. I'm sorry that the thread below is not
> > formatted the way you like it but I'm keeping it below for reference.
> > 
> > The discussion is about sms support in ModemManager. I've started
> > implementing it a bit but ran into issues.
> > 
> > I think that a SetStorageLocation method as suggested by Eric would be
> > sane to add. And have that mirror the CPMS command as Jason suggested.
> > 
> > Should I still include the mem-type in SmsReceived signal? Or should we
> > assume that the caller takes care of setting the storage correctly? The
> > same question is valid for the SmsGet method (and delete).
> > 
> > About the SmsReceived vs Completed signals, I interpreted it as the
> > SmsReceived signal is emitted for all sms and for the final sms _both_
> > the SmsReceived and Completed signals are emitted. I don't have the code
> > available right now though, so I'll have to look at it again later.
> > 
> > Another issue is how to handle the completed flag in the SmsReceived
> > signal and the Completed signals. I suppose one have to parse the pdu to
> > be able to tell?
> 
> using the SIM card as storage for SMS is a pretty much bad idea. It will
> work sort of nice at first, but in the long run it will give you a lot
> of problems. It is limited in storage space and vulnerable to denial of
> service attacks with multipart SMS.

So do you suggest defaulting to ME without even offering the option to the 
caller? In that case, I guess having the memory type in the SmsReceived signal 
is kind of redundant...

> 
> Another problem is trying to make text mode for SMS on an AT modem work.
> That will fail since every single modem makes a mistake with text mode
> handling. Once you enter the cases with non-English character sets, you
> will be fighting wind mills. You need to use PDU mode.

Agreed, and we don't even have the option to use text mode with Ericsson 
modems. Only pdus are supported. But as the dbus interface is specified now, we 
need to parse the pdu in ModemManager. Or do you mean that that should be left 
to the caller as well?

Regards
Torgny