ipfs / pinning-services-api-spec

Standalone, vendor-agnostic Pinning Service API for IPFS ecosystem
https://ipfs.github.io/pinning-services-api-spec/
Creative Commons Zero v1.0 Universal
100 stars 27 forks source link

Make providers a required field #24

Closed lidel closed 4 years ago

lidel commented 4 years ago

This PR adds Pin.providers and PinStatus.providers as lists of max 20 multiaddrs.

The goal is to encourage pinning services to provide own multiaddrs in pin response under PinStatus.providers to ensure clients behind restrictive NAT can dial back and reach pinning service to facilitate data transfer.

As per https://github.com/ipfs/pinning-services-api-spec/issues/22#issuecomment-657788004 https://github.com/ipfs/pinning-services-api-spec/issues/22#issuecomment-657793428

Closes #22 cc @obo20

lidel commented 4 years ago
lidel commented 4 years ago

I believe this is good enough to be merged.