MSU-CS4360-JDNS / JDNSS

A leaf DNS server in Java
1 stars 1 forks source link

NSEC getBytes changes #41

Closed wwhitecar closed 6 years ago

wwhitecar commented 6 years ago

I think nsec.getBytes is returning the correct information when called. Also some minor changes to other RR records. Some other minor changes here and there, but no big changes outside nsec.getbytes.

coveralls commented 6 years ago

Pull Request Test Coverage Report for Build 84


Changes Missing Coverage Covered Lines Changed/Added Lines %
src/main/java/edu/msudenver/cs/jdnss/Parser.java 0 2 0.0%
src/main/java/edu/msudenver/cs/jdnss/RR.java 32 66 48.48%
src/main/java/edu/msudenver/cs/jdnss/Response.java 5 40 12.5%
<!-- Total: 37 108 34.26% -->
Files with Coverage Reduction New Missed Lines %
src/main/java/edu/msudenver/cs/jdnss/Response.java 1 44.78%
src/main/java/edu/msudenver/cs/jdnss/RR.java 2 46.47%
<!-- Total: 3 -->
Totals Coverage Status
Change from base Build 79: 22.5%
Covered Lines: 1031
Relevant Lines: 1958

💛 - Coveralls
wwhitecar commented 6 years ago

Made changes to DNSKEY RR to return the correct get Bytes. Also when querying for the DNSKEY it will return what I think is correct, but will need to verify with 4035 that it is indeed behaving correctly. Dnssec system test is no longer passing which is to be expected since changes to dnssec is not complete.