Skip to content

Commit 61333bb

Browse files
authored
Merge pull request #115 from Pier-Two/fix-discovery
Fix discovery
2 parents 89ade1c + dd07560 commit 61333bb

34 files changed

Lines changed: 587 additions & 435 deletions

src/Lantern.Discv5.Enr/EnrBuilder.cs

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -4,9 +4,9 @@ namespace Lantern.Discv5.Enr;
44

55
public class EnrBuilder
66
{
7-
private readonly Dictionary<string, IEntry> _entries = new();
8-
private IIdentityVerifier _verifier;
9-
private IIdentitySigner _signer;
7+
private readonly Dictionary<string, IEntry> _entries = [];
8+
private IIdentityVerifier? _verifier;
9+
private IIdentitySigner? _signer;
1010

1111
public EnrBuilder WithIdentityScheme(IIdentityVerifier verifier, IIdentitySigner signer)
1212
{
@@ -25,9 +25,9 @@ public EnrBuilder WithEntry(string key, IEntry? entry)
2525

2626
public Enr Build()
2727
{
28-
if (_signer == null)
28+
if (_verifier == null)
2929
{
30-
throw new InvalidOperationException("Signer must be set before building the EnrRecord.");
30+
throw new InvalidOperationException("Verifier must be set before building the EnrRecord.");
3131
}
3232

3333
var enrRecord = new Enr(_entries, _verifier, _signer);
@@ -36,4 +36,4 @@ public Enr Build()
3636

3737
return enrRecord;
3838
}
39-
}
39+
}

src/Lantern.Discv5.Enr/EnrFactory.cs

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -26,8 +26,6 @@ public Enr CreateFromBytes(byte[] bytes, IIdentityVerifier verifier)
2626

2727
public Enr[] CreateFromMultipleEnrList(ReadOnlySpan<Rlp.Rlp> enrs, IIdentityVerifier verifier)
2828
{
29-
if (enrs == null) throw new ArgumentNullException(nameof(enrs));
30-
3129
Enr[] result = new Enr[enrs.Length];
3230

3331
for (int i = 0; i < enrs.Length; i++)
@@ -59,4 +57,4 @@ public Enr CreateFromRlp(Rlp.Rlp enrRlp, IIdentityVerifier verifier)
5957

6058
return enrRecord;
6159
}
62-
}
60+
}

0 commit comments

Comments
 (0)