go-algorand 3.4.0-beta #3613
Merged
go-algorand 3.4.0-beta #3613
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…writing the formula
removed unused field from signature fix: comment of HasValidRoot removed unused function fix: firstValid = 1 instead of firstvalid++ fix: removed uint test from wrong Test suite added tests for signer creation with round zero fix: removed unneeded unit-test fix: error in unit test with bad merkle proof inside signature added case to unit test of signer creation added test case to trim:wq
added key to the crypto library, made certain it supports signer interface. integrated it into the merklekeystore fix sanity-checks
… compactcertRounds
…ter when it is unspecified
## Summary Instead of performing a check in `SignerContext.GetVerifier`, I've added a unit test to verify that the output of the merklearray root has the same length as the Verifier object. ## Test Plan Unit test.
…me-integration ## Summary This PR adds the support for the State Proof Keys. ## Test Plan Unit tests added.
## Summary This piggybacks off the places where "session" is added to telemetry events, to additionally add a "v" key containing the version string. ## Test Plan Examine log output from tests.
## Summary Create an update path that would enable the following features: 1. Batch Verification (algorand#3031) 2. State Proof Keys (algorand#2990) 3. TEAL 6 (algorand#3397) 4. Expired Participation Keys (algorand#2924) 5. Fix rewards calculation bug (algorand#3403) ## Test Plan Unit tests added.
…ticipationExpirationTest ## Summary The state proof branch contained a change to `testExpirationAccounts`, changing the `partKeyValidityPeriod` from 10 to 150. This increase the duration of the test substantially, and (apparently) also causing the test to fail. This PR rolls back that change. ## Test Plan This is a test.
onetechnical
requested review from
bricerisingalgorand,
algobarb,
algojack and
egieseke
gmalouf
approved these changes
Feb 11, 2022
algojack
approved these changes
Feb 11, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This release requires a protocol upgrade.
This release contains a consensus protocol upgrade, which implements the following spec: https://github.com/algorandfoundation/specs/tree/85e6db1fdbdef00aa232c75199e10dc5fe9498f6
Highlights
Changelog
sortition.go. (Removed hard-coded constant insortition.go. #3558)NextRewardsState()(ledger: fixNextRewardsState()#3403)fillBytesmethod tobigIntToBytesfor lower golang version (RevertfillBytesmethod tobigIntToBytesfor lower golang version #3498)Additional Resources