Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow plugins to send custom messages #2101

Merged
merged 36 commits into from
Jan 6, 2021
Merged
Show file tree
Hide file tree
Changes from 23 commits
Commits
Show all changes
36 commits
Select commit Hold shift + click to select a range
30ca673
Allow plugins to send messages
shargon Nov 25, 2020
4d9c613
Any role
shargon Nov 25, 2020
0b632d2
Merge branch 'master' into plugin-msg
shargon Nov 26, 2020
f2c773b
Merge branch 'master' into plugin-msg
shargon Dec 1, 2020
ea0393c
Use a different enum
shargon Dec 7, 2020
2b126e6
Merge branch 'master' into plugin-msg
shargon Dec 7, 2020
b697823
Merge branch 'master' into plugin-msg
shargon Dec 9, 2020
08e1685
Merge branch 'master' into plugin-msg
vncoelho Dec 18, 2020
338face
Merge branch 'master' into plugin-msg
shargon Dec 21, 2020
e8650ad
Remove enum
shargon Dec 21, 2020
e7355d0
WhiteList p2p message
shargon Dec 21, 2020
0af7858
Clean
shargon Dec 21, 2020
4334d6e
Rename
shargon Dec 22, 2020
6d6bde2
Merge branch 'master' into plugin-msg
shargon Dec 22, 2020
7960e27
Rename
erikzhang Dec 22, 2020
8250aa1
Add single validators
shargon Dec 24, 2020
a8eca56
Add single validators
shargon Dec 24, 2020
9d2756a
Update Blockchain.cs
shargon Dec 24, 2020
c656f3b
IInventory
shargon Dec 24, 2020
7fd40ec
Use CreateBuilder
erikzhang Dec 27, 2020
58b5b0d
Merge branch 'master' into plugin-msg
shargon Dec 28, 2020
8338410
Merge branch 'master' into plugin-msg
shargon Dec 29, 2020
6c62a30
Add Height
shargon Dec 29, 2020
18d3e9a
Simplify
erikzhang Dec 30, 2020
16be616
Merge branch 'master' into plugin-msg
shargon Dec 30, 2020
4d8b0a2
Remove MessageType
shargon Dec 30, 2020
4c7fdaf
Merge branch 'master' into plugin-msg
shargon Dec 30, 2020
9c2a6f6
Update
shargon Dec 30, 2020
48a98a5
Fix UT
shargon Dec 30, 2020
eb41546
Rename
shargon Jan 1, 2021
5e1f684
Merge branch 'master' into plugin-msg
shargon Jan 3, 2021
98a7316
Add sender
shargon Jan 5, 2021
0b28d17
Update ExtensiblePayload.cs
erikzhang Jan 6, 2021
7f49339
Check format
erikzhang Jan 6, 2021
b6f505e
Fix UT
erikzhang Jan 6, 2021
e028b64
Merge branch 'master' into plugin-msg
erikzhang Jan 6, 2021
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
26 changes: 26 additions & 0 deletions src/neo/Ledger/Blockchain.cs
Original file line number Diff line number Diff line change
Expand Up @@ -10,10 +10,12 @@
using Neo.Persistence;
using Neo.Plugins;
using Neo.SmartContract;
using Neo.SmartContract.Native;
using Neo.VM;
using System;
using System.Collections.Concurrent;
using System.Collections.Generic;
using System.Collections.Immutable;
using System.Linq;
using System.Threading;

Expand Down Expand Up @@ -66,6 +68,7 @@ public class RelayResult { public IInventory Inventory; public VerifyResult Resu
private readonly Dictionary<uint, UnverifiedBlocksList> block_cache_unverified = new Dictionary<uint, UnverifiedBlocksList>();
internal readonly RelayCache RelayCache = new RelayCache(100);
private SnapshotView currentSnapshot;
private ImmutableHashSet<UInt160> extensibleWitnessWhiteList;

public IStore Store { get; }
public ReadOnlyView View { get; }
Expand Down Expand Up @@ -517,6 +520,29 @@ private void SendRelayResult(IInventory inventory, VerifyResult result)
private void UpdateCurrentSnapshot()
{
Interlocked.Exchange(ref currentSnapshot, GetSnapshot())?.Dispose();
var builder = ImmutableHashSet.CreateBuilder<UInt160>();
builder.Add(NativeContract.NEO.GetCommitteeAddress(currentSnapshot));
var validators = NativeContract.NEO.GetNextBlockValidators(currentSnapshot);
builder.Add(GetConsensusAddress(validators));
builder.UnionWith(validators.Select(u => Contract.CreateSignatureRedeemScript(u).ToScriptHash()));
var oracles = NativeContract.RoleManagement.GetDesignatedByRole(currentSnapshot, Role.Oracle, currentSnapshot.Height);
if (oracles.Length > 0)
{
builder.Add(GetConsensusAddress(oracles));
builder.UnionWith(oracles.Select(u => Contract.CreateSignatureRedeemScript(u).ToScriptHash()));
}
var stateValidators = NativeContract.RoleManagement.GetDesignatedByRole(currentSnapshot, Role.StateValidator, currentSnapshot.Height);
if (stateValidators.Length > 0)
{
builder.Add(GetConsensusAddress(stateValidators));
builder.UnionWith(stateValidators.Select(u => Contract.CreateSignatureRedeemScript(u).ToScriptHash()));
}
extensibleWitnessWhiteList = builder.ToImmutable();
}

internal bool IsExtensibleWitnessWhiteListed(UInt160 address)
{
return extensibleWitnessWhiteList.Contains(address);
}
}

