You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficcontrol.apache.org by oc...@apache.org on 2019/10/05 01:35:48 UTC
[trafficcontrol] branch master updated: Remove atstccfg unused code
(#3960)
This is an automated email from the ASF dual-hosted git repository.
ocket8888 pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/trafficcontrol.git
The following commit(s) were added to refs/heads/master by this push:
new 0450266 Remove atstccfg unused code (#3960)
0450266 is described below
commit 04502663d0a31a42fa88750848f0c7d3339fa1e2
Author: Robert Butts <ro...@users.noreply.github.com>
AuthorDate: Fri Oct 4 19:35:40 2019 -0600
Remove atstccfg unused code (#3960)
Removes code, where remap.config was replacing __HOSTNAME__ in server
parameters, but if you trace thru the code, the only usage
of that data is to check a 'dscp_remap' parameter. The hostname
replacement is totally unused, and at worst, deceptive.
---
lib/go-atscfg/remapdotconfig.go | 4 ----
1 file changed, 4 deletions(-)
diff --git a/lib/go-atscfg/remapdotconfig.go b/lib/go-atscfg/remapdotconfig.go
index 2ac8d04..1dc0d93 100644
--- a/lib/go-atscfg/remapdotconfig.go
+++ b/lib/go-atscfg/remapdotconfig.go
@@ -168,10 +168,6 @@ func GetServerConfigRemapDotConfigForEdge(
atsMajorVersion int,
header string,
) string {
- for name, val := range serverPackageParamData {
- serverPackageParamData[name] = strings.Replace(val, "__HOSTNAME__", server.HostName+"."+server.DomainName, -1)
- }
-
textLines := []string{}
for _, ds := range dses {