web3modal icon indicating copy to clipboard operation
web3modal copied to clipboard

[bug] GET return 404 from https://rpc.walletconnect.com/v1/profile/reverse/xxx

Open tuannx opened this issue 1 year ago • 14 comments

Link to minimal reproducible example

https://web3modal-mre.vercel.app/

Summary

Create new react ts with web3modal ethers.

Connect to smartwallet base testnet: chainId: 84532, name: "Base Sepolia", currency: "ETH", explorerUrl: "https://sepolia-explorer.base.org", rpcUrl: "https://sepolia.base.org",

After login got error 404: GET https://rpc.walletconnect.com/v1/profile/reverse/0x14338152970B7B9A24092382e4B2e9B78Ba13D65?projectId=28e14a91201028f94cee88eb9da0a227 404 (Not Found)

Potential problem: Smartwallet create smart contract address instead of normal wallet address.

smartwallet address on Base testnet: 0x14338152970B7B9A24092382e4B2e9B78Ba13D65

code: https://github.com/tuannx/web3modal-mre

List of related npm package versions

"@testing-library/jest-dom": "^5.17.0",
"@testing-library/react": "^13.4.0",
"@testing-library/user-event": "^13.5.0",
"@types/jest": "^27.5.2",
"@types/node": "^16.18.97",
"@types/react": "^18.3.3",
"@types/react-dom": "^18.3.0",
"@web3modal/ethers": "^4.2.3",
"ethers": "^6.12.1",
"react": "^18.3.1",
"react-dom": "^18.3.1",
"react-scripts": "5.0.1",
"typescript": "^4.9.5",
"web-vitals": "^2.1.4"

tuannx avatar Jun 02 '24 00:06 tuannx

APKT-543 [bug]

linear[bot] avatar Jun 02 '24 00:06 linear[bot]

Same here

peteAhn avatar Jun 20 '24 05:06 peteAhn

same issue here

kr-arunp avatar Jun 26 '24 04:06 kr-arunp

also, on mainnet!

VajraK avatar Jul 02 '24 09:07 VajraK

Is something not functioning properly or are you guys just seeing this error in the console?

chris13524 avatar Jul 07 '24 00:07 chris13524

it happens on https://lab.web3modal.com/ with rabby wallet on chrome

whichghost avatar Jul 07 '24 02:07 whichghost

So, as of now, it's been 5 weeks with seemingly no activity or assigned user, I messaged about it in Discord and was ignored, is it super borked, not a priority, gonna take another week, can we get some communication here?

ZachHandley avatar Jul 08 '24 21:07 ZachHandley

@ZachHandley I replied to the thread above

Is something broken or not-functional with the UI or SDK?

Or is the concern only that there are errors in the console? Our API returns 404s when things don't exist, but this is expected/normal behavior.

chris13524 avatar Jul 08 '24 21:07 chris13524

Yeah my balances don't show, which isn't a gigantic deal, though it definitely should be fixed cause it's not very clear (but it says I have money here, so do I need to transfer in? idk users are not the smartest), but I'm also curious what the profile is (the image for instance) and how to reduce the 2-4 errors in the console

my bad. I did not see your response, at all, sorry for the momentary crusade lmao

ZachHandley avatar Jul 08 '24 22:07 ZachHandley

I loaded the example from the issue description and my balance displays fine:

image

/profile is for loading information about ENS name registrations, it is unrelated to the balance request.

@ZachHandley can you please open up a new issue/discussion for your balance not showing issue and include a MRE?


We are planning a new API revision that will remove the use of 4xx's across our APIs for normal conditions, which will avoid the console errors seen here. Tracking issue here.

Closing this issue as it seems like the problem is exclusively the console errors and no functionality is broken.

chris13524 avatar Jul 08 '24 22:07 chris13524

Closing this issue as it seems like the problem is exclusively the console errors and no functionality is broken.

AppKit is still sending an invalid request so I think still needs fixing if I'm not mistaken.

glitch-txs avatar Jul 22 '24 10:07 glitch-txs

AppKit is still sending an invalid request so I think still needs fixing if I'm not mistaken.

@glitch-txs please elaborate on what is broken. Why do you think it is sending an invalid request?

chris13524 avatar Jul 22 '24 20:07 chris13524

@chris13524 @glitch-txs I'm following up with the current issue.

It's actually related to the issue https://github.com/WalletConnect/web3modal/issues/1888 I'm expecting a response when possible. Thanks a lot for your help in advance. I'm happy to highlight or elaborate on the issue if needed.

Thanks!

Victor-Salomon avatar Jul 26 '24 13:07 Victor-Salomon

i'm having the same issue. Any update or solution?

theMacAli avatar Sep 05 '24 02:09 theMacAli

This is not longer an issue. Please update to latest.

rtomas avatar Dec 11 '24 14:12 rtomas