Merge pull request #853 from libp2p/fix/autonat-constructor

fix: set the private key when constructing the autonat service

config/config.go

@@ -2,6 +2,7 @@ package config
 
 import (
 	"context"
+	"crypto/rand"
 	"fmt"
 
 	"github.com/libp2p/go-libp2p-core/connmgr"
@@ -283,6 +284,11 @@ func (cfg *Config) NewNode(ctx context.Context) (host.Host, error) {
 		return addrF(h.AllAddrs())
 	})}
 	if cfg.AutoNATService {
+		autonatPrivKey, _, err := crypto.GenerateEd25519Key(rand.Reader)
+		if err != nil {
+			return nil, err
+		}
+
 		// Pull out the pieces of the config that we _actually_ care about.
 		// Specifically, don't setup things like autorelay, listeners,
 		// identify, etc.
@@ -294,9 +300,11 @@ func (cfg *Config) NewNode(ctx context.Context) (host.Host, error) {
 			PSK:                cfg.PSK,
 			Filters:            cfg.Filters,
 			Reporter:           cfg.Reporter,
+			PeerKey:            autonatPrivKey,
 
 			Peerstore: pstoremem.NewPeerstore(),
 		}
+
 		dialer, err := autoNatCfg.makeSwarm(ctx)
 		if err != nil {
 			h.Close()

libp2p_test.go

@@ -77,6 +77,15 @@ func TestInsecure(t *testing.T) {
 	h.Close()
 }
 
+func TestAutoNATService(t *testing.T) {
+	ctx := context.Background()
+	h, err := New(ctx, EnableNATService())
+	if err != nil {
+		t.Fatal(err)
+	}
+	h.Close()
+}
+
 func TestDefaultListenAddrs(t *testing.T) {
 	ctx := context.Background()