[GitHub] [trafficcontrol] rimashah25 commented on a diff in pull request #7068: Renamed rascal to tm/traffic_monitor

[GitBox <gi...@apache.org>]

rimashah25 commented on code in PR #7068:
URL: https://github.com/apache/trafficcontrol/pull/7068#discussion_r989371625


##########
traffic_ops/traffic_ops_golang/server/servers.go:
##########
@@ -856,6 +856,12 @@ func getServers(h http.Header, params map[string]string, tx *sqlx.Tx, user *auth
 `
 	}
 
+	if version.Major >= 5 {
+		if params["type"] == "RASCAL" {
+			params["type"] = "TRAFFIC_MONITOR"
+		}
+	}
+

Review Comment:
   I see your point but how do I maintain backward compatibility in terms of T3C?
   
   This change was done to ensure T3c clients and TS that monitors servers and ask for servers with type rascal, TO sends the servers with type TM.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@trafficcontrol.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org