multiformats / multicodec

Compact self-describing codecs. Save space by using predefined multicodec tables.
MIT License
334 stars 200 forks source link

Add a transport code to indicate content is provided via Trustless HTTP Gateway #321

Closed willscott closed 1 year ago

willscott commented 1 year ago

per the trustless gateway spec

The exact semantics of this spec are evolving, but might as well reserve a code that we can use for indicating the existence of an HTTP endpoint for index providers.

rvagg commented 1 year ago

how will this code end up getting used in the network chatter?

willscott commented 1 year ago

how will this code end up getting used in the network chatter?

my expectation is that this will be used in the same context as transport-bitswap and transport-filecoin-graphsync-v1 - it will be a code understood in context routing contexts to indicate that this publisher can provide data over http.

rvagg commented 1 year ago

The only remaining hold-up here is the use of "gateway" unadorned, since that has a specific meaning for us but not all audiences of this table, ipfs-gateway would seem to be the appropriate way to solve that. Other than that it's good to go I think. I don't know about the comments about "CAR" but since that's in the description, it can be easily changed later if necessary.

hannahhoward commented 1 year ago

I'd support adding IPFS -- @willscott ?

willscott commented 1 year ago

updated name to be less ambiguous

willscott commented 1 year ago

I've updated the comment per @aschmahmann 's request. @rvagg i don't think we're waiting for anything else here and I believe you're the one with relevant approval powers

lidel commented 1 year ago

Another comment is that trustless gateway basic primitive is a block, not a CAR. CAR is only means of sending more than a single block - important distinction, because there is no trustless gateway without application/vnd.ipld.raw