-
Notifications
You must be signed in to change notification settings - Fork 12
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
[WIP] DA read range api #546
Conversation
Use dummy default private_key in dissemination app
@@ -47,6 +48,7 @@ impl Disseminate { | |||
let output = self.output.clone(); | |||
let (payload_sender, payload_rx) = tokio::sync::mpsc::unbounded_channel(); | |||
payload_sender.send(bytes).unwrap(); | |||
let private_key = PrivateKey::default(); // TODO: pass private_key via the cli args. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Uhm, what is this key used for? Shouldn't be the node itself the one to actually sign stuff?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've used this code to discuss an idea on how blob structure should look like, I'll summarize the discussion in the document. In short, this was added as proposal for blob validation during dissemination (before cert is created), but the idea was discarded.
pub trait DaProtocol { | ||
type Blob: Blob; | ||
type Auth: BlobAuth; | ||
type Attestation: Attestation; | ||
type Certificate: Certificate; | ||
type Settings: Clone; | ||
|
||
// Construct a new instance | ||
fn new(settings: Self::Settings) -> Self; | ||
/// Encode bytes into blobs | ||
fn encode<T: AsRef<[u8]>>(&self, data: T) -> Vec<Self::Blob>; | ||
fn encode<T: AsRef<[u8]>>(&self, auth: Self::Auth, data: T) -> Vec<Self::Blob>; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is part of the message itself not the blob per se, unless I am missing something.
Closing this as blob validation is not in a scope at the moment. |
A proof of concept DA api for blob retrieval by range, mostly for seeing and discussing what parts of various services need to be updated.