Implemented eth namespace apis using internally already implemented klaytnAPI #1124
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.
Proposed changes
This PR Implemented
eth
namespace apis using already implemented klaytnAPI.Related Confluence: How Ethereum APIs are supported in Klaytn
How to test this PR
First, please clone my origin repo.
1. Build ken with source codes related with this PR.
make ken
2. Run ken
eth
namepsace to RPC_API and WS_API in kend.conf3. Setup test scenario
Deploy kip7 contract
0x6A90F04A407258c4F516959E4f62CD2C0C309018
0x3e2ac308cd78ac2fe162f9522deb2b56d9da9499
0x8944d588cf8a4d255e2510f1c05b1eb4557bdaba
0xddf252ad1be2c89b69c2b068fc378daa952ba7f163c4a11628f55a4df523b3ef
(=keccak256("Transfer(address,address,uint256)")
)Deploy storage contract
0xDAf9C71C8a35b397250fb9f2ac1bD6b685E722Df
6080604052348015600f57600080fd5b506104d260008190555061162e600160003373ffffffffffffffffffffffffffffffffffffffff1673ffffffffffffffffffffffffffffffffffffffff16815260200190815260200160002081905550603580606c6000396000f3006080604052600080fd00a165627a7a72305820372ea18474d2fe1c7f0d56ce71fac400e8f8b1ca01c83b0544b7af09b7420b850029
[{"inputs":[],"payable":false,"stateMutability":"nonpayable","type":"constructor"}]
Test Logs
Click to expand
GetBlockTransactionCountByHash
GetBlockTransactionCountByNumber
NewPendingTransactionFilter
Subscription: NewPendingTransactions
NewBlockFilter*
Subscription: Logs
NewFilter
GetLogs
UninstallFilter
GetFilterChanges
GasPrice
MaxPriorityFeePerGas
Syncing
ChainId
BlockNumber
GetBalance
GetCode
GetStorageAt
Accounts
Types of changes
Please put an x in the boxes related to your change.
Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.
$ make test
)Related issues
Further comments
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...