-
Notifications
You must be signed in to change notification settings - Fork 524
AVM: FIDO2 OpCodes #3514
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
Merged
Merged
AVM: FIDO2 OpCodes #3514
Changes from all commits
Commits
Show all changes
85 commits
Select commit
Hold shift + click to select a range
430552f
fixed off-by-one error in the assembler's typeDig() and improved unit…
algoidurovic 390f589
Merge branch 'master' of https://github.com/algorand/go-algorand
algoidurovic a8f4f2d
Merge branch 'algorand:master' into master
algoidurovic 6957542
Merge branch 'master' of github.com:algoidurovic/go-algorand
algoidurovic 7ea33ef
adding json_ref
shiqizng 8ba214b
added json_ref opcode, tests, docs
shiqizng 2b312d5
resolve merge conflicts
shiqizng bb5ebf3
addressing reviews
shiqizng dba9d06
review changes, added json spec doc
shiqizng 6a3bc0a
fix lint error
shiqizng d865b5f
Merge branch 'master' of https://github.com/algorand/go-algorand into…
shiqizng baf8758
license update
shiqizng eda7565
add test cases for base64 strictness
ffa08bb
changes as per reviews
shiqizng 8ade1ca
more changes
shiqizng aa3e4eb
remove extra lines
shiqizng 9b38b6d
minor refactoring
shiqizng 1ed6da1
Merge branch 'master' of https://github.com/algorand/go-algorand
algoidurovic fa78d06
adding more tests
shiqizng b0ba83a
update docs
shiqizng 93bed2f
initial implementation w/ correctness and benchmark tests
algoidurovic 0e99d67
Fixed assembler test and limited signed message size for secp256r1 to…
algoidurovic f90e810
🤞
02b60a5
lint
40365d2
skip base64_decode specific tests
16bdaf8
todo comment
b1c6c86
more legible diff
1eb7fd2
lint
f645bd4
fidoVersion instead of incosideration + assert expected errors for pr…
eadc270
more git friendly nonsense
a5e7118
merge branch 'fido2-future' of https://github.com/tzaffi/go-algorand …
algoidurovic 7a83896
Merge branch 'master' of https://github.com/algorand/go-algorand into…
623f167
Merge branch 'json_ref' of https://github.com/shiqizng/go-algorand in…
741d5d5
updated version to avoid inclusion of r1 field in v6. The new field i…
algoidurovic 2f6ab8a
fixed typo for fidoVersion
algoidurovic 3da9460
Merge branch 'master' of https://github.com/algorand/go-algorand into…
algoidurovic cfbf3cd
pass assembler_test.go with json_ref
e111e91
pass json_ref eval tests
8ad0c78
codegen_verification removed // indirect
3ffc1be
resolve merge conflicts and swap int/byte for pushint/pushbytes in no…
algoidurovic 23c1ef3
undo deletion of json_ref opcode
algoidurovic d3a771a
removed generated catchpoint files
algoidurovic f18624e
do not run secp256r1 tests until logic version exceeds fido version
algoidurovic c2e27ce
backport UnmarshallCompressed implementation to use with < go 1.15
algoidurovic 22ff844
backported elliptic.MarshalCompressed
algoidurovic a2420f6
use backported bigint.fillbytes()
algoidurovic 2adb6ab
remove secp256r1 from opcode docs
algoidurovic f1984c0
run make in logic package
algoidurovic adf6786
Merge pull request #5 from algoidurovic/secp256r1
tzaffi 6378e43
Merge branch 'master' of https://github.com/algorand/go-algorand into…
b939066
test again on circle
a2d7ffe
logicversion++
78e8818
logicversion++
e8a54ae
add another trivial globals test case
9cefd52
Merge branch 'master' of https://github.com/algorand/go-algorand into…
af9a8d8
markdoownlint
7d3fc68
Merge branch 'base64-strictness-test' into fido2-future
295f5b8
Merge branch 'master' of https://github.com/algorand/go-algorand into…
0b0bc56
bad merge
b98b352
autogen
dd41a0d
tests pass, but much is commented out...
9b7f023
tests pass but much is commented out...
68804e7
wip
71769cb
Merge branch 'master' into fido2-future
55b54dd
json_ref in the doc
dcc3515
codegen
4b79f47
jsonref should be passing
d80c72e
Update data/transactions/logic/eval_test.go
tzaffi 4d613fe
remove line
bf1c362
add optional field version parameter
e16182d
pass the ecdsa tests
44ebddf
better comment
ba700af
Update data/transactions/logic/opcodes.go
tzaffi 8a73bae
Update data/transactions/logic/fields.go
tzaffi e9d58bc
updated future to LogicSigVersion=7
0c8f34f
Merge branch 'fido2-future' of github.com:tzaffi/go-algorand into fid…
a6182dc
set vFuture not v32's LogicSigVersion
c86590f
revert formatting change
c32f4b2
fidoVersion += 1
280c7dc
better comment to trigger retry after flaky??? test failure
tzaffi d566072
Merge branch 'master' of https://github.com/algorand/go-algorand into…
80b0254
fidoVersion = 7
e5279ff
revert
27411e5
lower case and retrigger failed build due to infra
tzaffi b38c152
flaky test
tzaffi File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.