Update calls to DoGeolocationLookup after API change

We've updated bitmask-core. Before the update, DoGeolocationLookup
returned string and error, now it's just an error.

pkg/vpn/bitmask.go

@@ -143,12 +143,8 @@ func Init() (*Bitmask, error) {
 			If the lookup succeeds, we save it in the config file and use it as fallback
 			the next time.
 		*/
-		countryCode, err := b.api.DoGeolocationLookup()
-		if err == nil {
-			log.Debug().
-				Str("countryCode", countryCode).
-				Msg("Successfully got country code")
-		} else {
+		err := b.api.DoGeolocationLookup()
+		if err != nil {
 			log.Warn().
 				Str("err", err.Error()).
 				Msgf("Could not do geolocation lookup")

pkg/vpn/bonafide/bonafide.go

@@ -412,6 +412,6 @@ func (b *Bonafide) GetSnowflakeCh() chan *snowflake.StatusEvent {
 	return b.snowflakeCh
 }
 
-func (b *Bonafide) DoGeolocationLookup() (string, error) {
-	return "", errors.New("DoGeolocationLookup is not supported in v3 (only implemented in bitmask-core)")
+func (b *Bonafide) DoGeolocationLookup() error {
+	return errors.New("DoGeolocationLookup is not supported in v3 (only implemented in bitmask-core)")
 }

pkg/vpn/interface.go

@@ -21,5 +21,5 @@ type apiInterface interface {
 	GetBestGateways(transport string) ([]bonafide.Gateway, error)
 	FetchAllGateways(transport string) error
 	GetSnowflakeCh() chan *snowflake.StatusEvent
-	DoGeolocationLookup() (string, error)
+	DoGeolocationLookup() error
 }

pkg/vpn/menshen/init.go

@@ -146,6 +146,6 @@ func (m *Menshen) GetSnowflakeCh() chan *snowflake.StatusEvent {
 	return nil
 }
 
-func (m *Menshen) DoGeolocationLookup() (string, error) {
+func (m *Menshen) DoGeolocationLookup() error {
 	return m.api.DoGeolocationLookup()
 }