Skip to content

Commit 9362c39

Browse files
authored
Merge pull request #2170 from gravitl/GRA-1493/dns-key
GRA-1493: fixed hard coded dns key
2 parents ec5cb01 + 1621c27 commit 9362c39

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

logic/security.go

+1-1
Original file line numberDiff line numberDiff line change
@@ -187,7 +187,7 @@ func authenticateDNSToken(tokenString string) bool {
187187
if len(tokens) < 2 {
188188
return false
189189
}
190-
return tokens[1] == servercfg.GetDNSKey()
190+
return len(servercfg.GetDNSKey()) > 0 && tokens[1] == servercfg.GetDNSKey()
191191
}
192192

193193
func ContinueIfUserMatch(next http.Handler) http.HandlerFunc {

servercfg/serverconf.go

+1-1
Original file line numberDiff line numberDiff line change
@@ -277,7 +277,7 @@ func GetMasterKey() string {
277277

278278
// GetDNSKey - gets the configured dns key of server
279279
func GetDNSKey() string {
280-
key := "secretkey"
280+
key := ""
281281
if os.Getenv("DNS_KEY") != "" {
282282
key = os.Getenv("DNS_KEY")
283283
} else if config.Config.Server.DNSKey != "" {

0 commit comments

Comments
 (0)