New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Auth: drop broken support for packet-specific SOA replies from backends #5512
Conversation
modules/luabackend/minimal.cc
Outdated
@@ -176,12 +176,12 @@ bool LUABackend::get(DNSResourceRecord &rr) { | |||
return !rr.content.empty(); | |||
} | |||
|
|||
bool LUABackend::getSOA(const DNSName &name, SOAData &soadata, DNSPacket *p) { | |||
bool LUABackend::getSOA(const string &name, SOAData &soadata) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
string
?!
modules/luabackend/minimal.cc
Outdated
@@ -176,12 +176,12 @@ bool LUABackend::get(DNSResourceRecord &rr) { | |||
return !rr.content.empty(); | |||
} | |||
|
|||
bool LUABackend::getSOA(const DNSName &name, SOAData &soadata, DNSPacket *p) { | |||
bool LUABackend::getSOA(const string &name, SOAData &soadata) { | |||
dnspacket = NULL; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LUABackend::dnspacket
can be removed completely now.
1bd9e3f
to
790ba6d
Compare
4dbb7a0
to
8e15bfc
Compare
After this force-push battle... my commit does compile now and also sets |
|
LGTM |
auth: scopeMask in the SOAData structure is unused after #5512
Short description
Remove DNSPacket pointer from getSOA. This was never okay to use (at least with caching enabled).
Split from #4545 per @Habbie's wish.
Checklist
I have: