-
Notifications
You must be signed in to change notification settings - Fork 38
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
Problem: inability to use remote infrastructure #46
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
Using IPC means bridge has to run alognside the node on the same machine. This, at times, presents problems in terms of efficiency or coupling of deployment. Solution: switch to RPC
The inner workings of choosing IPC vs RPC haven't been worked out. Solution: remove IPC in favour of RPC
RPC patch has introduced a separate basic test that, on the surface of it, is not too dissimilar from the basic deposit and withdrawal test already defined in integration tests. Solution: make sure integration tests run and remove newly introduced test
This directory doesn't reallly do anything to RPC. Solution: extract it
Solution: remove it from the manifest
This file is unrelated to the RPC patch Solution: remove it
Solution: remove them
Solution: use format!() macro instead of manual concatenation
Solution: remove `ipc` parameter to make them compile
As a part of the original feature request, there was a need for the bridge to be able to sign its own transactions. However, this didn't fully materialize in the original patch, and only configuration parameters were implemented. Solution: remove these last conflated bits and make this a pure transport change patch.
This means that the node has to sign the transaction itself. It might be acceptable in a localized setup, but can't be used with untrusted setups. For example, once HTTP RPC is supported, we can't really use infrastructure like INFURA to send transactions. Solution: switch to signing transactions in bridge This absolutely requires separating the accounts used by validators and administrative tasks as this will otherwise interfere with management of nonces.
On my computer, this takes approximately 0.3 seconds, which is clearly a deal-breaker. In retrospect, this is an obvious problem because of key derivation function use. Solution: unlock accounts permanently. This cut down time to sign one transaction is 0.0001 or so.
It isn't really necessary for anything that I can see. Solution: remove it
Solution: upgrade web3 to the version which has support for TLS
ghost
assigned yrashk
Apr 27, 2018
ghost
added
the
in progress
label
Apr 27, 2018
ghost
removed
the
in progress
label
Apr 27, 2018
This was referenced Apr 27, 2018
noot
pushed a commit
to noot/poa-bridge
that referenced
this pull request
Jul 18, 2018
Problem: inability to use remote infrastructure
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 pull request embeds two separately merged features: RPC and transaction signing