You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficcontrol.apache.org by dg...@apache.org on 2019/03/15 19:37:42 UTC
[trafficcontrol] branch 3.0.x updated: Lowercase incoming domain
name to ensure case matches when attempting to find matching DNSSEC key.
This resolves #3231. (#3233) (#3409)
This is an automated email from the ASF dual-hosted git repository.
dgelinas pushed a commit to branch 3.0.x
in repository https://gitbox.apache.org/repos/asf/trafficcontrol.git
The following commit(s) were added to refs/heads/3.0.x by this push:
new 8be8a5d Lowercase incoming domain name to ensure case matches when attempting to find matching DNSSEC key. This resolves #3231. (#3233) (#3409)
8be8a5d is described below
commit 8be8a5d747b6764be15af433a89d42a729d00b75
Author: Rawlin Peters <Ra...@gmail.com>
AuthorDate: Fri Mar 15 13:37:38 2019 -0600
Lowercase incoming domain name to ensure case matches when attempting to find matching DNSSEC key. This resolves #3231. (#3233) (#3409)
(cherry picked from commit 6de51c243f26fb697f7243fed6ae45b4f35b1fbe)
---
.../cdn/traffic_control/traffic_router/core/dns/SignatureManager.java | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/traffic_router/core/src/main/java/com/comcast/cdn/traffic_control/traffic_router/core/dns/SignatureManager.java b/traffic_router/core/src/main/java/com/comcast/cdn/traffic_control/traffic_router/core/dns/SignatureManager.java
index e19a9d9..fcdc024 100644
--- a/traffic_router/core/src/main/java/com/comcast/cdn/traffic_control/traffic_router/core/dns/SignatureManager.java
+++ b/traffic_router/core/src/main/java/com/comcast/cdn/traffic_control/traffic_router/core/dns/SignatureManager.java
@@ -536,7 +536,7 @@ public final class SignatureManager {
}
private boolean isDnssecEnabled(final Name name) {
- return dnssecEnabled && keyMap.containsKey(name.toString());
+ return dnssecEnabled && keyMap.containsKey(name.toString().toLowerCase());
}
private void setDnssecEnabled(final boolean dnssecEnabled) {