-
Notifications
You must be signed in to change notification settings - Fork 11
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
Various fixes in named.conf #55
Conversation
a better test case for the whitespace part
json is as currently parsed, that is, wrong
pod2html -o doc.html src/modules/DnsData.pm
# @return [Array<String>] | ||
def current_zone_allow_transfer | ||
items = [] | ||
@current_zone.fetch("options", []).each do |m| |
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.
I wasn't able to find explicit definition / initialization of @current_zone
. Is it an intention?
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.
Ah, good point. It is in DnsServerUi.rb which is an "envelope" module defining the variables and including the parts to work on them, dialog-masterzone.rb is one of them.
besides that it looks fine for me |
It looks also good to me |
bsc#976643 (L3)
https://trello.com/c/wSXpTL68
It's a follow-up for the fix for trailing whitespace (#54).
Fixed
{
would mean the rest of the file was misparsed (interpreted it as a beginning of a section)allow-transfer
, shown in zone editation Basics tab as "Enable zone transport", unnamed addresses (that do not have anacl
declaration) would be droppedTo Do:
addition of a zone record was lost(just a misunderstanding in BZ)