This repository has been archived by the owner on Aug 2, 2022. It is now read-only.
Optimize transaction signature recovery #6471
Merged
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
heifner
force-pushed
the
producer-txns
branch
from
9d9536f to
22ab63c
Compare
heifner
changed the title
This was referenced Dec 14, 2018
arhag
suggested changes
Dec 17, 2018
|
Hi @heifner , I've tested & compared your change in this ticket vs my change(#6460) vs original (develop branch). I got almost the same performance(~3600 TPS) between your implementation and mine, while the original code shows a significant lower TPS number(~2200 TPS). Here's the details: Testings steps: Your implementation(commit id 22ab63c): test 2 (111.379us / transaction, around 3591 TPS): test results of my PR: test 2 (112.083us / transaction, around 3568 TPS): Original code (around 2200 TPS): Also from my understanding, in |
|
@taokayan Thanks for testing on your machine. I was keeping it simple with one thread_pool since it is configurable and a user could just double the thread_pool size if this is an issue. But I can see adding another thread_pool to producer_plugin for these waits. I could also see using different thread_pools for apply_block sig recovery so that incoming transactions can't hold up apply_block. |
heifner
added a commit
that referenced
this pull request
Dec 17, 2018
… before application quit
arhag
suggested changes
Dec 17, 2018
arhag
suggested changes
Dec 18, 2018
arhag
approved these changes
Dec 18, 2018
heifner
added a commit
that referenced
this pull request
Dec 18, 2018
Vardhamana
pushed a commit
to boscore/bos
that referenced
this pull request
Feb 28, 2019
* state history plugin * bump version to 1.5.0-rc1 * Update README to reflect new package names * Change from 1.5.0 to 1.5.0-rc1 * Fix requires in spec file to resolve dependency resolution issues with rpms * Fix requires in spec file to resolve dependency resolution issues with rpms * Remove unneeded req_trx vector * Remove needless copy of signed_block. Only create pending_notify if needed. * Remove unused blk_buffer * separate out version suffix so that RPM packages produce an acceptable specfile * address PR review feedback * Make signed_block copy constructor private to avoid accidental copies. * add release suffix to doxygen cmake module * move signed_block instead of making copy * Avoid copy constructor for push_block * add release suffix to EOSIO_VERSION in tester cmake modules * Add which comment * Update tests for private signed_block copy constructor * Avoid making copies of signed_block * export the new bash vars * Use rvalue for signed_block * Update buildkite pipeline to use new queues * Update buildkite pipeline to use new queues * Special case msg_handler visit of signed_block to avoid hidden move from net_message * pass along trigger_send * Use move for rvalue * Added Mongo History API link * Added ZMQ Light History API * Added Chintai ZMQ Watcher * Added State History API * Stop the unix_server from listening in shutdown stage as well. * finish BOS basic functions * bump version to rc2 * update rpm names to match the real files * Fix deb package names in README * Fix package naming scheme for debs * Update LICENSE path * add a test that reduces the size of the producer set to validate that it does not create protocol level issues * promote rc2 to release * add missing release to deb packages * remove x86_64 arch dep specification on AMI2 build script I can't find any reason we need to explcitly request the x86_64 versions of packages for AMI2. Removing this apparently extraneous specification makes the build script "just work" on new ARM8 instances * Use 64-bit float printing of 128-bit floats on non x86_64 For platforms other then x86_64, have the 128-bit float console print API (printqf()) convert to a 64-bit float before printing. While this loses precision it’s enough to get the unit tests for printqf() working on ARM8, which actually was the only unit_test unit test that failed on that platform. * prebump to 1.6.0 and add develop suffix * Do not broadcast block if peer lib is larger than block number * Provide more efficient sha256_less * Simply by removing unused large_msg_notify * Remove unused node_transaction_state.packed_txn * Store serialized transaction as shared_ptr to minimize copies * Not possible for serialized_txn to be empty * Remove find_plugin overhead * Minimize shared_ptr copies * Use sha256_less for all sets/maps with ids * Add explicit constructors * Add accept_transaction that takes a packed_transaction_ptr to avoid copy * Pass id and packed_transaction by const& since never moved * Remove copy assignment operator. Remove used std::move. * Minimize packed_transaction copies. Store packed_transaction_ptr in transaction_metadata. * Cache chain_plugin lookup * Update tester for transaction_metadata packed_transaction_ptr * Remove packed_tansaction to net_message copy * Remove unused constructor * Explicitly disable copy/move construction * Remove inflight update of node_transaction_state as it is not needed * Remove connections for unused signals that only logged * Fix sync check for lib * add a test that verifies proposing an empty producer schedule does not break things * Allow disablement of net_plugin's listen socket Disable net_plugin's listen socket when p2p-listen-endpoint is empty. This is useful for security and for an easier way of running multiple nodeos without it conflicting when you don't actually need to service incoming connections. * More descriptive memo to distinguish from other transfers * Revert "Fix sync check for lib" This reverts commit 52a6f19. * Update the bios-boot-tutorial.py script and readme.md: 1. args.contracts_dir doesn't end in a forward slash, therefor when it is concatenated with a subpath, the subpath has to start with a forward slash; this pattern is present in other similar variables as well 2. system contract has to be initialized before it can be used (e.g. creating staked acconts) 3. commands list updated to include the new addition stepInitSystemContract 4. readme.md updated as well to reflect the current status of the script and requirements * Release/1.0.x (#11) * boscore basic improvement (#2) * kafka_plugin code * Automatic installation librdkafka/cppkafka * Feature/ci * Feature/48 kafka plugin * add CMakeModules/FindCppkafka.cmake * Production of block in time zone sequence * P2p self discovery * P2p self discovery * add notify_plugin * add api "get_block_detail" * add free res limit and blklst code * update free res limit and blklst code * update res code * update unittest code * revert submodule version * code typo * update blklist code * update sync name list db object error code * update code * update index code * Feature/5 ramdom * Revert "Merge branch 'feature/5-ramdom' into 'develop'" This reverts merge request !8 * adjust for setup BOSCore * change description * adjust the kafka plugin dependency be more special * use boscore repository to improve security * change version tag * finish for docker/builder * pass to build docker and update readme * add actionseed, global action sequence (#5) * delete renamed old file * BOSCore v1.0.1-1.4.3 * restructure the version schema * fix __gmpn_set_str error when build bos.contract * prepare for the v1.0.1 * add README files * update info * readme for kafka & add time for action (#5) * 重启 节点,黑名单 失效，fixes #7 (#8) * restart sync list db * recovery system account bos to eosio * recovery system account bos to eosio * recovery system account bos to eosio * Fix/#3 notify plugin (#10) * Add debug info * comment log * rm log for notify_plugin * README.md refactored - correcting spelling errors, and caps - better bash commands - simpler command line to launch the script - improved prerequisites - include steps to follow after prerequisites * Correct typos * Correct link format * Add eosio.cdt steps * merge v1.0.2 (#13) * boscore basic improvement (#2) * kafka_plugin code * Automatic installation librdkafka/cppkafka * Feature/ci * Feature/48 kafka plugin * add CMakeModules/FindCppkafka.cmake * Production of block in time zone sequence * P2p self discovery * P2p self discovery * add notify_plugin * add api "get_block_detail" * add free res limit and blklst code * update free res limit and blklst code * update res code * update unittest code * revert submodule version * code typo * update blklist code * update sync name list db object error code * update code * update index code * Feature/5 ramdom * Revert "Merge branch 'feature/5-ramdom' into 'develop'" This reverts merge request !8 * adjust for setup BOSCore * change description * adjust the kafka plugin dependency be more special * use boscore repository to improve security * change version tag * finish for docker/builder * pass to build docker and update readme * add actionseed, global action sequence (#5) * delete renamed old file * BOSCore v1.0.1-1.4.3 * restructure the version schema * fix __gmpn_set_str error when build bos.contract * prepare for the v1.0.1 * add README files * update info * prepare for v1.0.2 * Better name for eosio contracts directory * dummy checkin to trigger another build * Added scrit to support recursive search for files with FC_REFLECT or FC_REFLECT_ENUM to verify order and completeness. Supports comments with @swap to support different order from definition and @ignore to ignore a specific field for reflection. Still need to support FC_REFLECT_DERIVED. GH EOSIO#3127 * Added @ignore comment for field that is left out of reflect definition. GH EOSIO#3127 * Fixed pull request comments. * bump version to 1.5.1 * Consolidated Security Fixes for 1.5.1 - Only allow authorizations that are satisfiable by `eosio.code` for self-addressed deferred transactions - Only allow authorizations that are satisfiable by `eosio.code` OR on the parent action for self-addressed inline actions sent from direct actions - Only allow authorizations that are satisfiable by `eosio.code` for self-addressed inline actions sent from recipient handlers Co-authored-by: arhag <[email protected]> Co-authored-by: Bart Wyatt <[email protected]> * Peer review changes. Fix move. * Spelling correction * Spelling correction #2 * Remove unused max-implicit-request config * Switch interface from packed_transaction_ptr to transaction_metadata_ptr * Thread pool does not need to be optional * Add transaction_metadata create_signing_keys_future method * Start transaction signature earily in thread pool * Refactor packed_transaction for better encapsulation * Add transaction_metadata create_signing_keys_future method * Start transaction signature earily in thread pool * Update txn_test_gen_plugin to overlap transaction submit @taokayan * merge 1.0.3 to develop (#16) * boscore basic improvement (#2) * kafka_plugin code * Automatic installation librdkafka/cppkafka * Feature/ci * Feature/48 kafka plugin * add CMakeModules/FindCppkafka.cmake * Production of block in time zone sequence * P2p self discovery * P2p self discovery * add notify_plugin * add api "get_block_detail" * add free res limit and blklst code * update free res limit and blklst code * update res code * update unittest code * revert submodule version * code typo * update blklist code * update sync name list db object error code * update code * update index code * Feature/5 ramdom * Revert "Merge branch 'feature/5-ramdom' into 'develop'" This reverts merge request !8 * adjust for setup BOSCore * change description * adjust the kafka plugin dependency be more special * use boscore repository to improve security * change version tag * finish for docker/builder * pass to build docker and update readme * add actionseed, global action sequence (#5) * delete renamed old file * BOSCore v1.0.1-1.4.3 * restructure the version schema * fix __gmpn_set_str error when build bos.contract * prepare for the v1.0.1 * add README files * update info * prepare for v1.0.2 * merge v1.0.2 (#12) * boscore basic improvement (#2) * kafka_plugin code * Automatic installation librdkafka/cppkafka * Feature/ci * Feature/48 kafka plugin * add CMakeModules/FindCppkafka.cmake * Production of block in time zone sequence * P2p self discovery * P2p self discovery * add notify_plugin * add api "get_block_detail" * add free res limit and blklst code * update free res limit and blklst code * update res code * update unittest code * revert submodule version * code typo * update blklist code * update sync name list db object error code * update code * update index code * Feature/5 ramdom * Revert "Merge branch 'feature/5-ramdom' into 'develop'" This reverts merge request !8 * adjust for setup BOSCore * change description * adjust the kafka plugin dependency be more special * use boscore repository to improve security * change version tag * finish for docker/builder * pass to build docker and update readme * add actionseed, global action sequence (#5) * delete renamed old file * BOSCore v1.0.1-1.4.3 * restructure the version schema * fix __gmpn_set_str error when build bos.contract * prepare for the v1.0.1 * finish BOS basic functions * add README files * update info * Release/1.0.x (#11) * boscore basic improvement (#2) * kafka_plugin code * Automatic installation librdkafka/cppkafka * Feature/ci * Feature/48 kafka plugin * add CMakeModules/FindCppkafka.cmake * Production of block in time zone sequence * P2p self discovery * P2p self discovery * add notify_plugin * add api "get_block_detail" * add free res limit and blklst code * update free res limit and blklst code * update res code * update unittest code * revert submodule version * code typo * update blklist code * update sync name list db object error code * update code * update index code * Feature/5 ramdom * Revert "Merge branch 'feature/5-ramdom' into 'develop'" This reverts merge request !8 * adjust for setup BOSCore * change description * adjust the kafka plugin dependency be more special * use boscore repository to improve security * change version tag * finish for docker/builder * pass to build docker and update readme * add actionseed, global action sequence (#5) * delete renamed old file * BOSCore v1.0.1-1.4.3 * restructure the version schema * fix __gmpn_set_str error when build bos.contract * prepare for the v1.0.1 * add README files * update info * readme for kafka & add time for action (#5) * 重启 节点,黑名单 失效，fixes #7 (#8) * restart sync list db * recovery system account bos to eosio * recovery system account bos to eosio * recovery system account bos to eosio * Fix/#3 notify plugin (#10) * Add debug info * comment log * rm log for notify_plugin * prepare for v1.0.2 * patch the EOSIO 1.5.1 security bug fixes * prepare for v1.0.3 * adjust the slogon * Remove redundant signing_keys check * state history plugin: permission_object bug EOSIO#6495 * Add deadline to key recovery * Modify producer_plugin to have its own thead_pool instead of using chain-threads * Move thread_pool join/stop to plugin shutdown so that they are joined before application quit * Fix signature future deadline from starting too early * Fix overflow of deadline and deadline check * initial setup of billing CPU for signatures recovered earlier * Return 400 on get_block with uonexist block number. For issue 6374 * has_contract() : Determine whether to deploy the contract (#18) * fix issue 5488 * Fix comments and function names (#19) * Add action_trace.block_num index. Remove action_trace.trx_id index. * Fix return codes of build scripts so that buildkite can fail properly * Make recovery cache non-thread local and guard by mutex * Calculate cpu usage of signature recovery * Add signature-cpu-billable-pct option to chain_plugin * Add missing include of mutex * Assert signature-cpu-billable-pct is 0-100 * Fix capture of cpu_usage. move flat_set<public_key_type> into attribute * clear recovered_pub_keys to preserve previous behaviour * use `assign` instead of `resize`+`memcpy` to update `shared_blob` data Co-Authored-By: Kayan <[email protected]> * Add move into tuple creation * bump version to 1.6.0-rc1 * add explicitly defaulted move semantics * Store unpacked_trx as signed_transaction * get_uncached_id no longer needed for thread safety * get_raw_transaction no longer needed for thread safety * Remove cached signed_transaction since now cached in packed_transaction * Test should honor existing compression * Cleanup of packed_transaction interface. Fixes for comments on PR EOSIO#6471 * Update to fc with unpack verify * fc renamed reflector_verify to reflector_init * bump version to 1.5.2 * Update to latest fc with updated verify_init * Consolidated Security Fixes for 1.5.3 - Add missing implementation of dtor, copy ctor, etc for blob_types of fc::variant Co-Authored-By: Matt Witherspoon <[email protected]> * bump version to 1.5.3 * bump version to 1.6.0-rc2 * Consolidated Security Fixes for 1.6.0-rc2 - Add missing implementation of dtor, copy ctor, etc for blob_types of fc::variant Co-Authored-By: Matt Witherspoon <[email protected]> * update developer portal link update the getting started link to the latest version * fix tag version typo of cppkafka (#24) * get_contract_code (#26) * fixed #20Using greylist-account while starting nodeos for the first time throws exception (#23) * restart sync list db * recovery system account bos to eosio * catch exception plugin initialize sync list before initialize database * fixed 'cleos system bidname info' cmd exec return could not parse uint64_t (#30) * recovery system account bos to eosio * catch exception plugin initialize sync list before initialize database * fixed bidnameinfo could not parse uint64_t * prepare for 2.0.1 * adjust the location paramater * apply 1.5.3 patch * modify the location type * fix cmake error * Use https url for wabt submodule A user agent that doesn't respect hsts could potentially be coerced into downloading malicious sources for wabt via a mitm attack. Prevent this by using a https upstream like the other submodules already do * Add libtinfo5 dependency to deb package. Resolves EOSIO#6590 Also clean up unused variables and force control directory permissions to comply with Debian specifications regardless of umask. * fix aragument name error * Bump version to 1.6.0 * Consolidated Security Fixes for 1.6.0 - Force compilation support for c++ exceptions for the wabt submodule - Allow limiting the time a node will spend processing scheduled transactions Co-Authored-By: Matt Witherspoon <[email protected]> Co-Authored-By: Bart Wyatt <[email protected]> * Consolidated Security Fixes for 1.6.1 - net_plugin security fixes - Additional checktime calls to limit cpu usage - Limit memory usage in producer_plugin Co-Authored-By: Kevin Heifner <[email protected]> Co-Authored-By: Matt Witherspoon <[email protected]> Co-authored-by: Kayan <[email protected]> * Bump version to 1.6.1 * Fix boost 1.67 brew install The latest homebrew code balks at something in the old 1.67 package file. Fix the package file and store it locally in our repo for now. We try and pin the boost version because boost upgrades invalidate nodeos data files. * Set proper directory for baked in macOS LLVM_DIR Some users (including myself) were seeing llvm@4 unpacked to 4.0.1_1 instead of 4.0.1. Stuff unpacked to the Cellar directory appears to be a kind of implementation detail — /usr/local/opt is the proper place to reference here. * Add bk step to gather brew files for automatic update * Update version to 1.6.2 * add noninteractive option for build scripts * Limit assert message to 1024 chars * Don't unlink what we install via brew unlinking eveything we install makes no sense -- it means things like cmake aren't in the path any longer like the script expects. So don't do that any more. Unfortuately this old script requires that gettext be force linked. So implement that behavior explictly for now * Make sure python-devel is installed for amazon linux builds * fc version 1.6.x * merge missed code * merge missed code * merge missed code * prepare v2.0.2
Vardhamana
pushed a commit
to boscore/bos
that referenced
this pull request
Feb 28, 2019
* Remove unnecessary conversion of contractPath to empty path * enhance cleos multisig review to show proposed transaction ID and optionally show requested approvals EOSIO#6272 * add cleos multisig invalidate command to call eosio.msig::invalidate * state history plugin * fix GMP & secp256 linker order for EosioTesterBuild.cmake.in This is the same fix from pr EOSIO#6268 but for EosioTesterBuild.cmake.in too * state history plugin * bump version to 1.5.0-rc1 * Update README to reflect new package names * Change from 1.5.0 to 1.5.0-rc1 * Fix requires in spec file to resolve dependency resolution issues with rpms * Fix requires in spec file to resolve dependency resolution issues with rpms * Remove unneeded req_trx vector * Remove needless copy of signed_block. Only create pending_notify if needed. * Remove unused blk_buffer * separate out version suffix so that RPM packages produce an acceptable specfile * address PR review feedback * Make signed_block copy constructor private to avoid accidental copies. * add release suffix to doxygen cmake module * move signed_block instead of making copy * Avoid copy constructor for push_block * add release suffix to EOSIO_VERSION in tester cmake modules * Add which comment * Update tests for private signed_block copy constructor * Avoid making copies of signed_block * export the new bash vars * Use rvalue for signed_block * Update buildkite pipeline to use new queues * Update buildkite pipeline to use new queues * Special case msg_handler visit of signed_block to avoid hidden move from net_message * pass along trigger_send * Use move for rvalue * Added Mongo History API link * Added ZMQ Light History API * Added Chintai ZMQ Watcher * Added State History API * Stop the unix_server from listening in shutdown stage as well. * bump version to rc2 * update rpm names to match the real files * Fix deb package names in README * Fix package naming scheme for debs * Update LICENSE path * add a test that reduces the size of the producer set to validate that it does not create protocol level issues * promote rc2 to release * add missing release to deb packages * remove x86_64 arch dep specification on AMI2 build script I can't find any reason we need to explcitly request the x86_64 versions of packages for AMI2. Removing this apparently extraneous specification makes the build script "just work" on new ARM8 instances * Use 64-bit float printing of 128-bit floats on non x86_64 For platforms other then x86_64, have the 128-bit float console print API (printqf()) convert to a 64-bit float before printing. While this loses precision it’s enough to get the unit tests for printqf() working on ARM8, which actually was the only unit_test unit test that failed on that platform. * prebump to 1.6.0 and add develop suffix * Do not broadcast block if peer lib is larger than block number * Provide more efficient sha256_less * Simply by removing unused large_msg_notify * Remove unused node_transaction_state.packed_txn * Store serialized transaction as shared_ptr to minimize copies * Not possible for serialized_txn to be empty * Remove find_plugin overhead * Minimize shared_ptr copies * Use sha256_less for all sets/maps with ids * Add explicit constructors * Add accept_transaction that takes a packed_transaction_ptr to avoid copy * Pass id and packed_transaction by const& since never moved * Remove copy assignment operator. Remove used std::move. * Minimize packed_transaction copies. Store packed_transaction_ptr in transaction_metadata. * Cache chain_plugin lookup * Update tester for transaction_metadata packed_transaction_ptr * Remove packed_tansaction to net_message copy * Remove unused constructor * Explicitly disable copy/move construction * Remove inflight update of node_transaction_state as it is not needed * Remove connections for unused signals that only logged * Fix sync check for lib * add a test that verifies proposing an empty producer schedule does not break things * Allow disablement of net_plugin's listen socket Disable net_plugin's listen socket when p2p-listen-endpoint is empty. This is useful for security and for an easier way of running multiple nodeos without it conflicting when you don't actually need to service incoming connections. * More descriptive memo to distinguish from other transfers * Revert "Fix sync check for lib" This reverts commit 52a6f19. * Update the bios-boot-tutorial.py script and readme.md: 1. args.contracts_dir doesn't end in a forward slash, therefor when it is concatenated with a subpath, the subpath has to start with a forward slash; this pattern is present in other similar variables as well 2. system contract has to be initialized before it can be used (e.g. creating staked acconts) 3. commands list updated to include the new addition stepInitSystemContract 4. readme.md updated as well to reflect the current status of the script and requirements * README.md refactored - correcting spelling errors, and caps - better bash commands - simpler command line to launch the script - improved prerequisites - include steps to follow after prerequisites * Correct typos * Correct link format * Add eosio.cdt steps * merge v1.0.2 (#12) * boscore basic improvement (#2) * kafka_plugin code * Automatic installation librdkafka/cppkafka * Feature/ci * Feature/48 kafka plugin * add CMakeModules/FindCppkafka.cmake * Production of block in time zone sequence * P2p self discovery * P2p self discovery * add notify_plugin * add api "get_block_detail" * add free res limit and blklst code * update free res limit and blklst code * update res code * update unittest code * revert submodule version * code typo * update blklist code * update sync name list db object error code * update code * update index code * Feature/5 ramdom * Revert "Merge branch 'feature/5-ramdom' into 'develop'" This reverts merge request !8 * adjust for setup BOSCore * change description * adjust the kafka plugin dependency be more special * use boscore repository to improve security * change version tag * finish for docker/builder * pass to build docker and update readme * add actionseed, global action sequence (#5) * delete renamed old file * BOSCore v1.0.1-1.4.3 * restructure the version schema * fix __gmpn_set_str error when build bos.contract * prepare for the v1.0.1 * finish BOS basic functions * add README files * update info * Release/1.0.x (#11) * boscore basic improvement (#2) * kafka_plugin code * Automatic installation librdkafka/cppkafka * Feature/ci * Feature/48 kafka plugin * add CMakeModules/FindCppkafka.cmake * Production of block in time zone sequence * P2p self discovery * P2p self discovery * add notify_plugin * add api "get_block_detail" * add free res limit and blklst code * update free res limit and blklst code * update res code * update unittest code * revert submodule version * code typo * update blklist code * update sync name list db object error code * update code * update index code * Feature/5 ramdom * Revert "Merge branch 'feature/5-ramdom' into 'develop'" This reverts merge request !8 * adjust for setup BOSCore * change description * adjust the kafka plugin dependency be more special * use boscore repository to improve security * change version tag * finish for docker/builder * pass to build docker and update readme * add actionseed, global action sequence (#5) * delete renamed old file * BOSCore v1.0.1-1.4.3 * restructure the version schema * fix __gmpn_set_str error when build bos.contract * prepare for the v1.0.1 * add README files * update info * readme for kafka & add time for action (#5) * 重启 节点,黑名单 失效，fixes #7 (#8) * restart sync list db * recovery system account bos to eosio * recovery system account bos to eosio * recovery system account bos to eosio * Fix/#3 notify plugin (#10) * Add debug info * comment log * rm log for notify_plugin * prepare for v1.0.2 * Better name for eosio contracts directory * dummy checkin to trigger another build * Added scrit to support recursive search for files with FC_REFLECT or FC_REFLECT_ENUM to verify order and completeness. Supports comments with @swap to support different order from definition and @ignore to ignore a specific field for reflection. Still need to support FC_REFLECT_DERIVED. GH EOSIO#3127 * Added @ignore comment for field that is left out of reflect definition. GH EOSIO#3127 * Fixed pull request comments. * bump version to 1.5.1 * Consolidated Security Fixes for 1.5.1 - Only allow authorizations that are satisfiable by `eosio.code` for self-addressed deferred transactions - Only allow authorizations that are satisfiable by `eosio.code` OR on the parent action for self-addressed inline actions sent from direct actions - Only allow authorizations that are satisfiable by `eosio.code` for self-addressed inline actions sent from recipient handlers Co-authored-by: arhag <[email protected]> Co-authored-by: Bart Wyatt <[email protected]> * Peer review changes. Fix move. * Spelling correction * Spelling correction #2 * Remove unused max-implicit-request config * Switch interface from packed_transaction_ptr to transaction_metadata_ptr * Thread pool does not need to be optional * Add transaction_metadata create_signing_keys_future method * Start transaction signature earily in thread pool * Refactor packed_transaction for better encapsulation * Add transaction_metadata create_signing_keys_future method * Start transaction signature earily in thread pool * Update txn_test_gen_plugin to overlap transaction submit @taokayan * release v1.0.3 to master (#15) * boscore basic improvement (#2) * kafka_plugin code * Automatic installation librdkafka/cppkafka * Feature/ci * Feature/48 kafka plugin * add CMakeModules/FindCppkafka.cmake * Production of block in time zone sequence * P2p self discovery * P2p self discovery * add notify_plugin * add api "get_block_detail" * add free res limit and blklst code * update free res limit and blklst code * update res code * update unittest code * revert submodule version * code typo * update blklist code * update sync name list db object error code * update code * update index code * Feature/5 ramdom * Revert "Merge branch 'feature/5-ramdom' into 'develop'" This reverts merge request !8 * adjust for setup BOSCore * change description * adjust the kafka plugin dependency be more special * use boscore repository to improve security * change version tag * finish for docker/builder * pass to build docker and update readme * add actionseed, global action sequence (#5) * delete renamed old file * BOSCore v1.0.1-1.4.3 * restructure the version schema * fix __gmpn_set_str error when build bos.contract * prepare for the v1.0.1 * finish BOS basic functions * add README files * update info * Release/1.0.x (#11) * boscore basic improvement (#2) * kafka_plugin code * Automatic installation librdkafka/cppkafka * Feature/ci * Feature/48 kafka plugin * add CMakeModules/FindCppkafka.cmake * Production of block in time zone sequence * P2p self discovery * P2p self discovery * add notify_plugin * add api "get_block_detail" * add free res limit and blklst code * update free res limit and blklst code * update res code * update unittest code * revert submodule version * code typo * update blklist code * update sync name list db object error code * update code * update index code * Feature/5 ramdom * Revert "Merge branch 'feature/5-ramdom' into 'develop'" This reverts merge request !8 * adjust for setup BOSCore * change description * adjust the kafka plugin dependency be more special * use boscore repository to improve security * change version tag * finish for docker/builder * pass to build docker and update readme * add actionseed, global action sequence (#5) * delete renamed old file * BOSCore v1.0.1-1.4.3 * restructure the version schema * fix __gmpn_set_str error when build bos.contract * prepare for the v1.0.1 * add README files * update info * readme for kafka & add time for action (#5) * 重启 节点,黑名单 失效，fixes #7 (#8) * restart sync list db * recovery system account bos to eosio * recovery system account bos to eosio * recovery system account bos to eosio * Fix/#3 notify plugin (#10) * Add debug info * comment log * rm log for notify_plugin * prepare for v1.0.2 * merge v1.0.2 (#13) * boscore basic improvement (#2) * kafka_plugin code * Automatic installation librdkafka/cppkafka * Feature/ci * Feature/48 kafka plugin * add CMakeModules/FindCppkafka.cmake * Production of block in time zone sequence * P2p self discovery * P2p self discovery * add notify_plugin * add api "get_block_detail" * add free res limit and blklst code * update free res limit and blklst code * update res code * update unittest code * revert submodule version * code typo * update blklist code * update sync name list db object error code * update code * update index code * Feature/5 ramdom * Revert "Merge branch 'feature/5-ramdom' into 'develop'" This reverts merge request !8 * adjust for setup BOSCore * change description * adjust the kafka plugin dependency be more special * use boscore repository to improve security * change version tag * finish for docker/builder * pass to build docker and update readme * add actionseed, global action sequence (#5) * delete renamed old file * BOSCore v1.0.1-1.4.3 * restructure the version schema * fix __gmpn_set_str error when build bos.contract * prepare for the v1.0.1 * add README files * update info * prepare for v1.0.2 * patch the EOSIO 1.5.1 security bug fixes * prepare for v1.0.3 * adjust the slogon * Remove redundant signing_keys check * state history plugin: permission_object bug EOSIO#6495 * Add deadline to key recovery * Modify producer_plugin to have its own thead_pool instead of using chain-threads * Move thread_pool join/stop to plugin shutdown so that they are joined before application quit * Fix signature future deadline from starting too early * Fix overflow of deadline and deadline check * initial setup of billing CPU for signatures recovered earlier * Return 400 on get_block with uonexist block number. For issue 6374 * fix issue 5488 * Add action_trace.block_num index. Remove action_trace.trx_id index. * Fix return codes of build scripts so that buildkite can fail properly * Make recovery cache non-thread local and guard by mutex * Calculate cpu usage of signature recovery * Add signature-cpu-billable-pct option to chain_plugin * Add missing include of mutex * Assert signature-cpu-billable-pct is 0-100 * Fix capture of cpu_usage. move flat_set<public_key_type> into attribute * clear recovered_pub_keys to preserve previous behaviour * use `assign` instead of `resize`+`memcpy` to update `shared_blob` data Co-Authored-By: Kayan <[email protected]> * Add move into tuple creation * bump version to 1.6.0-rc1 * add explicitly defaulted move semantics * Store unpacked_trx as signed_transaction * get_uncached_id no longer needed for thread safety * get_raw_transaction no longer needed for thread safety * Remove cached signed_transaction since now cached in packed_transaction * Test should honor existing compression * Cleanup of packed_transaction interface. Fixes for comments on PR EOSIO#6471 * Update to fc with unpack verify * fc renamed reflector_verify to reflector_init * bump version to 1.5.2 * Update to latest fc with updated verify_init * Consolidated Security Fixes for 1.5.3 - Add missing implementation of dtor, copy ctor, etc for blob_types of fc::variant Co-Authored-By: Matt Witherspoon <[email protected]> * bump version to 1.5.3 * bump version to 1.6.0-rc2 * Consolidated Security Fixes for 1.6.0-rc2 - Add missing implementation of dtor, copy ctor, etc for blob_types of fc::variant Co-Authored-By: Matt Witherspoon <[email protected]> * update developer portal link update the getting started link to the latest version * prepare for 2.0.1 * adjust the location paramater * apply 1.5.3 patch * modify the location type * fix cmake error * Use https url for wabt submodule A user agent that doesn't respect hsts could potentially be coerced into downloading malicious sources for wabt via a mitm attack. Prevent this by using a https upstream like the other submodules already do * Add libtinfo5 dependency to deb package. Resolves EOSIO#6590 Also clean up unused variables and force control directory permissions to comply with Debian specifications regardless of umask. * fix aragument name error * merge v2.0.1 version (#36) * Bump version to 1.6.0 * Consolidated Security Fixes for 1.6.0 - Force compilation support for c++ exceptions for the wabt submodule - Allow limiting the time a node will spend processing scheduled transactions Co-Authored-By: Matt Witherspoon <[email protected]> Co-Authored-By: Bart Wyatt <[email protected]> * Consolidated Security Fixes for 1.6.1 - net_plugin security fixes - Additional checktime calls to limit cpu usage - Limit memory usage in producer_plugin Co-Authored-By: Kevin Heifner <[email protected]> Co-Authored-By: Matt Witherspoon <[email protected]> Co-authored-by: Kayan <[email protected]> * Bump version to 1.6.1 * Fix boost 1.67 brew install The latest homebrew code balks at something in the old 1.67 package file. Fix the package file and store it locally in our repo for now. We try and pin the boost version because boost upgrades invalidate nodeos data files. * Set proper directory for baked in macOS LLVM_DIR Some users (including myself) were seeing llvm@4 unpacked to 4.0.1_1 instead of 4.0.1. Stuff unpacked to the Cellar directory appears to be a kind of implementation detail — /usr/local/opt is the proper place to reference here. * Add bk step to gather brew files for automatic update * Update version to 1.6.2 * add noninteractive option for build scripts * Limit assert message to 1024 chars * Don't unlink what we install via brew unlinking eveything we install makes no sense -- it means things like cmake aren't in the path any longer like the script expects. So don't do that any more. Unfortuately this old script requires that gettext be force linked. So implement that behavior explictly for now * Make sure python-devel is installed for amazon linux builds * fc version 1.6.x * merge missed code * merge missed code * merge missed code * prepare v2.0.2
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
Change Description
Consensus Changes
None
API Changes
New
producer_pluginoption:--
--producer-threads arg (=2) Number of worker threads in producer thread poolNew
chain_pluginoption:--
--signature-cpu-billable-pct arg (=50) Percentage of actual signature recovery cpu to bill. Whole number percentages, e.g. 50 for 50%Documentation Additions
See API Changes above