Re: [PATCH V2] Add GetKnownServices api to connaman

[Michael Nazzareno Trimarchi <michael@amarulasolutions.com>]

Hi Daniel

I will resend those patches again because seems that people needs
them. We will try to combine our work on another
try

On Thu, Oct 28, 2021 at 8:31 AM Daniel Wagner <wagi@monom.org> wrote:
>
> Hi Michael,
>
> On Wed, Oct 27, 2021 at 05:25:32PM +0000, Michael Nazzareno Trimarchi wrote:
> > Hi Daniel
> >

Michael

> > I even have the patch to remove known-networks. I have few question:
> >
> > - what should be the behavior for an ethernet on?
>
> I might not understand you question correctly, I assumed we don't
> differentiate between types. Return the complete list and let the user
> decide what to do with it.
>
> > - for wifi, it's removed and if you are connected it's disconnected is
> > that correct?
>
> I haven't tested this but I would assume nothing will happen. These
> files are all considered under control of ConnMan. So if you remove
> those while ConnMan is running, it wont notice it. Hmm, to get this
> working correctly we might need to add a RemoveKnownNetwork call. I want
> to avoid to have ConnMan monitoring the storage directory (it's fragile
> stuff and difficult to get right).
>
> If we go down the road with RemoveKnownNetwork, then I suggest the
> GetKnownNetwork call returns the object path, not the storage path.
>
> Sorry for this late enlightenment from my side. I didn't realize all the
> impacts when we started the discussion.
>
> > Apart of that:
> >
> > I'm working on a patch that allow to store network configuration using
> > the config storage
> > but I have few questions:
> >
> > - I have added autoconnect as a configuration option. I would like to
> > store networks but not have them
> > in auto connect mode. This should allow me to connect when they are
> > visible. (ethernet??)
>
> I can't remember all the details around AutoConnect. It's a difficult
> feature as it set on the first successful connect attempt. If
> configuration files are involved there are other rules.
>
> > - I have add an option to store if I want it immutable or not. The
> > stored configuration should be applied
>
> Immutable is derived from the fact if there is a config file, the
> service immutable. It doesn't really make sense to make this
> configurable.
>
> HTH,
> Daniel



-- 
Michael Nazzareno Trimarchi
Co-Founder & Chief Executive Officer
M. +39 347 913 2170
michael@amarulasolutions.com
__________________________________

Amarula Solutions BV
Joop Geesinkweg 125, 1114 AB, Amsterdam, NL
T. +31 (0)85 111 9172
info@amarulasolutions.com
www.amarulasolutions.com