provider/google: Validate VPN tunnel peer_ip

[evandbrown]

No description provided.

[phinze]

Just a style nit, but we usually use the named result parameters for these validatefuncs.

func validatePeerAddr(i interface{}, val string) (ws []string, es []error) {
  ip := net.ParseIP(i.(string))
  if ip == nil {
    es = append(es, fmt.Errorf("..."))
    return
  }
  for _, test := range invalidPeerAddrs {
    if ... {
      es = append(es, fmt.Errorf("..."))
      break // i think only one collision is possible yeah?
    }
  }
  return
}

[phinze]

Little style nit I left as an FYI - this LGTM!

[ghost]

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.