Add plumbing for stream isolation

The stream ID is not yet passed to namecoind.
namecoin · Oct 18, 2019 · ed8fa68 · ed8fa68
1 parent 54d7d30
commit ed8fa68
Show file tree

Hide file tree

Showing 8 changed files with 35 additions and 23 deletions.
diff --git a/backend/backend.go b/backend/backend.go
@@ -2,7 +2,7 @@ package backend
 
 import "github.com/miekg/dns"
 import "github.com/golang/groupcache/lru"
-import "gopkg.in/hlandau/madns.v1/merr"
+import "gopkg.in/hlandau/madns.v2/merr"
 import "github.com/namecoin/ncdns/namecoin"
 import "github.com/namecoin/ncdns/util"
 import "github.com/namecoin/ncdns/ncdomain"
@@ -109,7 +109,7 @@ func convertEmail(email string) (string, error) {
 
 // Do low-level queries against an abstract zone file. This is the per-query
 // entrypoint from madns.
-func (b *Backend) Lookup(qname string) (rrs []dns.RR, err error) {
+func (b *Backend) Lookup(qname, streamIsolationID string) (rrs []dns.RR, err error) {
 	err = lookupReadyError()
 	if err != nil {
 		return
@@ -118,6 +118,7 @@ func (b *Backend) Lookup(qname string) (rrs []dns.RR, err error) {
 	btx := &btx{}
 	btx.b = b
 	btx.qname = qname
+	btx.streamIsolationID = streamIsolationID
 	return btx.Do()
 }
 
@@ -126,6 +127,8 @@ type btx struct {
 	b     *Backend
 	qname string
 
+	streamIsolationID string
+
 	subname, basename, rootname string
 }
 
@@ -269,7 +272,7 @@ func (tx *btx) doUserDomain() (rrs []dns.RR, err error) {
 		return
 	}
 
-	d, err := tx.b.getNamecoinEntry(ncname)
+	d, err := tx.b.getNamecoinEntry(ncname, tx.streamIsolationID)
 	if err != nil {
 		return nil, err
 	}
@@ -287,13 +290,13 @@ type domain struct {
 	ncv *ncdomain.Value
 }
 
-func (b *Backend) getNamecoinEntry(name string) (*domain, error) {
+func (b *Backend) getNamecoinEntry(name, streamIsolationID string) (*domain, error) {
 	d := b.getNamecoinEntryCache(name)
 	if d != nil {
 		return d, nil
 	}
 
-	d, err := b.getNamecoinEntryLL(name)
+	d, err := b.getNamecoinEntryLL(name, streamIsolationID)
 	if err != nil {
 		return nil, err
 	}
@@ -321,21 +324,21 @@ func (b *Backend) addNamecoinEntryToCache(name string, d *domain) {
 	b.cache.Add(name, d)
 }
 
-func (b *Backend) getNamecoinEntryLL(name string) (*domain, error) {
-	v, err := b.resolveName(name)
+func (b *Backend) getNamecoinEntryLL(name, streamIsolationID string) (*domain, error) {
+	v, err := b.resolveName(name, streamIsolationID)
 	if err != nil {
 		return nil, err
 	}
 
-	d, err := b.jsonToDomain(name, v)
+	d, err := b.jsonToDomain(name, v, streamIsolationID)
 	if err != nil {
 		return nil, err
 	}
 
 	return d, nil
 }
 
-func (b *Backend) resolveName(name string) (jsonValue string, err error) {
+func (b *Backend) resolveName(name, streamIsolationID string) (jsonValue string, err error) {
 	if fv, ok := b.cfg.FakeNames[name]; ok {
 		if fv == "NX" {
 			return "", merr.ErrNoSuchDomain
@@ -349,7 +352,7 @@ func (b *Backend) resolveName(name string) (jsonValue string, err error) {
 	// Namecoin JSON-RPC seem sluggish sometimes.
 	result := make(chan struct{}, 1)
 	go func() {
-		jsonValue, err = b.nc.Query(name)
+		jsonValue, err = b.nc.Query(name, streamIsolationID)
 		log.Errore(err, "failed to query namecoin")
 		result <- struct{}{}
 	}()
@@ -362,10 +365,14 @@ func (b *Backend) resolveName(name string) (jsonValue string, err error) {
 	}
 }
 
-func (b *Backend) jsonToDomain(name, jsonValue string) (*domain, error) {
+func (b *Backend) jsonToDomain(name, jsonValue, streamIsolationID string) (*domain, error) {
 	d := &domain{}
 
-	v := ncdomain.ParseValue(name, jsonValue, b.resolveExtraName, nil)
+	resolveExtraIsolated := func(n string) (string, error) {
+		return b.resolveExtraName(n, streamIsolationID)
+	}
+
+	v := ncdomain.ParseValue(name, jsonValue, resolveExtraIsolated, nil)
 	if v == nil {
 		return nil, fmt.Errorf("couldn't parse value")
 	}
@@ -375,8 +382,8 @@ func (b *Backend) jsonToDomain(name, jsonValue string) (*domain, error) {
 	return d, nil
 }
 
-func (b *Backend) resolveExtraName(name string) (jsonValue string, err error) {
-	return b.resolveName(name)
+func (b *Backend) resolveExtraName(name, streamIsolationID string) (jsonValue string, err error) {
+	return b.resolveName(name, streamIsolationID)
 }
 
 func (tx *btx) doUnderDomain(d *domain) (rrs []dns.RR, err error) {

diff --git a/namecoin/namecoin.go b/namecoin/namecoin.go
@@ -4,7 +4,7 @@ package namecoin
 import (
 	extratypes "github.com/hlandau/ncbtcjsontypes"
 	"github.com/hlandauf/btcjson"
-	"gopkg.in/hlandau/madns.v1/merr"
+	"gopkg.in/hlandau/madns.v2/merr"
 
 	"expvar"
 	"fmt"
@@ -57,9 +57,14 @@ func (nc *Conn) rpcSend(cmd btcjson.Cmd) (btcjson.Reply, error) {
 // Query the Namecoin daemon for a Namecoin domain (e.g. d/example).
 // If the domain exists, returns the value stored in Namecoin, which should be JSON.
 // Note that this will return domain data even if the domain is expired.
-func (nc *Conn) Query(name string) (v string, err error) {
+func (nc *Conn) Query(name string, streamIsolationID string) (v string, err error) {
 	cQueryCalls.Add(1)
 
+	// TODO: Pass stream isolation ID to namecoind, and remove this error
+	if streamIsolationID != "" {
+		return "", fmt.Errorf("Stream isolation ID '%s' is not yet passed to namecoind", streamIsolationID)
+	}
+
 	cmd, err := extratypes.NewNameShowCmd(newID(), name)
 	if err != nil {
 		//log.Info("NC NEWCMD ", err)

diff --git a/ncdt/ncdt.go b/ncdt/ncdt.go
@@ -41,7 +41,7 @@ func translateValue(k, v string) (string, error) {
 
 		f = os.NewFile(uintptr(n), "-")
 	} else if len(v) == 1 {
-		return conn.Query(k)
+		return conn.Query(k, "")
 	} else {
 		f, err = os.Open(v)
 	}

diff --git a/ncdumpzone/ncdumpzone.go b/ncdumpzone/ncdumpzone.go
@@ -55,7 +55,7 @@ func dumpName(item *extratypes.NameFilterItem, conn namecoin.Conn,
 	}
 
 	getNameFunc := func(k string) (string, error) {
-		return conn.Query(k)
+		return conn.Query(k, "")
 	}
 
 	var errors []error

diff --git a/server/server.go b/server/server.go
@@ -14,7 +14,7 @@ import (
 	"github.com/miekg/dns"
 	"github.com/namecoin/ncdns/backend"
 	"github.com/namecoin/ncdns/namecoin"
-	"gopkg.in/hlandau/madns.v1"
+	"gopkg.in/hlandau/madns.v2"
 )
 
 var log, Log = xlog.New("ncdns.server")

diff --git a/server/web.go b/server/web.go
@@ -140,7 +140,7 @@ func (ws *webServer) handleLookup(rw http.ResponseWriter, req *http.Request) {
 	info.JSONValue = req.FormValue("value")
 	info.Value = strings.Trim(info.JSONValue, " \t\r\n")
 	if info.Value == "" {
-		info.Value, info.ExistenceError = ws.s.namecoinConn.Query(info.NamecoinName)
+		info.Value, info.ExistenceError = ws.s.namecoinConn.Query(info.NamecoinName, "")
 		if info.ExistenceError != nil {
 			return
 		}
@@ -170,7 +170,7 @@ func (ws *webServer) handleLookup(rw http.ResponseWriter, req *http.Request) {
 }
 
 func (ws *webServer) resolveFunc(name string) (string, error) {
-	return ws.s.namecoinConn.Query(name)
+	return ws.s.namecoinConn.Query(name, "")
 }
 
 func (ws *webServer) ServeHTTP(rw http.ResponseWriter, req *http.Request) {

diff --git a/util/util.go b/util/util.go
@@ -1,7 +1,7 @@
 package util
 
 import "strings"
-import "gopkg.in/hlandau/madns.v1/merr"
+import "gopkg.in/hlandau/madns.v2/merr"
 import "fmt"
 import "regexp"
 import "net/mail"

diff --git a/util/util_test.go b/util/util_test.go
@@ -2,7 +2,7 @@ package util_test
 
 import "testing"
 import "github.com/namecoin/ncdns/util"
-import "gopkg.in/hlandau/madns.v1/merr"
+import "gopkg.in/hlandau/madns.v2/merr"
 
 type item struct {
 	input            string