Expand Down
1 change: 1 addition & 0 deletions src/neo/Network/P2P/Message.cs
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,7 @@ public static Message Create(MessageCommand command, ISerializable payload = nul
bool tryCompression =
command == MessageCommand.Block ||
command == MessageCommand.Consensus ||
command == MessageCommand.Extensible ||
command == MessageCommand.Transaction ||
command == MessageCommand.Headers ||
command == MessageCommand.Addr ||
Expand Down
2 changes: 2 additions & 0 deletions src/neo/Network/P2P/MessageCommand.cs
Original file line number Diff line number Diff line change
Expand Up @@ -40,6 +40,8 @@ public enum MessageCommand : byte
Block = 0x2c,
[ReflectionCache(typeof(ConsensusPayload))]
Consensus = 0x2d,
[ReflectionCache(typeof(ExtensiblePayload))]
Extensible = 0x2e,
Reject = 0x2f,

//SPV protocol
Expand Down
98 changes: 98 additions & 0 deletions src/neo/Network/P2P/Payloads/ExtensiblePayload.cs
Original file line number Diff line number Diff line change
@@ -0,0 +1,98 @@
using Neo.IO;
using Neo.Ledger;
using Neo.Persistence;
using Neo.SmartContract;
using System;
using System.IO;

namespace Neo.Network.P2P.Payloads
{
public class ExtensiblePayload : IInventory
erikzhang marked this conversation as resolved.
Show resolved Hide resolved
{
public string Receiver;
shargon marked this conversation as resolved.
Show resolved Hide resolved
public byte MessageType;
public uint ValidBlockStart;
public uint ValidBlockEnd;
public byte[] Data;
public Witness Witness;

private UInt256 _hash = null;
public UInt256 Hash
{
get
{
if (_hash == null)
{
_hash = this.CalculateHash();
}
return _hash;
}
}

InventoryType IInventory.InventoryType => InventoryType.Extensible;

public int Size =>
Receiver.GetVarSize() + //Receiver
sizeof(byte) + //MessageType
sizeof(uint) + //ValidBlockStart
sizeof(uint) + //ValidBlockEnd
Data.GetVarSize() + //Data
1 + Witness.Size; //Witness

Witness[] IVerifiable.Witnesses
{
get
{
return new[] { Witness };
}
set
{
if (value.Length != 1) throw new ArgumentException();
Witness = value[0];
}
}

void ISerializable.Deserialize(BinaryReader reader)
{
((IVerifiable)this).DeserializeUnsigned(reader);
if (reader.ReadByte() != 1) throw new FormatException();
Witness = reader.ReadSerializable<Witness>();
}

void IVerifiable.DeserializeUnsigned(BinaryReader reader)
{
Receiver = reader.ReadVarString(32);
MessageType = reader.ReadByte();
ValidBlockStart = reader.ReadUInt32();
ValidBlockEnd = reader.ReadUInt32();
Data = reader.ReadVarBytes(ushort.MaxValue);
}

UInt160[] IVerifiable.GetScriptHashesForVerifying(StoreView snapshot)
{
return new[] { Witness.ScriptHash }; // This address should be checked by consumer
}
erikzhang marked this conversation as resolved.
Show resolved Hide resolved

void ISerializable.Serialize(BinaryWriter writer)
{
((IVerifiable)this).SerializeUnsigned(writer);
writer.Write((byte)1); writer.Write(Witness);
}

void IVerifiable.SerializeUnsigned(BinaryWriter writer)
{
writer.WriteVarString(Receiver);
writer.Write(MessageType);
writer.Write(ValidBlockStart);
writer.Write(ValidBlockEnd);
writer.WriteVarBytes(Data);
}

public bool Verify(StoreView snapshot)
{
if (snapshot.PersistingBlock.Index < ValidBlockStart || snapshot.PersistingBlock.Index > ValidBlockEnd) return false;
if (!Blockchain.Singleton.IsExtensibleWitnessWhiteListed(Witness.ScriptHash)) return false;
return this.VerifyWitnesses(snapshot, 0_02000000);
}
}
}
1 change: 1 addition & 0 deletions src/neo/Network/P2P/Payloads/InventoryType.cs
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ public enum InventoryType : byte
{
TX = MessageCommand.Transaction,
Block = MessageCommand.Block,
Extensible = MessageCommand.Extensible,
Consensus = MessageCommand.Consensus
}
}
9 changes: 9 additions & 0 deletions src/neo/Network/P2P/RemoteNode.ProtocolHandler.cs
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,12 @@ protected override UInt256 GetKeyForItem((UInt256, DateTime) item)

