From: Denis Kenzior <denkenz@gmail.com>
To: ofono@ofono.org
Subject: Re: [PATCH] gprs: Use "internet" apn for auto-created context
Date: Sun, 19 Jan 2014 21:12:57 -0600 [thread overview]
Message-ID: <52DC9439.5010109@gmail.com> (raw)
In-Reply-To: <52DBA831.20000@jolla.com>
[-- Attachment #1: Type: text/plain, Size: 1728 bytes --]
Hi Slava,
> How can you tell an empty context created by a provisioning plugin from
> an empty context created by gprs.c after all provisioning plugins have
> failed? Or from the access point edited by the user in such a way that
> it looks exactly like the empty context created in gprs.c? You can't.
> They are all identical. Empty context is not a reliable indication of a
> provisioning failure.
>
If your provisioning plugin returns bogus data, then the provisioning
plugin is broken and needs to be fixed. What exactly are you arguing?
If people start deploying plugins that return bogus / invalid
information, then the core can validate the provisioning information
just like it validates the D-Bus API.
> A better, unambiguous indication would be to not create any contexts at
> all. Actually, I was assuming that it was the case and was quite
> surprised to find out that it was ofono creating the default context.
No, it is not. All contexts can be removed by the application, this is
not an 'impossible' condition. A context with an empty APN cannot be
created programmatically.
>
> It's surely not breaking anything for Jolla. It would only improve
> chances of GPRS access to work out of the box. But of course in theory
> it may break something on some other platform. Any change, even an
> obvious bug fix can break existing behavior.
>
> Finally, about 15% of all GPRS Internet access points in the world are
> called "internet" and have no user name or password. It's by far the
> most common use case. So it's not a wild guess at all. It's real life
> experience.
And you can easily implement this behavior as a plugin if you so desire.
Regards,
-Denis
prev parent reply other threads:[~2014-01-20 3:12 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-01-16 14:43 [PATCH] gprs: Use "internet" apn for auto-created context Slava Monich
2014-01-16 17:04 ` Denis Kenzior
2014-01-16 20:52 ` Slava Monich
2014-01-17 3:43 ` Denis Kenzior
2014-01-19 10:25 ` Slava Monich
2014-01-20 3:12 ` Denis Kenzior [this message]
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=52DC9439.5010109@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.