Do not modify the config if the default browser is not Safari

Author: MatteoPologruto

certificates/certificates.go

@@ -283,9 +283,6 @@ func isExpired() (bool, error) {
 
 // PromptInstallCertsSafari prompts the user to install the HTTPS certificates if they are using Safari
 func PromptInstallCertsSafari() bool {
-	if GetDefaultBrowserName() != "Safari" {
-		return false
-	}
 	buttonPressed := utilities.UserPrompt("display dialog \"The Arduino Agent needs a local HTTPS certificate to work correctly with Safari.\nIf you use Safari, you need to install it.\" buttons {\"Do not install\", \"Install the certificate for Safari\"} default button 2 with title \"Arduino Agent: Install Certificates\"")
 	return strings.Contains(string(buttonPressed), "button returned:Install the certificate for Safari")
 }

main.go

@@ -223,7 +223,7 @@ func loop() {
 
 	// if the default browser is Safari, prompt the user to install HTTPS certificates
 	// and eventually install them
-	if runtime.GOOS == "darwin" {
+	if runtime.GOOS == "darwin" && cert.GetDefaultBrowserName() == "Safari" {
 		if exist, err := installCertsKeyExists(configPath.String()); err != nil {
 			log.Panicf("config.ini cannot be parsed: %s", err)
 		} else if !exist {
@@ -370,7 +370,7 @@ func loop() {
 	}
 
 	// check if the HTTPS certificates are expired and prompt the user to update them on macOS
-	if runtime.GOOS == "darwin" {
+	if runtime.GOOS == "darwin" && cert.GetDefaultBrowserName() == "Safari" {
 		if *installCerts {
 			if config.CertsExist() {
 				cert.PromptExpiredCerts(config.GetCertificatesDir())