Browse Source

Fix: don't select TLS profile for HTTP meeks
- tls_profile in metrics was populated with unused selection

Rod Hynes 7 years ago
parent
commit
1aa2ec9294
1 changed files with 9 additions and 6 deletions
  1. 9 6
      psiphon/tunnel.go

+ 9 - 6
psiphon/tunnel.go

@@ -731,12 +731,15 @@ func initMeekConfig(
 	}
 
 	// Pin the TLS profile for the entire meek connection.
-	selectedTLSProfile := SelectTLSProfile(
-		config.clientParameters,
-		config.UseIndistinguishableTLS,
-		useObfuscatedSessionTickets,
-		true,
-		config.TrustedCACertificatesFilename != "")
+	selectedTLSProfile := ""
+	if protocol.TunnelProtocolUsesMeekHTTPS(selectedProtocol) {
+		selectedTLSProfile = SelectTLSProfile(
+			config.clientParameters,
+			config.UseIndistinguishableTLS,
+			useObfuscatedSessionTickets,
+			true,
+			config.TrustedCACertificatesFilename != "")
+	}
 
 	return &MeekConfig{
 		ClientParameters:              config.clientParameters,