Skip to content
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

Feature/1603 ens offchain integration #1617

Conversation

andrii-kl
Copy link
Contributor

What does this PR do?

Add support of new functions of ENS. eip-3668 | EIP 1577

Why is it needed?

To work with new ENS functions.

@andrii-kl andrii-kl added the Don't merge PR is not ready for merge label Feb 3, 2022
@andrii-kl andrii-kl requested a review from AlexandrouR February 3, 2022 11:31
@andrii-kl andrii-kl self-assigned this Feb 3, 2022
@andrii-kl andrii-kl requested a review from conor10 as a code owner February 3, 2022 11:31
@andrii-kl andrii-kl linked an issue Feb 3, 2022 that may be closed by this pull request
@makoto
Copy link

makoto commented Mar 25, 2022

Hi @andrii-kl thanks for the integration PR and sorry for not being able to respond quickly.
I have a couple of comments

  1. Does it have any usage example in doc or in the test?
  2. Could you test with real data 1.offchainexample.eth as described in our doc and see if it returns the record as expected?

Thanks.

Makoto

@makoto
Copy link

makoto commented Mar 29, 2022

FYI. Added some test casce snenario for web3.py which you can refer to as well ethereum/web3.py#2406

@stale
Copy link

stale bot commented Apr 19, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale no activity for 21 days label Apr 19, 2022
@conor10 conor10 removed the stale no activity for 21 days label Apr 19, 2022
@andrii-kl
Copy link
Contributor Author

andrii-kl commented Apr 28, 2022

@makoto
I have change a bit my realisation according to example from
https://docs.ens.domains/ens-improvement-proposals/ensip-10-wildcard-resolution
but I have an error when I try to call resolver.resolve(dnsEncodedName, calldata)

"Contract Call has been reverted by the EVM with the reason: 'execution reverted'."

@andrii-kl
Copy link
Contributor Author

andrii-kl commented Apr 29, 2022

@makoto Same response in the etherscan too. (ensName: 1.offchainexample.eth)
Screen Shot 2022-04-29 at 12 47 18 PM

Could you provide right execution parameters for this method? (1.offchainexample.eth)

@andrii-kl andrii-kl requested a review from gtebrean May 3, 2022 20:32
@andrii-kl andrii-kl removed the Don't merge PR is not ready for merge label May 4, 2022
Copy link
Contributor

@gtebrean gtebrean left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good for me, only few questions

@@ -32,33 +34,44 @@
* or the org.web3j.codegen.SolidityFunctionWrapperGenerator in the
* <a href="https://github.com/web3j/web3j/tree/master/codegen">codegen module</a> to update.
*
* <p>Generated with web3j version 4.0.0.
* <p>Generated with web3j version 1.4.1.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

shouldn't this be generated with a newer version?

utils/src/main/java/org/web3j/utils/Numeric.java Outdated Show resolved Hide resolved
core/src/main/java/org/web3j/ens/EnsResolver.java Outdated Show resolved Hide resolved
* or the org.web3j.codegen.SolidityFunctionWrapperGenerator in the
* <a href="https://github.com/web3j/web3j/tree/master/codegen">codegen module</a> to update.
*
* <p>Generated with web3j version 1.4.1.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

shouldn't this be generated with a newer version?

core/src/main/java/org/web3j/ens/EnsResolver.java Outdated Show resolved Hide resolved
@makoto
Copy link

makoto commented May 4, 2022

@makoto Same response in the etherscan too. (ensName: 1.offchainexample.eth) Screen Shot 2022-04-29 at 12 47 18 PM

Could you provide right execution parameters for this method? (1.offchainexample.eth)

That should show custom error type with gateway URL like the below as long as you put two bytecode

     Error: call revert exception [ See: https://links.ethers.org/v5-errors-CALL_EXCEPTION ] (method="resolve(bytes,bytes)", errorArgs=["0x5FbDB2315678afecb367f032d93F642f64180aa3",["http://localhost:8080/"],"0x9061b92300000000000000000000000000000000000000000000000000000000000000400000000000000000000000000000000000000000000000000000000000000080000000000000000000000000000000000000000000000000000000000000001701310f6f6666636861696e6578616d706c6503657468000000000000000000000000000000000000000000000000000000000000000000000000000000000000","0xf4d4d2f8","0x9061b92300000000000000000000000000000000000000000000000000000000000000400000000000000000000000000000000000000000000000000000000000000080000000000000000000000000000000000000000000000000000000000000001701310f6f6666636861696e6578616d706c6503657468000000000000000000000000000000000000000000000000000000000000000000000000000000000000"], errorName="OffchainLookup", errorSignature="OffchainLookup(address,string[],bytes,bytes4,bytes)", reason=null, code=CALL_EXCEPTION, version=abi/5.6.0)
```

For some reason, it's not showing as you demonstrated. Let me check with etherscan team if they do support showing custom error type

@andrii-kl
Copy link
Contributor Author

@makoto Same response in the etherscan too. (ensName: 1.offchainexample.eth) Screen Shot 2022-04-29 at 12 47 18 PM
Could you provide right execution parameters for this method? (1.offchainexample.eth)

That should show custom error type with gateway URL like the below as long as you put two bytecode

     Error: call revert exception [ See: https://links.ethers.org/v5-errors-CALL_EXCEPTION ] (method="resolve(bytes,bytes)", errorArgs=["0x5FbDB2315678afecb367f032d93F642f64180aa3",["http://localhost:8080/"],"0x9061b92300000000000000000000000000000000000000000000000000000000000000400000000000000000000000000000000000000000000000000000000000000080000000000000000000000000000000000000000000000000000000000000001701310f6f6666636861696e6578616d706c6503657468000000000000000000000000000000000000000000000000000000000000000000000000000000000000","0xf4d4d2f8","0x9061b92300000000000000000000000000000000000000000000000000000000000000400000000000000000000000000000000000000000000000000000000000000080000000000000000000000000000000000000000000000000000000000000001701310f6f6666636861696e6578616d706c6503657468000000000000000000000000000000000000000000000000000000000000000000000000000000000000"], errorName="OffchainLookup", errorSignature="OffchainLookup(address,string[],bytes,bytes4,bytes)", reason=null, code=CALL_EXCEPTION, version=abi/5.6.0)

For some reason, it's not showing as you demonstrated. Let me check with etherscan team if they do support showing custom error type

Yes, it would be great to add support on the etherscan.

On our side, everything is fine, we have dealt with everything, thank you.

@andrii-kl andrii-kl merged commit 85270be into hyperledger-web3j:master May 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ENS L2/offchain data integration
4 participants