Remove serverRegion paramater from NoticeActiveTunnel

The parameter will be reported by `NoticeConnectedServerRegion`
Psiphon-Labs · Jun 6, 2024 · 50dfa02 · 50dfa02
1 parent a2cb7dc
commit 50dfa02
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/psiphon/controller.go b/psiphon/controller.go
@@ -1001,8 +1001,7 @@ loop:
 			NoticeActiveTunnel(
 				connectedTunnel.dialParams.ServerEntry.GetDiagnosticID(),
 				connectedTunnel.dialParams.TunnelProtocol,
-				connectedTunnel.dialParams.ServerEntry.SupportsSSHAPIRequests(),
-				connectedTunnel.dialParams.ServerEntry.Region)
+				connectedTunnel.dialParams.ServerEntry.SupportsSSHAPIRequests())
 
 			NoticeConnectedServerRegion(connectedTunnel.dialParams.ServerEntry.Region)
 

diff --git a/psiphon/notice.go b/psiphon/notice.go
@@ -684,13 +684,12 @@ func NoticeRequestedTactics(dialParams *DialParameters) {
 }
 
 // NoticeActiveTunnel is a successful connection that is used as an active tunnel for port forwarding
-func NoticeActiveTunnel(diagnosticID, protocol string, isTCS bool, serverRegion string) {
+func NoticeActiveTunnel(diagnosticID, protocol string, isTCS bool) {
 	singletonNoticeLogger.outputNotice(
 		"ActiveTunnel", noticeIsDiagnostic,
 		"diagnosticID", diagnosticID,
 		"protocol", protocol,
-		"isTCS", isTCS,
-		"serverRegion", serverRegion)
+		"isTCS", isTCS)
 }
 
 // NoticeConnectedServerRegion reports the region of the connected server