private void OnMessage(Message msg)
{
if (msg.Command == MessageCommand.Extensible)
{
var payload = (ExtensiblePayload)msg.Payload;
if (!knownHashes.Add(payload.Hash)) return;
if (!payload.Verify(Blockchain.Singleton.GetSnapshot())) return;
}
erikzhang marked this conversation as resolved.
Show resolved Hide resolved
foreach (IP2PPlugin plugin in Plugin.P2PPlugins)
if (!plugin.OnP2PMessage(msg))
return;
Expand Down Expand Up @@ -67,6 +73,9 @@ private void OnMessage(Message msg)
case MessageCommand.Consensus:
OnInventoryReceived((ConsensusPayload)msg.Payload);
break;
case MessageCommand.Extensible:
OnInventoryReceived((ExtensiblePayload)msg.Payload);
break;
case MessageCommand.FilterAdd:
OnFilterAddMessageReceived((FilterAddPayload)msg.Payload);
break;
Expand Down
2 changes: 2 additions & 0 deletions src/neo/Network/P2P/RemoteNode.cs
Original file line number Diff line number Diff line change
Expand Up @@ -87,6 +87,7 @@ private void EnqueueMessage(Message message)
{
case MessageCommand.Alert:
case MessageCommand.Consensus:
case MessageCommand.Extensible:
case MessageCommand.FilterAdd:
case MessageCommand.FilterClear:
case MessageCommand.FilterLoad:
Expand Down Expand Up @@ -223,6 +224,7 @@ internal protected override bool IsHighPriority(object message)
switch (msg.Command)
{
case MessageCommand.Consensus:
case MessageCommand.Extensible:
case MessageCommand.FilterAdd:
case MessageCommand.FilterClear:
case MessageCommand.FilterLoad:
Expand Down
2 changes: 1 addition & 1 deletion src/neo/Network/P2P/TaskManager.cs
Original file line number Diff line number Diff line change
Expand Up @@ -342,7 +342,7 @@ internal protected override bool IsHighPriority(object message)
case TaskManager.RestartTasks _:
return true;
case TaskManager.NewTasks tasks:
if (tasks.Payload.Type == InventoryType.Block || tasks.Payload.Type == InventoryType.Consensus)
if (tasks.Payload.Type == InventoryType.Block || tasks.Payload.Type == InventoryType.Consensus || tasks.Payload.Type == InventoryType.Extensible)
return true;
return false;
default:
Expand Down
44 changes: 44 additions & 0 deletions tests/neo.UnitTests/Network/P2P/Payloads/UT_ExtensiblePayload.cs
Original file line number Diff line number Diff line change
@@ -0,0 +1,44 @@
using FluentAssertions;
using Microsoft.VisualStudio.TestTools.UnitTesting;
using Neo.IO;
using Neo.Network.P2P.Payloads;

namespace Neo.UnitTests.Network.P2P.Payloads
{
[TestClass]
public class UT_ExtensiblePayload
{
[TestMethod]
public void Size_Get()
{
var test = new ExtensiblePayload()
{
Receiver = "123",
Data = new byte[] { 1, 2, 3 },
Witness = new Witness() { InvocationScript = new byte[] { 3, 5, 6 }, VerificationScript = new byte[0] }
};
test.Size.Should().Be(23);
}

[TestMethod]
public void DeserializeAndSerialize()
{
var test = new ExtensiblePayload()
{
Receiver = "123",
MessageType = 123,
ValidBlockEnd = 456,
ValidBlockStart = 789,
Data = new byte[] { 1, 2, 3 },
Witness = new Witness() { InvocationScript = new byte[] { 3, 5, 6 }, VerificationScript = new byte[0] }
};
var clone = test.ToArray().AsSerializable<ExtensiblePayload>();

Assert.AreEqual(test.Hash, clone.Hash);
Assert.AreEqual(test.ValidBlockStart, clone.ValidBlockStart);
Assert.AreEqual(test.ValidBlockEnd, clone.ValidBlockEnd);
Assert.AreEqual(test.Receiver, clone.Receiver);
Assert.AreEqual(test.MessageType, clone.MessageType);
}
}
}