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

Hash anything signed #60

Closed
wants to merge 1 commit into from

Conversation

garious
Copy link
Contributor

@garious garious commented Mar 9, 2018

Only the first entry in the log is allowed to have zero hashes.

@garious
Copy link
Contributor Author

garious commented Mar 9, 2018

Alternatively, we could remove hash IDs from everything but Ticks, and have the tick hash all entries since the last tick.

@codecov-io
Copy link

Codecov Report

Merging #60 into master will not change coverage.
The diff coverage is 0%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #60   +/-   ##
=======================================
  Coverage   92.63%   92.63%           
=======================================
  Files          12       12           
  Lines         720      720           
=======================================
  Hits          667      667           
  Misses         53       53
Impacted Files Coverage Δ
src/entry.rs 93.22% <ø> (ø) ⬆️
src/event.rs 40% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 17926ff...eba71ad. Read the comment docs.

@garious
Copy link
Contributor Author

garious commented Mar 9, 2018

I don't like this fix, closing. I'd rather remove hashes than add them. #61

@garious garious closed this Mar 9, 2018
CriesofCarrots pushed a commit that referenced this pull request Feb 28, 2020
mvines referenced this pull request in mvines/solana Jun 15, 2020
mvines pushed a commit that referenced this pull request Jun 15, 2020
vkomenda pushed a commit to vkomenda/solana that referenced this pull request Aug 29, 2021
bw-solana added a commit to bw-solana/solana that referenced this pull request Mar 4, 2024
yihau pushed a commit to yihau/solana that referenced this pull request Mar 5, 2024
)

update mio to 0.8.11 (solana-labs#60)

(cherry picked from commit f591210)

Co-authored-by: Brennan <brennan.watt@anza.xyz>
yihau added a commit to yihau/solana that referenced this pull request Mar 5, 2024
willhickey pushed a commit that referenced this pull request Mar 5, 2024
tao-stones pushed a commit to tao-stones/solana that referenced this pull request Mar 8, 2024
* Revert "v1.18: update mio to 0.8.11 (backport of solana-labs#60) (solana-labs#71)"

This reverts commit 9b784a2.

* ci: ignore mio audit
willhickey pushed a commit that referenced this pull request Mar 9, 2024
update mio to 0.8.11 (#60)

(cherry picked from commit f591210)

Co-authored-by: Brennan <brennan.watt@anza.xyz>
willhickey pushed a commit that referenced this pull request Mar 9, 2024
* Revert "v1.18: update mio to 0.8.11 (backport of #60) (#71)"

This reverts commit 9b784a2.

* ci: ignore mio audit
willhickey pushed a commit that referenced this pull request Mar 9, 2024
update mio to 0.8.11 (#60)

(cherry picked from commit f591210)

Co-authored-by: Brennan <brennan.watt@anza.xyz>
willhickey pushed a commit that referenced this pull request Mar 9, 2024
* Revert "v1.18: update mio to 0.8.11 (backport of #60) (#71)"

This reverts commit 9b784a2.

* ci: ignore mio audit
segfaultdoc pushed a commit to jito-labs/solana that referenced this pull request Aug 7, 2024
mango-dee pushed a commit to blockworks-foundation/solana that referenced this pull request Dec 5, 2024
* Revert "v1.18: update mio to 0.8.11 (backport of solana-labs#60) (solana-labs#71)"

This reverts commit 9b784a2.

* ci: ignore mio audit
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants