Join GitHub today
GitHub is home to over 28 million developers working together to host and review code, manage projects, and build software together.Sign up
dnsdist: avoid LuaWrapper assert on errors in newServer() #6403
Calling newServer with something like "foo:53" would previously end up like this (at least in assert-enabled builds):
After this change:
IIRC running newServer from a remote console this would also crash the daemon process.
I have to say that starting up with the invalid newServer is bad, but judging by the code this was the intended behaviour.
I've pushed the