search

nigoroll / libvmod-dynamic

The Varnish dns/named director continued
BSD 2-Clause "Simplified" License
95 stars 34 forks source link

shell -> feature cmd #98

Closed gquintard closed 1 year ago

gquintard commented 1 year ago

hopefuly, it's the correct fix for https://github.com/nigoroll/libvmod-dynamic/issues/97

nigoroll commented 1 year ago

the || true dues not make much sense with feature, does it?

gquintard commented 1 year ago

None whatsoever :-)

On Fri, Mar 17, 2023, 02:04 Nils Goroll @.***> wrote:

the || true dues not make much sense with feature, does it?

— Reply to this email directly, view it on GitHub https://github.com/nigoroll/libvmod-dynamic/pull/98#issuecomment-1473457364, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA42AKJ55D3AFWD2FCBTWKTW4QSITANCNFSM6AAAAAAV6CO5VM . You are receiving this because you authored the thread.Message ID: @.***>

gquintard commented 1 year ago

updated to actually make sense

gquintard commented 1 year ago

@nigoroll , can we have that in soon please? Sorry to rush, but the docker images need it and I'd rather build from this repo (I could point to mine, but using the official one is better)

gquintard commented 1 year ago

Thank you Nils, the images are live! https://hub.docker.com/_/varnish