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

decide whether we want to accept empty submissions / produce empty transactions #592

Closed
bitonic opened this issue Apr 18, 2019 · 3 comments · Fixed by #3270
Closed

decide whether we want to accept empty submissions / produce empty transactions #592

bitonic opened this issue Apr 18, 2019 · 3 comments · Fixed by #3270
Labels
component/ledger Sandbox and Ledger API

Comments

@bitonic
Copy link
Contributor

bitonic commented Apr 18, 2019

@gerolf-da noted that right now we accept empty submissions, or more generally commands that generate empty transactions. this is somewhat odd since it lets people spam the ledger without really adding anything to it.

it is not a security issue since the submissions would still be authorized, but it would probably be a good idea to just reject such empty submissions, or maybe accept them but produce no transaction for them.

cc @meiersi-da @gaboraranyossy-da @dajmaki

@bitonic bitonic added the component/ledger Sandbox and Ledger API label Apr 18, 2019
@bitonic bitonic added this to the Backlog milestone Apr 18, 2019
@gabor-aranyossy
Copy link
Contributor

I am fine with rejecting it. I recall this was under consideration during the ledger api design, but I don't remember the argument defending it. @mziolekda might have it documented somewhere

@meiersi-da
Copy link
Contributor

meiersi-da commented Apr 18, 2019 via email

@mziolekda
Copy link
Contributor

I am fine with rejecting empty submissions. I don't find anything with regards to that in my notes. So if it was discussed at the time, as Gabor suggests, it must have been such low priority, that I didn't bother writing it down anywhere.

stefanobaghino-da added a commit that referenced this issue Oct 28, 2019
stefanobaghino-da added a commit that referenced this issue Oct 29, 2019
gerolf-da pushed a commit that referenced this issue Oct 31, 2019
* Disallow empty command submission

Fixes #592
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/ledger Sandbox and Ledger API
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants