-
Notifications
You must be signed in to change notification settings - Fork 280
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
[master] v1 metamask connect to isolated server + testnet #2820
Merged
Merged
Changes from 68 commits
Commits
Show all changes
75 commits
Select commit
Hold shift + click to select a range
00e5606
v1 metamask connect - try testnet, too
n-hutton 26f2440
fixup
n-hutton 676d6a9
bump travis/jenkins
n-hutton ef23f54
tidying
n-hutton 31cf230
add debug for testnet
n-hutton 86ec18e
add killer rpc call for debug purpose
n-hutton cd3b4af
hacks
n-hutton b8639f8
Merge branch 'master' into feature/evm_rpc
n-hutton 2d2faf0
fix unfortunate error
n-hutton 3052f3b
WIP
n-hutton afa4c22
testnet
n-hutton 878768d
fixme
n-hutton f6dc7d3
about to switch TX way
n-hutton 3d23a94
fix versioning
n-hutton 05c9349
fixes
n-hutton 7f19390
fixes
n-hutton b4bc76c
bump and turn off travis
n-hutton 56b53af
try more stuff
n-hutton f75267b
fix balance
n-hutton abbf7bd
fix balance querying
n-hutton e7af88a
WIP
n-hutton 989125c
thing thing
n-hutton 46af9a6
WORKING FIRST VERSION!
n-hutton 7ca3ecb
seems to be working... try for another testnet deployment
n-hutton 18ca008
another testnet deployment
n-hutton f78ebec
bump
n-hutton f09aaa8
add missing cmake file
n-hutton ecac377
add other file
n-hutton d2360c9
fix bad build
n-hutton cbf1da4
do not strip symbols
n-hutton 544964b
Add debugging
n-hutton 0f0e8f0
add more debug
n-hutton 4a1a0d1
fixes
n-hutton b0b32cb
more fixes
n-hutton d97645c
add new eth library
n-hutton 2e71a66
add additional debug
n-hutton a145230
bump it
n-hutton 30caf65
add libEth
n-hutton 6d3110a
ignore more files
n-hutton cb0ba1a
update schnorr...
n-hutton 37ae6df
does this change the build materially?
n-hutton 0538893
revert schnorr???
n-hutton c222b3c
fix hopefullyQ
n-hutton 4607714
bump gitignore further
n-hutton 4646afa
cleanup
n-hutton a89b2ae
fix build errors
n-hutton e90857b
tidy
n-hutton 042ccd2
fix errors...
n-hutton 21e3c06
attempt at recovery from corrupted git file
n-hutton 6914897
fixes...
n-hutton 0d9c546
transfer demo still works...
n-hutton 780df7c
cleanup
n-hutton b5164a9
initial commit add python test file
059349b
Merge branch 'feature/evm_rpc' of github.com:Zilliqa/Zilliqa into fea…
n-hutton 0f17863
v1 test
f0d9538
cleanup
n-hutton 33ac463
Merge branch 'feature/evm_rpc' of github.com:Zilliqa/Zilliqa into fea…
n-hutton 8b0253b
fixed balance issues
n-hutton 35ce4fe
tests seemingly added
n-hutton e2d02df
fix build...
n-hutton 6d062d7
sigh
n-hutton a080c5c
format fix
n-hutton 9b25bac
include linting
n-hutton cb97c00
fix build error...
n-hutton 525bddd
clang tidy fix
n-hutton 1d9c7f2
sigh
n-hutton 7601a21
fix test
n-hutton 2904777
fix another test
n-hutton 21a5a34
merge to master
n-hutton 1139ff3
fix build
n-hutton b019d75
Merge branch 'master' into feature/evm_rpc
n-hutton 152d26e
pr feedback except for unit tests
af1b70d
add unit tests
n-hutton dd8a036
style...
n-hutton 109df60
more style
n-hutton 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 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 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
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -45,6 +45,8 @@ find_package(g3logger CONFIG REQUIRED) | |
|
||
include(InstallCryptoUtils) | ||
|
||
include(ProjectSecp256k1) | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. external library from a bitcoin-core fork |
||
|
||
include(InstallMongo) | ||
find_package(mongocxx CONFIG REQUIRED) | ||
|
||
|
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
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,39 @@ | ||
include(ExternalProject) | ||
|
||
if (MSVC) | ||
set(_only_release_configuration -DCMAKE_CONFIGURATION_TYPES=Release) | ||
set(_overwrite_install_command INSTALL_COMMAND cmake --build <BINARY_DIR> --config Release --target install) | ||
endif() | ||
|
||
set(prefix "${CMAKE_BINARY_DIR}/deps") | ||
set(SECP256K1_LIBRARY "${prefix}/lib/${CMAKE_STATIC_LIBRARY_PREFIX}secp256k1${CMAKE_STATIC_LIBRARY_SUFFIX}") | ||
set(SECP256K1_INCLUDE_DIR "${prefix}/include") | ||
|
||
ExternalProject_Add( | ||
secp256k1 | ||
PREFIX "${prefix}" | ||
DOWNLOAD_NAME secp256k1-ac8ccf29.tar.gz | ||
DOWNLOAD_NO_PROGRESS 1 | ||
URL https://github.com/chfast/secp256k1/archive/ac8ccf29b8c6b2b793bc734661ce43d1f952977a.tar.gz | ||
URL_HASH SHA256=02f8f05c9e9d2badc91be8e229a07ad5e4984c1e77193d6b00e549df129e7c3a | ||
PATCH_COMMAND ${CMAKE_COMMAND} -E copy_if_different | ||
${CMAKE_CURRENT_LIST_DIR}/secp256k1/CMakeLists.txt <SOURCE_DIR> | ||
CMAKE_ARGS -DCMAKE_INSTALL_PREFIX=<INSTALL_DIR> | ||
-DCMAKE_POSITION_INDEPENDENT_CODE=${BUILD_SHARED_LIBS} | ||
-DCMAKE_C_COMPILER=${CMAKE_C_COMPILER} | ||
-DCMAKE_CXX_COMPILER=${CMAKE_CXX_COMPILER} | ||
${_only_release_configuration} | ||
LOG_CONFIGURE 1 | ||
BUILD_COMMAND "" | ||
${_overwrite_install_command} | ||
LOG_INSTALL 1 | ||
BUILD_BYPRODUCTS "${SECP256K1_LIBRARY}" | ||
) | ||
|
||
# Create imported library | ||
add_library(Secp256k1 STATIC IMPORTED) | ||
file(MAKE_DIRECTORY "${SECP256K1_INCLUDE_DIR}") # Must exist. | ||
set_property(TARGET Secp256k1 PROPERTY IMPORTED_CONFIGURATIONS Release) | ||
set_property(TARGET Secp256k1 PROPERTY IMPORTED_LOCATION_RELEASE "${SECP256K1_LIBRARY}") | ||
set_property(TARGET Secp256k1 PROPERTY INTERFACE_INCLUDE_DIRECTORIES "${SECP256K1_INCLUDE_DIR}") | ||
add_dependencies(Secp256k1 secp256k1) |
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,38 @@ | ||
# This CMake config file for secp256k1 project from https://github.com/bitcoin-core/secp256k1 | ||
# | ||
# The secp256k1 project has been configured following official docs with following options: | ||
# | ||
# ./configure --disable-shared --disable-tests --disable-coverage --disable-openssl-tests --disable-exhaustive-tests --disable-jni --with-bignum=no --with-field=64bit --with-scalar=64bit --with-asm=no | ||
# | ||
# Build static context: | ||
# make src/ecmult_static_context.h | ||
# | ||
# Copy src/ecmult_static_context.h and src/libsecp256k1-config.h | ||
# | ||
# Copy CFLAGS from Makefile to COMPILE_OPTIONS. | ||
|
||
cmake_minimum_required(VERSION 3.4) | ||
project(secp256k1 LANGUAGES C) | ||
|
||
set(COMMON_COMPILE_FLAGS ENABLE_MODULE_RECOVERY ENABLE_MODULE_ECDH USE_ECMULT_STATIC_PRECOMPUTATION USE_FIELD_INV_BUILTIN USE_NUM_NONE USE_SCALAR_INV_BUILTIN) | ||
if (MSVC) | ||
set(COMPILE_FLAGS USE_FIELD_10X26 USE_SCALAR_8X32) | ||
set(COMPILE_OPTIONS "") | ||
else() | ||
set(COMPILE_FLAGS USE_FIELD_5X52 USE_SCALAR_4X64 HAVE_BUILTIN_EXPECT HAVE___INT128) | ||
set(COMPILE_OPTIONS -O3 -W -std=c89 -pedantic -Wall -Wextra -Wcast-align -Wnested-externs -Wshadow -Wstrict-prototypes -Wno-unused-function -Wno-long-long -Wno-overlength-strings -fvisibility=hidden) | ||
endif() | ||
|
||
add_executable(gen_context src/gen_context.c) | ||
target_include_directories(gen_context PRIVATE ${CMAKE_SOURCE_DIR}) | ||
|
||
add_custom_target(ecmult_static_context gen_context WORKING_DIRECTORY ${CMAKE_SOURCE_DIR}) | ||
|
||
add_library(secp256k1 STATIC src/secp256k1.c) | ||
target_compile_definitions(secp256k1 PRIVATE ${COMMON_COMPILE_FLAGS} ${COMPILE_FLAGS}) | ||
target_include_directories(secp256k1 PRIVATE ${CMAKE_SOURCE_DIR} ${CMAKE_SOURCE_DIR}/src) | ||
target_compile_options(secp256k1 PRIVATE ${COMPILE_OPTIONS}) | ||
add_dependencies(secp256k1 ecmult_static_context) | ||
|
||
install(TARGETS secp256k1 ARCHIVE DESTINATION lib) | ||
install(DIRECTORY include/ DESTINATION include) |
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
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
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
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
Submodule Schnorr
updated
2 files
+0 −6 | src/libSchnorr/src/MultiSig_CommitPointHash.cpp | |
+1 −1 | src/libSchnorr/src/Schnorr_PubKey.cpp |
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,2 +1,3 @@ | ||
add_library(EthCrypto EthCrypto.cpp) | ||
target_include_directories(EthCrypto PUBLIC ${PROJECT_SOURCE_DIR}/src) | ||
target_link_libraries(EthCrypto Secp256k1) |
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.
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.
3rd party library for public key recovery