Remove some FIXME comments
authorDavid Blacka <davidb@verisign.com>
Mon, 6 Jul 2020 13:49:07 +0000 (13:49 +0000)
committerDavid Blacka <davidb@verisign.com>
Mon, 6 Jul 2020 13:49:07 +0000 (13:49 +0000)
src/com/verisign/tat/dnssec/CaptiveValidator.java
src/com/verisign/tat/dnssec/ValUtils.java

index 5304967..05b8d65 100644 (file)
@@ -494,7 +494,7 @@ public class CaptiveValidator {
             int type = rrsets[i].getType();
 
             // The NS RRset won't be signed, but everything else
-            // should be.  FIXME: if we have an unexpected type here
+            // should be.  If we have an unexpected type here
             // with a bad signature, we will fail when we otherwise
             // might just have warned about the odd record.  Consider
             // checking the types first, then validating.
@@ -1055,7 +1055,7 @@ public class CaptiveValidator {
             }
         }
 
-        // FIXME: it is unclear if we should actually normalize our
+        // It is unclear if we should actually normalize our
         // responses Instead, maybe we should just fail if they are
         // not normal?
         message = normalize(message);
index 4b8d1cf..e9246f5 100644 (file)
@@ -131,7 +131,7 @@ public class ValUtils {
      *         null if the response isn't signed.
      */
     public Name findSigner(SMessage m) {
-        // FIXME: this used to classify the message, then look in the pertinent
+        // This used to classify the message, then look in the pertinent
         // section. Now we just find the first RRSIG in the ANSWER and AUTHORIY
         // sections.
         for (int section = Section.ANSWER; section < Section.ADDITIONAL; ++section) {