tailcfg: redefine Location.CountryCode as upper case

Because other than TLDs, it's traditionally upper case: https://en.wikipedia.org/wiki/ISO_3166-1_alpha-2 Updates #cleanup Signed-off-by: Brad Fitzpatrick <bradfitz@tailscale.com>
2023-06-25 13:11:10 -07:00
1 changed files with 4 additions and 4 deletions
--- a/tailcfg/tailcfg.go
+++ b/tailcfg/tailcfg.go
@ -535,10 +535,10 @@ type Service struct {
 // Tailscale host. Location is optional and only set if
 // explicitly declared by a node.
 type Location struct {
-	Country     string `json:",omitempty"` // User friendly country name, with proper capitalization, e.g "Canada"
+	Country     string `json:",omitempty"` // User friendly country name, with proper capitalization ("Canada")
-	CountryCode string `json:",omitempty"` // ISO 3166-1 alpha-2 in lower case, e.g "ca"
+	CountryCode string `json:",omitempty"` // ISO 3166-1 alpha-2 in upper case ("CA")
-	City        string `json:",omitempty"` // User friendly city name, with proper capitalization, e.g. "Squamish"
+	City        string `json:",omitempty"` // User friendly city name, with proper capitalization ("Squamish")
-	CityCode    string `json:",omitempty"`
+	CityCode    string `json:",omitempty"` // TODO(charlotte): document
 	// Priority determines the priority an exit node is given when the
 	// location data between two or more nodes is tied.