Hi,

On Thu, 2012-10-04 at 12:56 +0300, Jukka Rissanen wrote:
> +               array{string} Nameservers [readonly]
> +
> +                       The list of currently active nameservers.
> +

This is a list of nameservers that the VPN has set, they are not
necessarily the active ones as they can be overridden in ConnMan.

> +               array{string} UserRoutes
> +
> +                       The list of currently active user activated
> +                       routes.
> +

UserRoutes should be similar dicts as ServerRoute?

> +               dict ServerRoute [readonly]
> +
> +                       int ProtocolFamily
> +
> +                               Protocol family of the route. Set to 4
> +                               if IPv4 and 6 if IPv6 route.
> +
> +                       string Host
> +
> +                               The host part of the route.

Host? Or Network?

> +
> +                       string Netmask
> +
> +                               The netmask (if IPv4) or prefix length
> (if IPv6)

Preferably use just netmask, it can be supported by both versions.

> +                               of the route.
> +
> +                       string Gateway
> +
> +                               Gateway address of the route.
> +
> +                       The VPN server activated route. If there are
> multiple
> +                       routes, then each one is sent separately. 

This should come as array{dict}, since it's not clear otherwise whether
all or the last ServerRoutes are valid. This especially happens when
connecting and the ServerRoute property is updated many times in a row
when there is more than one route given by the VPN server.

Cheers,

        Patrik


_______________________________________________
connman mailing list
connman@connman.net
http://lists.connman.net/listinfo/connman

Reply via email to