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

Add Crust pinning service to compliance checks #154

Open
2 tasks
SgtPooki opened this issue Jul 18, 2022 · 7 comments · Fixed by #162
Open
2 tasks

Add Crust pinning service to compliance checks #154

SgtPooki opened this issue Jul 18, 2022 · 7 comments · Fixed by #162
Labels
area/pinning service provider An issue having to do with pinning service providers released

Comments

@SgtPooki
Copy link
Member

SgtPooki commented Jul 18, 2022

See ipfs/ipfs-webui#1854

In order to add a new pinning service provider the changes should be made in the following locations:

@SgtPooki SgtPooki added the area/pinning service provider An issue having to do with pinning service providers label Jul 18, 2022
@badkk
Copy link
Contributor

badkk commented Aug 3, 2022

Hey @SgtPooki , PR already been created, for the SECRETS:

  1. For CRUST_API_ENDPOINT, you can use the public endpoint, which is https://pin.crustcode.com/psa
  2. For CRUST_API_TOKEN, you can follow this article to generate yourself: https://wiki.crust.network/docs/en/buildIPFSW3AuthPin#usage

@github-actions
Copy link

🎉 This issue has been resolved in version 1.3.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@SgtPooki
Copy link
Member Author

@badkk it looks crust is failing all checks. is this expected? https://ipfs-shipyard.github.io/pinning-service-compliance/pin.crustcode.com.html

@SgtPooki SgtPooki reopened this Sep 23, 2022
@badkk
Copy link
Contributor

badkk commented Sep 26, 2022

@badkk it looks crust is failing all checks. is this expected? https://ipfs-shipyard.github.io/pinning-service-compliance/pin.crustcode.com.html

Hey @SgtPooki I saw you merged the PR, do you still got question?

@SgtPooki
Copy link
Member Author

@badkk yes, why is crust failing all of the compliance checks? Ideally, we would want to ensure crust is somewhat compliant with the pinning service spec prior to resolving ipfs/ipfs-webui#1854

@SgtPooki
Copy link
Member Author

@badkk Crust still seems to be failing all checks with "reason": "UNAUTHORIZED", "details": "Invalid Signature" even though the signature was generated using previously given instructions.

Please let us know if there is some account setup or something that needs to be done. Otherwise, we will be removing crust from the compliance checker.

@badkk
Copy link
Contributor

badkk commented Oct 10, 2023

@badkk Crust still seems to be failing all checks with "reason": "UNAUTHORIZED", "details": "Invalid Signature" even though the signature was generated using previously given instructions.

Please let us know if there is some account setup or something that needs to be done. Otherwise, we will be removing crust from the compliance checker.

Would you mind to test w/this example?

curl --location 'https://pin.crustcode.com/psa/pins' \
--header 'authorization: Bearer c3ViLWNUR01xVHdjck5FZnJ1VVV2SG1pS012MnVnSzJiemM1d2p0TmI3SkNxM0R0UG04c3Y6MHgzODU1ZDNkY2Y0OTYwYjhhMzY3MmEzYTRmNGRiYjNhMWFjMDkyNDNhODdmNDgxOTdmZmUxNTRkNDExZmUzZjQ2MDhhZDY4ODUzOGNmMTk3MWE2Zjg5NTVhZjJmOWFjYzkwYjdiYzZlZGVkMzBjNmIyZGM2NGZhZGI4ZGQ1MmM4Ng==' \
--header 'Content-Type: application/json' \
--data '{
    "cid": "QmaK2WnUQ8FPVq3gGhHZi7VuWt3ikkrCCnPcqjFUg7vQDP",
    "name": "crust-evm-arch"
}'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/pinning service provider An issue having to do with pinning service providers released
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants