caddy-l4 icon indicating copy to clipboard operation
caddy-l4 copied to clipboard

[FEATURE]: Add support for lookup_srv for upstreams discovery

Open danlsgiga opened this issue 4 years ago • 6 comments

The same way as the caddy http app, it would be nice to have not only the dial option but also the lookup_srv for upstream service discovery.

danlsgiga avatar Jan 06 '21 16:01 danlsgiga

I like https://github.com/lucaslorentz/caddy-docker-proxy label configuration. Something similiar would be awesome

pwFoo avatar Jul 06 '21 14:07 pwFoo

Docker labels are out of scope for this plugin.

francislavoie avatar Jul 06 '21 14:07 francislavoie

So not label based or entire feature of auto generated config by other containers labels / env?

pwFoo avatar Jul 06 '21 14:07 pwFoo

caddy-l4 has nothing to do with Docker. It's not in scope for this plugin.

francislavoie avatar Jul 06 '21 14:07 francislavoie

It would be great to see SRV support for dynamic upstreams here. Awesome that caddy supports SRV based lookups on the http side already. I'm trying to wrap my head around why so few proxy products support SRV lookups. AFAICT nginx (non-commercial) doesn't, traefik doesn't (equivalent functionality needs to use a DCS, like Consul, for service discovery), envoy doesn't even seem to yet.

Since the http app already has the SRV lookup feature (dynamic srv stanza in reverse_proxy for example), maybe the level of effort wouldn't be huge to re-use a good part of that for the tcp app?

johnalotoski avatar Sep 12 '22 13:09 johnalotoski

@johnalotoski Yeah, this should be pretty straightforward to port over from the http app now.

mholt avatar Sep 12 '22 14:09 mholt