application-research / estuary-www

https://estuary.tech
https://estuary.tech
Other
35 stars 31 forks source link

Show both estuary and dweb retrieval urls #92

Closed neelvirdy closed 1 year ago

neelvirdy commented 1 year ago

~Replaces any link that is intended to point to estuary-hosted content with the estuary gw instead of dweb.~ Shows both gateway links throughout the UI, labeled as "Estuary retrieval url" and "Dweb retrieval url". The estuary gw defers to dweb if the cid is not actively pinned by estuary or has been offloaded, so it is safe to point to the estuary gw as long as estuary API nodes are up. Dweb is still provided as a back up option until we decide to only show the estuary url.

Note: For pinned directory nodes, users will get a much faster response on average through the estuary url but a much less dressed-up gateway UI, until we update our gateway rendering.


Screen Shot 2022-11-10 at 12 39 16 PM image image image image
alvin-reyes commented 1 year ago

I say we should put both links (dweb and our gateway) for now until we stablize the gateway. There are instances where the gateway can't serve some old data.

Once we have the infra stable, we can remove the dweb link.

neelvirdy commented 1 year ago

I say we should put both links (dweb and our gateway) for now until we stablize the gateway. There are instances where the gateway can't serve some old data.

Once we have the infra stable, we can remove the dweb link.

Is the main concern estuary uptime, or the gateway implementation itself? On uptime - can we add a DNS failover to hit dweb.link if api.estuary.tech/gw is not responding?

On gateway implementation - is there a case where estuary thinks it has data available (according to db) but is still unable to serve it correctly? If so and we can't clean it up quickly, then agreed we should put both links. Otherwise it would seem to me like we could fully switch to our gateway if the DNS failover is possible.

alvin-reyes commented 1 year ago

I say we should put both links (dweb and our gateway) for now until we stablize the gateway. There are instances where the gateway can't serve some old data. Once we have the infra stable, we can remove the dweb link.

Is the main concern estuary uptime, or the gateway implementation itself? On uptime - can we add a DNS failover to hit dweb.link if api.estuary.tech/gw is not responding?

On gateway implementation - is there a case where estuary thinks it has data available (according to db) but is still unable to serve it correctly? If so and we can't clean it up quickly, then agreed we should put both links. Otherwise it would seem to me like we could fully switch to our gateway if the DNS failover is possible.

Right now, there is a case for that because of the corrupted lmdb. There are CIDs that are available and some that are not. Until we fix this (or AR has fully indexed all CIDs), we shouldn't solely use our gw. Add both dweb and the gw for now so that our users have options.

neelvirdy commented 1 year ago

gonna push a change soon to make formatting utils and just call them, i.e. U.formatEstuaryRetrievalUrl(cid) and U.formatDwebRetrievalUrl(cid)

neelvirdy commented 1 year ago

removed waiting condition since dweb link is also still available now