From: Denis Kenzior <denkenz@gmail.com>
To: ofono@ofono.org
Subject: Re: [RFC PATCH v2 1/4] sms_util: status_report_expiration
Date: Fri, 10 Sep 2010 10:49:55 -0500 [thread overview]
Message-ID: <4C8A53A3.7000001@gmail.com> (raw)
In-Reply-To: <1284128902-16727-1-git-send-email-petteri.tikander@ixonos.com>
[-- Attachment #1: Type: text/plain, Size: 1122 bytes --]
Hi Petteri,
On 09/10/2010 09:28 AM, Petteri Tikander wrote:
> ---
> src/smsutil.c | 51 ++++++++++++++++++++++++++++++++++++++++++++++++---
> src/smsutil.h | 3 +--
> 2 files changed, 49 insertions(+), 5 deletions(-)
All four patches have been applied. Thanks :)
The only thing pending on this part is to handle the Status Report
recipient address being 'tweaked' by the network.
I've been hearing reports that some networks actually rewrite the
address (e.g. change national number to international, etc) which causes
our status reports to not match.
We can do two things:
1. Rely on message reference only. This seems dangerous
2. Fallback to a 'fuzzy'. E.g. try to match on the reference and last n
digits of the phone number, ignoring ton/npi. This can get tricky as
some national numbers can get different prefixes from the equivalent
international number, so we need to pick a nice value of 'n'.
E.g. in Australia:
+61 4 1234 5678 is the international number or:
04 1234 5678 is the national number.
Let me know if you can think of a third option.
Regards,
-Denis
next prev parent reply other threads:[~2010-09-10 15:49 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-09-10 14:28 [RFC PATCH v2 1/4] sms_util: status_report_expiration Petteri Tikander
2010-09-10 14:28 ` [RFC PATCH v2 2/4] unit: Add unit test for status report expiration Petteri Tikander
2010-09-10 14:28 ` [RFC PATCH v2 3/4] sms_util: remove unused parameter Petteri Tikander
2010-09-10 14:28 ` [RFC PATCH v2 4/4] sms_util: fix status report backuping to support uint msg_id Petteri Tikander
2010-09-10 15:49 ` Denis Kenzior [this message]
2010-09-10 17:39 ` [RFC PATCH v2 1/4] sms_util: status_report_expiration Petteri Tikander
2010-09-10 19:24 ` Aki Niemi
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4C8A53A3.7000001@gmail.com \
--to=denkenz@gmail.com \
--cc=ofono@ofono.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.