ipfs-shipyard / pinning-service-compliance

This repo checks the compliance of IPFS Pinning Services against the pinning spec
https://ipfs-shipyard.github.io/pinning-service-compliance/
Other
14 stars 10 forks source link

Feature Request: Make it easier to communicate compliance failures to pinning service providers #125

Closed SgtPooki closed 2 years ago

SgtPooki commented 2 years ago

In order to keep manual effort to a minimum, we should automate communication of compliance failures as much as possible. There are a few methods I've thought about to accomplish this:


Pinning service provider repos and/or URLs:

SgtPooki commented 2 years ago

Got a response from Pinata:

hey [ @SgtPooki ] we don’t have an official place for this right now, but an issue on the actual pinning services spec repo might be helpful here - @obo20

Also related is #140 so that services could add checks into their CI/CD workflow.

SgtPooki commented 2 years ago

@lidel are you cool with me opening pinning compliance issues for Pinata on the spec repo, or should I open them here?

SgtPooki commented 2 years ago

For now, we're going to point providers towards https://github.com/ipfs-shipyard/pinning-service-compliance/issues/140, and have them handle their own destiny. We will, as a one-time only thing, open a single issue in the appropriate repositories to let providers know of any failures, but after that it will be up to providers to handle the aforementioned destiny.

lidel commented 2 years ago

@SgtPooki I think your approach is sensible :+1:

Providing NPM package that pinning services can run on their own is more than enough to run in any CI setup services have:

npx @ipfs-shipyard/pinning-service-compliance -s <endpoint> <auth>

I like the idea of going the extra mile and doing one-time ping + providing Github Action (#140) , but don't think we should be doing more handholding than that :-)

SgtPooki commented 2 years ago

Cool thanks for the second set of eyes :)