fixing bug with new glue array

tools/constants_test.go

@@ -0,0 +1,7 @@
+package tools_test
+
+// Using default softHSM configuration. Change it if necessary.
+const p11Lib = "/usr/lib/softhsm/libsofthsm2.so" // Path used by Ubuntu Bionic Beaver
+const p11Key = "1234"
+const p11LabelRSA = "test-hsm-rsa"
+const p11LabelECDSA = "test-hsm-ecdsa"

tools/context.go

@@ -51,7 +51,6 @@ func NewContext(config *ContextConfig, log *log.Logger) (ctx *Context, err error
 		Config:        config,
 		Log:           log,
 		SignAlgorithm: algorithm,
-		Glue:          make(map[string]struct{}),
 	}
 
 	if len(config.FilePath) > 0 {
@@ -89,6 +88,9 @@ func (ctx *Context) ReadAndParseZone(updateSerial bool) error {
 	if ctx.File == nil {
 		return fmt.Errorf("no file defined on context")
 	}
+	if ctx.Glue == nil {
+		ctx.Glue = make(map[string]struct{})
+	}
 
 	rrs := make(RRArray, 0)
 

tools/pkcs11_signer_test.go

@@ -9,12 +9,6 @@ import (
 	"github.com/niclabs/dns-tools/tools"
 )
 
-// Using default softHSM configuration. Change it if necessary.
-const p11Lib = "/usr/lib/softhsm/libsofthsm2.so" // Path used by Ubuntu Bionic Beaver
-const p11Key = "1234"
-const p11LabelRSA = "test-hsm-rsa"
-const p11LabelECDSA = "test-hsm-ecdsa"
-
 func TestSession_PKCS11RSASign(t *testing.T) {
 	ctx := &tools.Context{
 		Config: &tools.ContextConfig{

tools/rr_set.go

@@ -286,12 +286,8 @@ func (ctx *Context) addNSEC3Records() error {
 		if typeMap[dns.TypeSOA] {
 			typeMap[dns.TypeNSEC3PARAM] = true
 		}
-		if !typeMap[dns.TypeDS] && !typeMap[dns.TypeDNSKEY] {
-			if ctx.Config.OptOut {
-				continue
-			} else {
-				ctx.Log.Printf("opt-out is not set and owner name %s has not a DS RR", rrSet[0].Header().Name)
-			}
+		if !typeMap[dns.TypeDS] && !typeMap[dns.TypeDNSKEY] && ctx.Config.OptOut {
+			continue
 		}
 		// Add current NSEC3 RR
 		err := nsec3list.add(rrSet[0].Header().Name, param, typeMap)