Config.is_valid_client_config needs additional checks for Client_static

[reynir]

If I remember correctly the engine will query for Ifconfig, Route_gateway and maybe more. Maybe it is a bad config, but I was missing route-gateway (I did not intend to route the traffic outside the virtual private network).

[hannesm]

Aren't some of these options only later transported (by the server) via the PUSH_NOTIFY?

[reynir]

Hmm, true. These are usually pushed by the server (but can be configured by the client). Maybe the way forward would be to not expect Ifconfig and Route_gateway to be present, and maybe the latter only when the former is present.

[hannesm]

Turns out, for the static client, these should indeed be present -- and should be checked by the is_valid_client_config.