transport: Consider removing the negotiate step from transport trait

[lexnv]

litep2p/src/transport/mod.rs

Lines 192 to 219 in f1b4e0b

	pub(crate) trait Transport: Stream + Unpin + Send {
	/// Dial `address` and negotiate connection.
	fn dial(&mut self, connection_id: ConnectionId, address: Multiaddr) -> crate::Result<()>;
	/// Accept negotiated connection.
	fn accept(&mut self, connection_id: ConnectionId) -> crate::Result<()>;
	/// Accept pending connection.
	fn accept_pending(&mut self, connection_id: ConnectionId) -> crate::Result<()>;
	/// Reject pending connection.
	fn reject_pending(&mut self, connection_id: ConnectionId) -> crate::Result<()>;
	/// Reject negotiated connection.
	fn reject(&mut self, connection_id: ConnectionId) -> crate::Result<()>;
	/// Attempt to open connection to remote peer over one or more addresses.
	fn open(&mut self, connection_id: ConnectionId, addresses: Vec<Multiaddr>)
	-> crate::Result<()>;
	/// Negotiate opened connection.
	fn negotiate(&mut self, connection_id: ConnectionId) -> crate::Result<()>;
	/// Cancel opening connections.
	///
	/// This is a no-op for connections that have already succeeded/canceled.
	fn cancel(&mut self, connection_id: ConnectionId);
	}

Extracted from:
#495 (comment)

[burdges]

Do we have any resumption or pre-shared key protocol for when some recent connection existed? I probably doesn't matter much..

[lexnv]

Do we have any resumption or pre-shared key protocol for when some recent connection existed?

I don't believe we do at the moment. I'll have a look into this since it can improve the CPU usage even more, maybe we can extend the noise protocol in a backwards compatible way