From fd9e15e51dfb1139b19bacb81e55054579dde230 Mon Sep 17 00:00:00 2001 From: David Blacka Date: Mon, 6 Jul 2020 13:49:07 +0000 Subject: [PATCH] Remove some FIXME comments --- src/com/verisign/tat/dnssec/CaptiveValidator.java | 4 ++-- src/com/verisign/tat/dnssec/ValUtils.java | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/com/verisign/tat/dnssec/CaptiveValidator.java b/src/com/verisign/tat/dnssec/CaptiveValidator.java index 5304967..05b8d65 100644 --- a/src/com/verisign/tat/dnssec/CaptiveValidator.java +++ b/src/com/verisign/tat/dnssec/CaptiveValidator.java @@ -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); diff --git a/src/com/verisign/tat/dnssec/ValUtils.java b/src/com/verisign/tat/dnssec/ValUtils.java index 4b8d1cf..e9246f5 100644 --- a/src/com/verisign/tat/dnssec/ValUtils.java +++ b/src/com/verisign/tat/dnssec/ValUtils.java @@ -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) { -- 2.36.6