On Mon, Aug 14, 2017 at 8:37 AM, David Woodhouse <dwmw2 at infradead.org> wrote:
>
> On Sat, 2017-05-20 at 15:43 -0700, Daniel Lenski wrote:
> >
> > diff --git a/openconnect.h b/openconnect.h
> > index e97dacb..06bf48e 100644
> > --- a/openconnect.h
> > +++ b/openconnect.h
> > @@ -208,6 +208,8 @@ struct oc_vpn_proto {
> >
> > #define OC_FORM_OPT_IGNORE 0x0001
> > #define OC_FORM_OPT_NUMERIC 0x0002
> > +#define OC_FORM_OPT_FILL_USERNAME 0x0004
> > +#define OC_FORM_OPT_FILL_PASSWORD 0x0008
>
> Hm... that's a public API, and you're using it for something which is
> purely internal. That special-casing for 'username' and 'password'
> fields is something that only openconnect(8) / main.c does.
I see what you mean, but I was thinking that the NM GUI or other GUIs
could *also* use these to receive hints from the server about the
correct function of the form fields, rather than (re-)implementing
their own special-casing.
Not a good idea?
-Dan
