estuary icon indicating copy to clipboard operation
estuary copied to clipboard

Unify shuttle, shuttle-proxy and main estuary APIs

Open gmelodie opened this issue 3 years ago • 2 comments

We shouldn't need to maintain the same API on three separate packages. We should have a way to declare the estuary API and then, on another file, just specify which endpoints will be available on the Shuttle API and which will be on the shuttle-proxy API.

gmelodie avatar Jun 09 '22 13:06 gmelodie

Hey @whyrusleeping, do you think this makes sense to do? And who do you think is the best person to delegate this to?

corinne-antonia avatar Jun 20 '22 15:06 corinne-antonia

I was thinking maybe having a base API communicator which implements endpoints necessary to talk to other players (such as a shuttle or a main node), then we'll have files for methods specific to these communicators/players (like a shuttle_api.go).

P.S.: just writing this comment makes me feel like this is overengineering the thing. What I wanted originally is to solve the hassle of having to replicate code for shuttle, proxy and main node. That's what we need to fix

gmelodie avatar Jun 28 '22 15:06 gmelodie