-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Enable NFT pinning flag #29017
Labels
feature/web3/ipfs
feature/web3/wallet/core
feature/web3/wallet
Integrating Ethereum+ wallet support
OS/Desktop
priority/P3
The next thing for us to work on. It'll ride the trains.
QA Pass-Linux
QA Pass-macOS
QA Pass-Win64
QA/Yes
release-notes/include
Milestone
Comments
cypt4
added
OS/Desktop
feature/web3/ipfs
feature/web3/wallet
Integrating Ethereum+ wallet support
labels
Mar 13, 2023
jamesmudgett
added
the
priority/P3
The next thing for us to work on. It'll ride the trains.
label
Mar 17, 2023
25 tasks
cypt4
added a commit
to brave/brave-core
that referenced
this issue
Mar 27, 2023
Security & privacy review done in https://github.com/brave/security/issues/1182 |
The above requires |
Verification passed on
Verified steps from issue descriptionEnabling the feature - Passed
Pinning - Passed
Unpinning - PassedPins Validation - Passed
Disable feature - Passed
Cleanup - PassedLocal node behaviour - Passed
Error handling - Passed
Verification passed on
Verified steps from issue descriptionEnabling the feature - Passed
Pinning - Passed
Unpinning - PassedPins Validation - Passed
Disable feature - Passed
Cleanup - PassedLocal node behaviour - Passed
Error handling - Passed
Verification passed on
Verified steps from issue descriptionEnabling the feature - Passed
Pinning - Passed
Unpinning - PassedPins Validation - Passed
Disable feature - Passed
Cleanup - PassedLocal node behaviour - Passed
Error handling - Passed
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
feature/web3/ipfs
feature/web3/wallet/core
feature/web3/wallet
Integrating Ethereum+ wallet support
OS/Desktop
priority/P3
The next thing for us to work on. It'll ride the trains.
QA Pass-Linux
QA Pass-macOS
QA Pass-Win64
QA/Yes
release-notes/include
Figma: https://www.figma.com/file/4e7rsWzBUKv3dCdxCqB6CR/NFT-IPFS?t=afIihL0uQG3FO3gV-0
Related issues:
#19283
#26828
#28304
#28775
#28928
#28233
#28073
#28169
#28986
Related description(sec review):
Testing plan:
Enabling the feature
Pinnable NFTs are NFTs which have ipfs:// scheme for metadata and for image.
For example BAYC tokens are pinnable https://opensea.io/assets/ethereum/0xbc4ca0eda7647a8ab7c2061c2e118a18a936f13d/4352
Also metadata and image may have gateway-like url format. Such tokens are also pinnable
https://opensea.io/assets/ethereum/0x9401518f4ebba857baa879d9f76e1cc8b31ed197/400
Flow is described here : https://www.figma.com/file/4e7rsWzBUKv3dCdxCqB6CR/NFT-IPFS?node-id=1660%3A95153&t=CAsaMQ99tSq1M9NW-0
Pinning
https://www.figma.com/file/4e7rsWzBUKv3dCdxCqB6CR/NFT-IPFS?node-id=1044%3A102005&t=CAsaMQ99tSq1M9NW-0
curl http://127.0.0.1:<api port could be viewed at chrome://ipfs-internal>/api/v0/pin/ls?type=recursive
while running the Kubo daemon.
Result should contain metadata CIDs and image CIDs of the pinned tokens.
Unpinning
curl http://127.0.0.1:<api port could be viewed at chrome://ipfs-internal>/api/v0/pin/ls?type=recursive
NOTE: Some of NFTs may share CIDs, so CID won't be unpinned until all related NFTs are removed.
Pins validation
curl http://127.0.0.1:<api port could be viewed at chrome://ipfs-internal>/api/v0/pin/ls?type=recursive
should contain them.Disabling the feature
12) Disabling the feature should lead to hiding pinning status or pinning-related UI on the wallet page.
13) Tokens are not unpinned unless manually cleaned up.
Cleanup
14) After feature is disabled in the settings, there should be an option to remove all pinned items. Cleanup should be verified using
curl http://127.0.0.1:<api port could be viewed at chrome://ipfs-internal>/api/v0/pin/ls?type=recursive
.Local node behaviour
15) Check whether local node is automatically started after browser starts if the NFT pinning setting is enabled and not started otherwise.
Errors handling
16) Content type of NFT's image url should be "image/*". Request is made to https:///ipfs/CID and metadata field content-type or mime-type is read. You can test behavior by sniffing such requests as described here brave/brave-core#17392 (comment)
17) Pinning may fail due some reasons. It would be restarted after a time or on the next browser start. You can fail pinning by turning internet off while pinning is in progress and then test if it is continued after restart.
The text was updated successfully, but these errors were encountered: